From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [PATCH v6 02/17] dts: mpc512x: introduce dt-bindings/clock/ header Date: Mon, 2 Dec 2013 09:46:56 +0000 Message-ID: <20131202094656.GA12952@e106331-lin.cambridge.arm.com> References: <1385851897-23475-1-git-send-email-gsi@denx.de> <1385851897-23475-3-git-send-email-gsi@denx.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1385851897-23475-3-git-send-email-gsi@denx.de> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Gerhard Sittig Cc: "devicetree@vger.kernel.org" , Mike Turquette , Pawel Moll , Detlev Zundel , Stephen Warren , "rob.herring@calxeda.com" , Scott Wood , Anatolij Gustschin , "linuxppc-dev@lists.ozlabs.org" , "linux-arm-kernel@lists.infradead.org" , Ian Campbell List-Id: devicetree@vger.kernel.org On Sat, Nov 30, 2013 at 10:51:22PM +0000, Gerhard Sittig wrote: > introduce a dt-bindings/ header file for MPC512x clocks, > providing symbolic identifiers for those SoC clocks which > clients will reference from their device tree nodes There should be a binding document update to go with this, pointing out that this include file defines the set of clock IDs for the MPC512x clocks. Otherwise, this looks fine to me. Mark. > > Cc: Rob Herring > Cc: Pawel Moll > Cc: Mark Rutland > Cc: Stephen Warren > Cc: Ian Campbell > Cc: devicetree@vger.kernel.org > Reviewed-by: Mike Turquette # for v3: w/o bdlc, PSC ipg > Signed-off-by: Gerhard Sittig > --- > include/dt-bindings/clock/mpc512x-clock.h | 69 +++++++++++++++++++++++++++++ > 1 file changed, 69 insertions(+) > create mode 100644 include/dt-bindings/clock/mpc512x-clock.h > > diff --git a/include/dt-bindings/clock/mpc512x-clock.h b/include/dt-bindings/clock/mpc512x-clock.h > new file mode 100644 > index 000000000000..9e81b3b99a32 > --- /dev/null > +++ b/include/dt-bindings/clock/mpc512x-clock.h > @@ -0,0 +1,69 @@ > +/* > + * This header provides constants for MPC512x clock specs in DT bindings. > + */ > + > +#ifndef _DT_BINDINGS_CLOCK_MPC512x_CLOCK_H > +#define _DT_BINDINGS_CLOCK_MPC512x_CLOCK_H > + > +#define MPC512x_CLK_DUMMY 0 > +#define MPC512x_CLK_REF 1 > +#define MPC512x_CLK_SYS 2 > +#define MPC512x_CLK_DIU 3 > +#define MPC512x_CLK_VIU 4 > +#define MPC512x_CLK_CSB 5 > +#define MPC512x_CLK_E300 6 > +#define MPC512x_CLK_IPS 7 > +#define MPC512x_CLK_FEC 8 > +#define MPC512x_CLK_SATA 9 > +#define MPC512x_CLK_PATA 10 > +#define MPC512x_CLK_NFC 11 > +#define MPC512x_CLK_LPC 12 > +#define MPC512x_CLK_MBX_BUS 13 > +#define MPC512x_CLK_MBX 14 > +#define MPC512x_CLK_MBX_3D 15 > +#define MPC512x_CLK_AXE 16 > +#define MPC512x_CLK_USB1 17 > +#define MPC512x_CLK_USB2 18 > +#define MPC512x_CLK_I2C 19 > +#define MPC512x_CLK_MSCAN0_MCLK 20 > +#define MPC512x_CLK_MSCAN1_MCLK 21 > +#define MPC512x_CLK_MSCAN2_MCLK 22 > +#define MPC512x_CLK_MSCAN3_MCLK 23 > +#define MPC512x_CLK_BDLC 24 > +#define MPC512x_CLK_SDHC 25 > +#define MPC512x_CLK_PCI 26 > +#define MPC512x_CLK_PSC_MCLK_IN 27 > +#define MPC512x_CLK_SPDIF_TX 28 > +#define MPC512x_CLK_SPDIF_RX 29 > +#define MPC512x_CLK_SPDIF_MCLK 30 > +#define MPC512x_CLK_SPDIF 31 > +#define MPC512x_CLK_AC97 32 > +#define MPC512x_CLK_PSC0_MCLK 33 > +#define MPC512x_CLK_PSC1_MCLK 34 > +#define MPC512x_CLK_PSC2_MCLK 35 > +#define MPC512x_CLK_PSC3_MCLK 36 > +#define MPC512x_CLK_PSC4_MCLK 37 > +#define MPC512x_CLK_PSC5_MCLK 38 > +#define MPC512x_CLK_PSC6_MCLK 39 > +#define MPC512x_CLK_PSC7_MCLK 40 > +#define MPC512x_CLK_PSC8_MCLK 41 > +#define MPC512x_CLK_PSC9_MCLK 42 > +#define MPC512x_CLK_PSC10_MCLK 43 > +#define MPC512x_CLK_PSC11_MCLK 44 > +#define MPC512x_CLK_PSC_FIFO 45 > +#define MPC512x_CLK_PSC0 46 > +#define MPC512x_CLK_PSC1 47 > +#define MPC512x_CLK_PSC2 48 > +#define MPC512x_CLK_PSC3 49 > +#define MPC512x_CLK_PSC4 50 > +#define MPC512x_CLK_PSC5 51 > +#define MPC512x_CLK_PSC6 52 > +#define MPC512x_CLK_PSC7 53 > +#define MPC512x_CLK_PSC8 54 > +#define MPC512x_CLK_PSC9 55 > +#define MPC512x_CLK_PSC10 56 > +#define MPC512x_CLK_PSC11 57 > + > +#define MPC512x_CLK_LAST_PUBLIC 57 > + > +#endif > -- > 1.7.10.4 > >