From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Stuebner Subject: Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Date: Fri, 04 Sep 2015 00:01:35 +0200 Message-ID: <5450060.vCTDljrI7r@phil> References: <1441229148-12095-1-git-send-email-dianders@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Anderson Cc: "open list:ARM/Rockchip SoC..." , Alexandru Stan , Brian Norris , Yakir Yang , =?utf-8?B?5aea5pm65oOF?= , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-rockchip.vger.kernel.org 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 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 currently have > > graphics up and running on upstream on veyron. Posting this anyway > > since it's pretty clear that it's needed. If someone else wants to > > try it out that'd be nice, otherwise I'll put it on my list to figure > > out how to get myself setup for graphics upstream. ;) > > OK, I've used Heiko's "somewhat stable" branch to test this against > something very close to mainline. for people playing along, that is [0]. Essentially bleeding edge mainline + some backports from other maintainer trees +the last bits in the works, like the eDP support. > Without my patch I can't read the HDMI EDID. With my patch I can. :) The change follows the current dt-binding in the kernel for the (rockchip- variant of the) dw_hdmi, so if nobody keeps protesting loudly, I'd like to pick this up to get the hdmi on Veyron devices going. And I guess we'll remove this again (also in the other boards), once the dw_hdmi-internal i2c stuff is included - so this should be temporary as Rob hoped. Heiko [0] https://github.com/mmind/linux-rockchip/tree/v4.2-rockchip-somewhat-stable -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko Stuebner) Date: Fri, 04 Sep 2015 00:01:35 +0200 Subject: [PATCH] ARM: dts: Add ddc i2c reference to veyron In-Reply-To: References: <1441229148-12095-1-git-send-email-dianders@chromium.org> Message-ID: <5450060.vCTDljrI7r@phil> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 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 currently have > > graphics up and running on upstream on veyron. Posting this anyway > > since it's pretty clear that it's needed. If someone else wants to > > try it out that'd be nice, otherwise I'll put it on my list to figure > > out how to get myself setup for graphics upstream. ;) > > OK, I've used Heiko's "somewhat stable" branch to test this against > something very close to mainline. for people playing along, that is [0]. Essentially bleeding edge mainline + some backports from other maintainer trees +the last bits in the works, like the eDP support. > Without my patch I can't read the HDMI EDID. With my patch I can. :) The change follows the current dt-binding in the kernel for the (rockchip- variant of the) dw_hdmi, so if nobody keeps protesting loudly, I'd like to pick this up to get the hdmi on Veyron devices going. And I guess we'll remove this again (also in the other boards), once the dw_hdmi-internal i2c stuff is included - so this should be temporary as Rob hoped. Heiko [0] https://github.com/mmind/linux-rockchip/tree/v4.2-rockchip-somewhat-stable From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757681AbbICWB5 (ORCPT ); Thu, 3 Sep 2015 18:01:57 -0400 Received: from gloria.sntech.de ([95.129.55.99]:46548 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752570AbbICWB4 (ORCPT ); Thu, 3 Sep 2015 18:01:56 -0400 From: Heiko Stuebner To: Doug Anderson Cc: "open list:ARM/Rockchip SoC..." , Alexandru Stan , Brian Norris , Yakir Yang , =?utf-8?B?5aea5pm65oOF?= , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] ARM: dts: Add ddc i2c reference to veyron Date: Fri, 04 Sep 2015 00:01:35 +0200 Message-ID: <5450060.vCTDljrI7r@phil> User-Agent: KMail/4.14.1 (Linux/4.1.0-1-amd64; KDE/4.14.2; x86_64; ; ) In-Reply-To: References: <1441229148-12095-1-git-send-email-dianders@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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 currently have > > graphics up and running on upstream on veyron. Posting this anyway > > since it's pretty clear that it's needed. If someone else wants to > > try it out that'd be nice, otherwise I'll put it on my list to figure > > out how to get myself setup for graphics upstream. ;) > > OK, I've used Heiko's "somewhat stable" branch to test this against > something very close to mainline. for people playing along, that is [0]. Essentially bleeding edge mainline + some backports from other maintainer trees +the last bits in the works, like the eDP support. > Without my patch I can't read the HDMI EDID. With my patch I can. :) The change follows the current dt-binding in the kernel for the (rockchip- variant of the) dw_hdmi, so if nobody keeps protesting loudly, I'd like to pick this up to get the hdmi on Veyron devices going. And I guess we'll remove this again (also in the other boards), once the dw_hdmi-internal i2c stuff is included - so this should be temporary as Rob hoped. Heiko [0] https://github.com/mmind/linux-rockchip/tree/v4.2-rockchip-somewhat-stable