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

2015-09-13 Thread Heiko Stübner
Am Mittwoch, 2. September 2015, 14:25:48 schrieb Douglas Anderson: > The ddc-i2c-bus property was missing from the veyron dtsi file since > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > nobody noticed when the veyron dtsi was sent upstream. Add it. > > Signed-off-by: D

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

2015-09-03 Thread Brian Norris
On Thu, Sep 03, 2015 at 02:00:26PM -0700, Doug Anderson wrote: > On Wed, Sep 2, 2015 at 2:25 PM, Douglas Anderson > wrote: > > The ddc-i2c-bus property was missing from the veyron dtsi file since > > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > > nobody noticed when t

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

2015-09-03 Thread Heiko Stuebner
Am Donnerstag, 3. September 2015, 14:00:26 schrieb Doug Anderson: > Hi, > > On Wed, Sep 2, 2015 at 2:25 PM, Douglas Anderson wrote: > > The ddc-i2c-bus property was missing from the veyron dtsi file since > > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > > nobody noti

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

2015-09-03 Thread Doug Anderson
Hi, On Wed, Sep 2, 2015 at 2:25 PM, Douglas Anderson wrote: > The ddc-i2c-bus property was missing from the veyron dtsi file since > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > nobody noticed when the veyron dtsi was sent upstream. Add it. > > Signed-off-by: Douglas

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

2015-09-03 Thread Doug Anderson
Lucas, On Thu, Sep 3, 2015 at 9:13 AM, Lucas Stach wrote: >> 6. Once you start using the dw_hdmi's i2c block with the currently >> posted patch against mainline (to do this you not only need the patch >> but you need to remove the ddc-i2c-bus property, set the pinmux, and >> disable i2c5) then yo

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

2015-09-03 Thread Lucas Stach
Am Donnerstag, den 03.09.2015, 09:04 -0700 schrieb Doug Anderson: > Hi, > > On Thu, Sep 3, 2015 at 8:46 AM, Rob Herring wrote: > > On Thu, Sep 3, 2015 at 10:18 AM, Russell King - ARM Linux > > wrote: > >> On Thu, Sep 03, 2015 at 09:46:38AM -0500, Rob Herring wrote: > >>> Yes, that is fairly comm

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

2015-09-03 Thread Doug Anderson
Hi, On Thu, Sep 3, 2015 at 8:46 AM, Rob Herring wrote: > On Thu, Sep 3, 2015 at 10:18 AM, Russell King - ARM Linux > 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 cas

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

2015-09-03 Thread Doug Anderson
Hi, On Thu, Sep 3, 2015 at 8:18 AM, Russell King - ARM Linux 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 i

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

2015-09-03 Thread Rob Herring
On Thu, Sep 3, 2015 at 10:18 AM, Russell King - ARM Linux 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

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

2015-09-03 Thread Russell King - ARM Linux
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 t

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

2015-09-03 Thread Rob Herring
On Thu, Sep 3, 2015 at 3:22 AM, Russell King - ARM Linux wrote: > On Wed, Sep 02, 2015 at 07:13:24PM -0500, Rob Herring wrote: >> On Wed, Sep 2, 2015 at 4:25 PM, Douglas Anderson >> wrote: >> > The ddc-i2c-bus property was missing from the veyron dtsi file since >> > downstream the ddc-i2c-bus w

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

2015-09-03 Thread Russell King - ARM Linux
On Wed, Sep 02, 2015 at 07:13:24PM -0500, Rob Herring wrote: > On Wed, Sep 2, 2015 at 4:25 PM, Douglas Anderson > wrote: > > The ddc-i2c-bus property was missing from the veyron dtsi file since > > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > > nobody noticed when the

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

2015-09-02 Thread Doug Anderson
Rob, On Wed, Sep 2, 2015 at 5:13 PM, Rob Herring wrote: > On Wed, Sep 2, 2015 at 4:25 PM, Douglas Anderson > wrote: >> The ddc-i2c-bus property was missing from the veyron dtsi file since >> downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and >> nobody noticed when the veyro

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

2015-09-02 Thread Rob Herring
On Wed, Sep 2, 2015 at 4:25 PM, Douglas Anderson wrote: > The ddc-i2c-bus property was missing from the veyron dtsi file since > downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and > nobody noticed when the veyron dtsi was sent upstream. Add it. > > Signed-off-by: Douglas Ande

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

2015-09-02 Thread Douglas Anderson
The ddc-i2c-bus property was missing from the veyron dtsi file since downstream the ddc-i2c-bus was still being specified in rk3288.dtsi and nobody noticed when the veyron dtsi was sent upstream. Add it. Signed-off-by: Douglas Anderson --- Note: I noticed that this was wrong but I don't currentl