[PATCH v2 2/3] drm/bridge: Add ti-tfp410 DVI transmitter driver
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Thu Nov 17 09:45:12 UTC 2016
Hi Jyri,
On Wednesday 16 Nov 2016 16:39:28 Jyri Sarha wrote:
> On 11/16/16 15:33, Rob Herring wrote:
> >> +Optional properties
> >>
> >>> + - reg: I2C address. If and only if present the driver node
I assume you meant device node, not driver node ?
> >>> + should be placed into the i2c controller node where the
> >>> + tfp410 i2c is connected to (the current implementation does
> >>> + not yet support this).
> >
> > So this chip can work without programming I guess?
>
> Yes. Just powering it up is enough for most application.
>
> > reg should only be not present if I2C is not connected in the design. It
> > can't be a function of what the driver supports. In otherwords, you
> > can't be moving this node around based on when you add I2C control.
>
> Ok, I'll try to implement a dummy i2c driver at the same time too. I can
> not test anything related to it because I do not have a piece of HW with
> tfp410 i2c wires connected, but it should not matter as long as I am
> able to probe it as a i2c client.
I think that Rob's point was that whether the current implementation supports
this or not is irrelevant from a DT bindings point of view. It should not be
mentioned in the bindings document.
--
Regards,
Laurent Pinchart
More information about the dri-devel
mailing list