Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron

From	Rob Herring <robherring2@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron
Date	2015-09-03 18:00 +0200
Message-ID	<q4F5E-83m-17@gated-at.bofh.it> (permalink)
References	<q4nLr-bH-5@gated-at.bofh.it> <q4qpX-3ZU-1@gated-at.bofh.it> <q4y4a-6wl-5@gated-at.bofh.it> <q4DZT-6wk-23@gated-at.bofh.it> <q4EsX-7kn-73@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Sep 3, 2015 at 10:18 AM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Thu, Sep 03, 2015 at 09:46:38AM -0500, Rob Herring wrote:
>> Yes, that is fairly common (ADV75xx is same), and we would not
>> describe an I2C bus in DT in that case. Same with HPD directly handled
>> vs. a GPIO line. That is no different than what Doug has said:
>> ddc-i2c-bus is present if using the SOC's I2C host and absent if using
>> the HDMI block's DDC functionality. I'm only questioning the location
>> of the property.
>
> No, I don't think that's what Doug wants.  Doug wants the bridge's
> internal I2C host to be exposed, so he can number it through a DT
> alias.

See his earlier reply and other patch[1] which states once the dw_hdmi
built-in I2C controller support is added in mainline, then this
property is not needed. For now, the SOC's general purpose I2C
controller is used.

Rob

[1] https://lkml.org/lkml/2015/9/2/571
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH] ARM: dts: Add ddc i2c reference to veyron Douglas Anderson <dianders@chromium.org> - 2015-09-02 23:30 +0200
  Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Rob Herring <robherring2@gmail.com> - 2015-09-03 02:20 +0200
    Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Doug Anderson <dianders@chromium.org> - 2015-09-03 02:30 +0200
    Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-09-03 10:30 +0200
      Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Rob Herring <robherring2@gmail.com> - 2015-09-03 16:50 +0200
        Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-09-03 17:20 +0200
          Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Rob Herring <robherring2@gmail.com> - 2015-09-03 18:00 +0200
            Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Doug Anderson <dianders@chromium.org> - 2015-09-03 18:10 +0200
              Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Lucas Stach <l.stach@pengutronix.de> - 2015-09-03 18:20 +0200
                Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Doug Anderson <dianders@chromium.org> - 2015-09-03 18:30 +0200
          Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Doug Anderson <dianders@chromium.org> - 2015-09-03 18:00 +0200
  Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Doug Anderson <dianders@chromium.org> - 2015-09-03 23:10 +0200
    Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Heiko Stuebner <heiko@sntech.de> - 2015-09-04 00:10 +0200
    Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Brian Norris <briannorris@chromium.org> - 2015-09-04 01:00 +0200

csiph-web