[PATCH v3 2/2] drm: bridge/dw_hdmi: add dw hdmi i2c bus adapter support
Doug Anderson
dianders at chromium.org
Wed Sep 2 16:00:07 PDT 2015
Russell,
On Wed, Sep 2, 2015 at 3:50 PM, Russell King - ARM Linux
<linux at arm.linux.org.uk> wrote:
>> I think you may want to add "adap->dev.of_node = hdmi->dev->of_node;"
>> here. That will allow device trees to specify the i2c bus by using an
>> alias.
>
> Never copy the of_node from one device to another. That allows the
> bus matching to unintentionally match the of_node against the wrong
> driver.
So does that mean we need to create a sub-node for the i2c bus if we
want something like this?
> Also, is it appropriate to hook non-DDC devices to a DDC bus? I suspect
> that's asking for trouble.
I doubt it's appropriate. Why do you ask?
-Doug
More information about the dri-devel
mailing list