[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140508080344.GE5307@lee--X1>
Date: Thu, 8 May 2014 09:03:44 +0100
From: Lee Jones <lee.jones@...aro.org>
To: Gabriel Fernandez <gabriel.fernandez@...aro.org>
Cc: Gabriel FERNANDEZ <gabriel.fernandez@...com>,
Srinivas Kandagatla <srinivas.kandagatla@...il.com>,
Maxime Coquelin <maxime.coquelin@...com>,
Patrice Chotard <patrice.chotard@...com>,
Rob Herring <robh+dt@...nel.org>,
Pawel Moll <pawel.moll@....com>,
Mark Rutland <mark.rutland@....com>,
Ian Campbell <ijc+devicetree@...lion.org.uk>,
Kumar Gala <galak@...eaurora.org>,
Russell King <linux@....linux.org.uk>,
linux-arm-kernel@...ts.infradead.org, kernel@...inux.com,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Pankaj Dev <pankaj.dev@...com>
Subject: Re: [PATCH 1/9] ARM: STi: DT: STiH416: 416 DT Entry for clockgen
A0/1/10/11/12
> >> Patch adds DT entries for clockgen A0/1/10/11/12
> >>
> >> Signed-off-by: Pankaj Dev <pankaj.dev@...com>
> >> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@...aro.org>
> >> ---
> >> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists