From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Thu, 8 May 2014 09:03:44 +0100 Subject: [PATCH 1/9] ARM: STi: DT: STiH416: 416 DT Entry for clockgen A0/1/10/11/12 In-Reply-To: References: <1399305223-18703-1-git-send-email-gabriel.fernandez@linaro.org> <1399305223-18703-2-git-send-email-gabriel.fernandez@linaro.org> <20140506071843.GD5206@lee--X1> Message-ID: <20140508080344.GE5307@lee--X1> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org > >> Patch adds DT entries for clockgen A0/1/10/11/12 > >> > >> Signed-off-by: Pankaj Dev > >> Signed-off-by: Gabriel Fernandez > >> --- > >> arch/arm/boot/dts/stih416-clks.h | 15 ++ > > > > This should live in include/dt-bindings/clk > > > include/dt-bindings/clk or include/dt-bindings/clock ? I have no idea what the difference between the two are. I guess you are in a better position to answer that. My point was just that the includes should be in the include directory, rather than being held locally in the 'dts' directory. > >> arch/arm/boot/dts/stih416-clock.dtsi | 477 ++++++++++++++++++++++++++++++++++ > >> arch/arm/boot/dts/stih416.dtsi | 10 +- > >> 3 files changed, 497 insertions(+), 5 deletions(-) > >> create mode 100644 arch/arm/boot/dts/stih416-clks.h -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 1/9] ARM: STi: DT: STiH416: 416 DT Entry for clockgen A0/1/10/11/12 Date: Thu, 8 May 2014 09:03:44 +0100 Message-ID: <20140508080344.GE5307@lee--X1> References: <1399305223-18703-1-git-send-email-gabriel.fernandez@linaro.org> <1399305223-18703-2-git-send-email-gabriel.fernandez@linaro.org> <20140506071843.GD5206@lee--X1> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Gabriel Fernandez Cc: Gabriel FERNANDEZ , Srinivas Kandagatla , Maxime Coquelin , Patrice Chotard , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pankaj Dev List-Id: devicetree@vger.kernel.org > >> Patch adds DT entries for clockgen A0/1/10/11/12 > >> > >> Signed-off-by: Pankaj Dev > >> Signed-off-by: Gabriel Fernandez > >> --- > >> arch/arm/boot/dts/stih416-clks.h | 15 ++ > > > > This should live in include/dt-bindings/clk > > > include/dt-bindings/clk or include/dt-bindings/clock ? I have no idea what the difference between the two are. I guess you are in a better position to answer that. My point was just that the includes should be in the include directory, rather than being held locally in the 'dts' directory. > >> arch/arm/boot/dts/stih416-clock.dtsi | 477 ++++++++++++++++++++++= ++++++++++++ > >> arch/arm/boot/dts/stih416.dtsi | 10 +- > >> 3 files changed, 497 insertions(+), 5 deletions(-) > >> create mode 100644 arch/arm/boot/dts/stih416-clks.h --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n 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 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753481AbaEHIDz (ORCPT ); Thu, 8 May 2014 04:03:55 -0400 Received: from mail-ie0-f180.google.com ([209.85.223.180]:41808 "EHLO mail-ie0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752932AbaEHIDw (ORCPT ); Thu, 8 May 2014 04:03:52 -0400 Date: Thu, 8 May 2014 09:03:44 +0100 From: Lee Jones To: Gabriel Fernandez Cc: Gabriel FERNANDEZ , Srinivas Kandagatla , Maxime Coquelin , Patrice Chotard , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , linux-arm-kernel@lists.infradead.org, kernel@stlinux.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Pankaj Dev Subject: Re: [PATCH 1/9] ARM: STi: DT: STiH416: 416 DT Entry for clockgen A0/1/10/11/12 Message-ID: <20140508080344.GE5307@lee--X1> References: <1399305223-18703-1-git-send-email-gabriel.fernandez@linaro.org> <1399305223-18703-2-git-send-email-gabriel.fernandez@linaro.org> <20140506071843.GD5206@lee--X1> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > >> Patch adds DT entries for clockgen A0/1/10/11/12 > >> > >> Signed-off-by: Pankaj Dev > >> Signed-off-by: Gabriel Fernandez > >> --- > >> arch/arm/boot/dts/stih416-clks.h | 15 ++ > > > > This should live in include/dt-bindings/clk > > > include/dt-bindings/clk or include/dt-bindings/clock ? I have no idea what the difference between the two are. I guess you are in a better position to answer that. My point was just that the includes should be in the include directory, rather than being held locally in the 'dts' directory. > >> arch/arm/boot/dts/stih416-clock.dtsi | 477 ++++++++++++++++++++++++++++++++++ > >> arch/arm/boot/dts/stih416.dtsi | 10 +- > >> 3 files changed, 497 insertions(+), 5 deletions(-) > >> create mode 100644 arch/arm/boot/dts/stih416-clks.h -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog