From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Fri, 24 Jul 2015 14:47:03 +0200 Subject: [U-Boot] [PATCH 15/19] ARM: zynq: DT: Get rid of ps-clk-frequency In-Reply-To: <20150722155128.GT2531@xsjsorenbubuntu> References: <944923d793c411840ef909c229e472d5b680f318.1437579483.git.michal.simek@xilinx.com> <20150722155128.GT2531@xsjsorenbubuntu> Message-ID: <55B233C7.1000206@xilinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/22/2015 05:51 PM, S?ren Brinkmann wrote: > On Wed, 2015-07-22 at 05:38PM +0200, Michal Simek wrote: >> ps-clk-frequency is platform specific setting and shouldn't be the part >> of DTSI. >> >> Signed-off-by: Michal Simek >> --- >> >> Sync with mainline. >> >> --- >> arch/arm/dts/zynq-7000.dtsi | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/arch/arm/dts/zynq-7000.dtsi b/arch/arm/dts/zynq-7000.dtsi >> index a1de993b9812..095c0f67e167 100644 >> --- a/arch/arm/dts/zynq-7000.dtsi >> +++ b/arch/arm/dts/zynq-7000.dtsi >> @@ -241,7 +241,6 @@ >> clkc: clkc at 100 { >> #clock-cells = <1>; >> compatible = "xlnx,ps7-clkc"; >> - ps-clk-frequency = <33333333>; > > If you remove it here, shouldn't it be added in the board dts files at > the same time (the Linux implementation may default to 33MHz, but that > is implementation and not guaranteed according to the bindings)? If you look it is already there and platform with full description are rewriting it. Platform without full description are useless for Linux anyway. Thanks, Michal