From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751409AbeAUVQg (ORCPT ); Sun, 21 Jan 2018 16:16:36 -0500 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:24978 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751224AbeAUVQd (ORCPT ); Sun, 21 Jan 2018 16:16:33 -0500 From: Philippe CORNU To: Rob Herring CC: Mark Rutland , David Airlie , "dri-devel@lists.freedesktop.org" , "Laurent Pinchart" , Mickael REULIER , Russell King , "Fabien DESSENNE" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , Alexandre TORGUE , Arnd Bergmann , Vincent ABRIOU , Ludovic BARRE , "linux-kernel@vger.kernel.org" , Yannick FERTRE , Maxime Coquelin , Gabriel FERNANDEZ , Bhumika Goyal Subject: Re: [PATCH v2] dt-bindings: display: stm32: add pixel clock mandatory property Thread-Topic: [PATCH v2] dt-bindings: display: stm32: add pixel clock mandatory property Thread-Index: AQHTkXb3kBYCc48JnUaAnYmdRunp8KN+xlWA Date: Sun, 21 Jan 2018 21:15:54 +0000 Message-ID: <3bd6bddd-db0d-2b5e-a676-2d56b7e5ca41@st.com> References: <20180112153034.20353-1-philippe.cornu@st.com> <20180119224335.exrxcfq3xc6op6iy@rob-hp-laptop> In-Reply-To: <20180119224335.exrxcfq3xc6op6iy@rob-hp-laptop> Accept-Language: fr-FR, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.47] Content-Type: text/plain; charset="utf-8" Content-ID: <94347A0566A1014285B96459D9DEA480@st.com> MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2018-01-21_10:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id w0LLGgKx012725 Hi Rob, On 01/19/2018 11:43 PM, Rob Herring wrote: > On Fri, Jan 12, 2018 at 04:30:34PM +0100, Philippe Cornu wrote: >> Add the DPI/RGB input pixel clock in mandatory properties >> because it really offers a better preciseness for timing >> computations. >> Note: Fix also the DSI panel example where "ref" & "pclk" >> clocks were swapped. >> >> Signed-off-by: Philippe Cornu >> --- >> Changes in v2: put new clock in last position (Rob Herring) >> >> Documentation/devicetree/bindings/display/st,stm32-ltdc.txt | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt >> index 029252253ad4..942b7237ae87 100644 >> --- a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt >> +++ b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt >> @@ -29,6 +29,7 @@ Mandatory properties specific to STM32 DSI: >> - compatible: "st,stm32-dsi". >> - clock-names: >> - phy pll reference clock string name, must be "ref". >> + - DPI/RGB input pixel clock string name, must be "px_clk". >> - resets: see [5]. >> - reset-names: see [5]. >> >> @@ -97,8 +98,9 @@ Example 2: DSI panel >> #size-cells = <0>; >> compatible = "st,stm32-dsi"; >> reg = <0x40016c00 0x800>; >> - clocks = <&rcc 1 CLK_F469_DSI>, <&clk_hse>; >> - clock-names = "ref", "pclk"; >> + clocks = <&rcc 1 CLK_F469_DSI>, <&clk_hse>, >> + <&rcc 1 CLK_LCD>; >> + clock-names = "pclk", "ref", "px_clk"; > > You have the existing names reversed here. And many thanks for your review. Names are "reversed" as explained in the note in the commit message. Well, maybe the note is not clear enough... So I have just sent 2 separate patches that I hope will be better to understand. Many thanks, Philippe : ) > >> resets = <&rcc STM32F4_APB2_RESET(DSI)>; >> reset-names = "apb"; >> >> -- >> 2.15.1 >> > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel >