From mboxrd@z Thu Jan 1 00:00:00 1970 From: k.kozlowski@samsung.com (Krzysztof Kozlowski) Date: Thu, 02 Jun 2016 08:18:26 +0200 Subject: [PATCH 3/3] ARM: dts: exynos: Add watchdog amd Security SubSystem to Exynos5410 In-Reply-To: References: <1464774351-23604-1-git-send-email-k.kozlowski@samsung.com> <1464774351-23604-3-git-send-email-k.kozlowski@samsung.com> Message-ID: <574FCFB2.1080607@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 06/01/2016 05:16 PM, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 06/01/2016 05:45 AM, Krzysztof Kozlowski wrote: >> Move watchdog and Security SubSystem nodes from exynos5420.dtsi to file >> shared with Exynos5410 and configure the clocks on the latter. >> >> Signed-off-by: Krzysztof Kozlowski >> --- > > [snip] > >> >> +&sss { >> + clocks = <&clock CLK_SSS>; >> + clock-names = "secss"; >> +}; >> + > > [snip] > >> >> +&watchdog { >> + clocks = <&clock CLK_WDT>; >> + clock-names = "watchdog"; >> + samsung,syscon-phandle = <&pmu_system_controller>; >> +}; >> + > > Since you used the same ID than Exynos5420 for the SSS and WDT clocks, > I wonder if the clock and clock-names shouldn't be just defined in the > shared exynos54xx.dtsi as is the case for the interrupts that are also > the same? > > The patch looks good to me though, so whatever you decide: > > Reviewed-by: Javier Martinez Canillas Thanks for review and comments. The 'clock' property cannot be moved - there is no clock phandle and the clock IDs are not included in 54xx. Best regards, Krzysztof