From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yakir Yang Subject: Re: [RFC PATCH v2 4/4] dt-bindings: add document for rk3229-hdmi Date: Mon, 11 Jan 2016 11:48:15 +0800 Message-ID: <569325FF.10902@rock-chips.com> References: <1452156811-18150-1-git-send-email-ykk@rock-chips.com> <1452157511-18891-1-git-send-email-ykk@rock-chips.com> <20160111020618.GA20680@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160111020618.GA20680@rob-hp-laptop> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring Cc: Mark Yao , Heiko Stuebner , Russell King , Philipp Zabel , Andy Yan , David Airlie , Kumar Gala , Zheng Yang , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-rockchip.vger.kernel.org Hi Rob, On 01/11/2016 10:06 AM, Rob Herring wrote: > On Thu, Jan 07, 2016 at 05:05:11PM +0800, Yakir Yang wrote: >> Signed-off-by: Yakir Yang >> --- >> Changes in v2: None >> >> .../devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> index 668091f..1cdc627 100644 >> --- a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> +++ b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> @@ -2,7 +2,7 @@ Rockchip specific extensions to the Synopsys Designware HDMI >> ================================ >> >> Required properties: >> -- compatible: "rockchip,rk3288-dw-hdmi"; >> +- compatible: "rockchip,rk3288-dw-hdmi", "rockchip,rk3229-dw-hdmi"; >> - reg: Physical base address and length of the controller's registers. >> - clocks: phandle to hdmi iahb and isfr clocks. >> - clock-names: should be "iahb" "isfr" >> @@ -15,8 +15,10 @@ Required properties: >> rk3288 platform >> >> Optional properties >> +- reg: Physical base address and length of the external HDMI PHY's registers. > This is already required... > The original reg only include the DW HDMI controller registers: reg = <0x200a0000 0x20000>; But for now, I need to pass the external HDMI PHY registers: reg = <0x200a0000 0x20000>, <0x12030000 0x10000>; >> - ddc-i2c-bus: phandle of an I2C controller used for DDC EDID probing >> - clocks, clock-names: phandle to the HDMI CEC clock, name should be "cec" >> + phandle to the external HDMI PHY clock, name should be "extphy" > Does this change apply to rk3288 or just rk3229? If the latter, you > should make that clear. The driver needs to allow for old dtb's that > only have 1 clock. It's the latter one, driver have allowed that by judging the device type, but I should make the document more clear. > Based on Philipp's feedback, you also need to add a phys property. Yes, I'm preparing the new patches that create a new separate phy driver for external HDMI phy, there would be much difference from this version. Thanks for your respond, - Yakir > > Rob > > > -- 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: ykk@rock-chips.com (Yakir Yang) Date: Mon, 11 Jan 2016 11:48:15 +0800 Subject: [RFC PATCH v2 4/4] dt-bindings: add document for rk3229-hdmi In-Reply-To: <20160111020618.GA20680@rob-hp-laptop> References: <1452156811-18150-1-git-send-email-ykk@rock-chips.com> <1452157511-18891-1-git-send-email-ykk@rock-chips.com> <20160111020618.GA20680@rob-hp-laptop> Message-ID: <569325FF.10902@rock-chips.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Rob, On 01/11/2016 10:06 AM, Rob Herring wrote: > On Thu, Jan 07, 2016 at 05:05:11PM +0800, Yakir Yang wrote: >> Signed-off-by: Yakir Yang >> --- >> Changes in v2: None >> >> .../devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> index 668091f..1cdc627 100644 >> --- a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> +++ b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> @@ -2,7 +2,7 @@ Rockchip specific extensions to the Synopsys Designware HDMI >> ================================ >> >> Required properties: >> -- compatible: "rockchip,rk3288-dw-hdmi"; >> +- compatible: "rockchip,rk3288-dw-hdmi", "rockchip,rk3229-dw-hdmi"; >> - reg: Physical base address and length of the controller's registers. >> - clocks: phandle to hdmi iahb and isfr clocks. >> - clock-names: should be "iahb" "isfr" >> @@ -15,8 +15,10 @@ Required properties: >> rk3288 platform >> >> Optional properties >> +- reg: Physical base address and length of the external HDMI PHY's registers. > This is already required... > The original reg only include the DW HDMI controller registers: reg = <0x200a0000 0x20000>; But for now, I need to pass the external HDMI PHY registers: reg = <0x200a0000 0x20000>, <0x12030000 0x10000>; >> - ddc-i2c-bus: phandle of an I2C controller used for DDC EDID probing >> - clocks, clock-names: phandle to the HDMI CEC clock, name should be "cec" >> + phandle to the external HDMI PHY clock, name should be "extphy" > Does this change apply to rk3288 or just rk3229? If the latter, you > should make that clear. The driver needs to allow for old dtb's that > only have 1 clock. It's the latter one, driver have allowed that by judging the device type, but I should make the document more clear. > Based on Philipp's feedback, you also need to add a phys property. Yes, I'm preparing the new patches that create a new separate phy driver for external HDMI phy, there would be much difference from this version. Thanks for your respond, - Yakir > > Rob > > > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758188AbcAKDsX (ORCPT ); Sun, 10 Jan 2016 22:48:23 -0500 Received: from lucky1.263xmail.com ([211.157.147.131]:38146 "EHLO lucky1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757937AbcAKDsV (ORCPT ); Sun, 10 Jan 2016 22:48:21 -0500 X-263anti-spam: KSV:0; X-MAIL-GRAY: 1 X-MAIL-DELIVERY: 0 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-ADDR-CHECKED: 0 X-RL-SENDER: ykk@rock-chips.com X-FST-TO: linux-arm-kernel@lists.infradead.org X-SENDER-IP: 103.29.142.67 X-LOGIN-NAME: ykk@rock-chips.com X-UNIQUE-TAG: X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Subject: Re: [RFC PATCH v2 4/4] dt-bindings: add document for rk3229-hdmi To: Rob Herring References: <1452156811-18150-1-git-send-email-ykk@rock-chips.com> <1452157511-18891-1-git-send-email-ykk@rock-chips.com> <20160111020618.GA20680@rob-hp-laptop> Cc: Mark Yao , Heiko Stuebner , Russell King , Philipp Zabel , Andy Yan , David Airlie , Kumar Gala , Zheng Yang , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org From: Yakir Yang Message-ID: <569325FF.10902@rock-chips.com> Date: Mon, 11 Jan 2016 11:48:15 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20160111020618.GA20680@rob-hp-laptop> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 01/11/2016 10:06 AM, Rob Herring wrote: > On Thu, Jan 07, 2016 at 05:05:11PM +0800, Yakir Yang wrote: >> Signed-off-by: Yakir Yang >> --- >> Changes in v2: None >> >> .../devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> index 668091f..1cdc627 100644 >> --- a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> +++ b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> @@ -2,7 +2,7 @@ Rockchip specific extensions to the Synopsys Designware HDMI >> ================================ >> >> Required properties: >> -- compatible: "rockchip,rk3288-dw-hdmi"; >> +- compatible: "rockchip,rk3288-dw-hdmi", "rockchip,rk3229-dw-hdmi"; >> - reg: Physical base address and length of the controller's registers. >> - clocks: phandle to hdmi iahb and isfr clocks. >> - clock-names: should be "iahb" "isfr" >> @@ -15,8 +15,10 @@ Required properties: >> rk3288 platform >> >> Optional properties >> +- reg: Physical base address and length of the external HDMI PHY's registers. > This is already required... > The original reg only include the DW HDMI controller registers: reg = <0x200a0000 0x20000>; But for now, I need to pass the external HDMI PHY registers: reg = <0x200a0000 0x20000>, <0x12030000 0x10000>; >> - ddc-i2c-bus: phandle of an I2C controller used for DDC EDID probing >> - clocks, clock-names: phandle to the HDMI CEC clock, name should be "cec" >> + phandle to the external HDMI PHY clock, name should be "extphy" > Does this change apply to rk3288 or just rk3229? If the latter, you > should make that clear. The driver needs to allow for old dtb's that > only have 1 clock. It's the latter one, driver have allowed that by judging the device type, but I should make the document more clear. > Based on Philipp's feedback, you also need to add a phys property. Yes, I'm preparing the new patches that create a new separate phy driver for external HDMI phy, there would be much difference from this version. Thanks for your respond, - Yakir > > Rob > > >