From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-bk0-f53.google.com ([209.85.214.53]:33173 "EHLO mail-bk0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755983Ab3HWTnN (ORCPT ); Fri, 23 Aug 2013 15:43:13 -0400 From: Tomasz Figa Subject: Re: [PATCH v4 5/7] ARM: dts: add clocks for exynos5420 hdmi subsystem Date: Fri, 23 Aug 2013 21:43:06 +0200 Message-ID: <3634494.5b0t0b50Cd@flatron> In-Reply-To: <1377253461-20851-6-git-send-email-rahul.sharma@samsung.com> References: <1377253461-20851-1-git-send-email-rahul.sharma@samsung.com> <1377253461-20851-6-git-send-email-rahul.sharma@samsung.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: devicetree-owner@vger.kernel.org To: Rahul Sharma Cc: linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, sw0312.kim@samsung.com, inki.dae@samsung.com, s.nawrocki@samsung.com, joshi@samsung.com, r.sh.open@gmail.com List-ID: Hi Rahul, On Friday 23 of August 2013 15:54:19 Rahul Sharma wrote: > Add clocks for hdmi and mixer for exynos5420 SoC. > > Signed-off-by: Rahul Sharma > --- > arch/arm/boot/dts/exynos5420.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos5420.dtsi > b/arch/arm/boot/dts/exynos5420.dtsi index 7c6214d..1405074 100644 > --- a/arch/arm/boot/dts/exynos5420.dtsi > +++ b/arch/arm/boot/dts/exynos5420.dtsi > @@ -279,6 +279,10 @@ > compatible = "samsung,exynos4212-hdmi"; > reg = <0x14530000 0x70000>; > interrupts = <0 95 0>; > + clocks = <&clock 413>, <&clock 143>, <&clock 144>, > + <&clock 158>, <&clock 1024>; > + clock-names = "hdmi", "sclk_hdmi", "sclk_pixel", > + "sclk_hdmiphy", "mout_hdmi"; > status = "disabled"; > }; > > @@ -286,6 +290,8 @@ > compatible = "samsung,exynos5420-mixer"; > reg = <0x14450000 0x10000>; > interrupts = <0 94 0>; > + clocks = <&clock 431>, <&clock 143>; > + clock-names = "mixer", "sclk_hdmi"; > status = "disabled"; > }; > }; You just added all the nodes in previous patch. There is no need for a separate patch adding clocks. Please squash this with previous patch. Best regards, Tomasz