Re: [PATCH v6 3/4] ARM: dts: Add STiH407 SoC support

[Maxime Coquelin]

On 05/20/2014 02:17 PM, Lee Jones wrote:
> > The STiH407 is advanced multi-HD AVC processor with 3D graphics acceleration
> > and 1.5-GHz ARM Cortex-A9 SMP CPU.
> >
> > Acked-by: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>
> > Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > Acked-by: Patrice Chotard <patrice.chotard@xxxxxx>
> > Signed-off-by: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>
> > Signed-off-by: Maxime Coquelin <maxime.coquelin@xxxxxx>
> > ---
> >   arch/arm/boot/dts/stih407-clock.dtsi   |  40 +++
> >   arch/arm/boot/dts/stih407-pinctrl.dtsi | 615 +++++++++++++++++++++++++++++++++
> >   arch/arm/boot/dts/stih407.dtsi         | 263 ++++++++++++++
> >   3 files changed, 918 insertions(+)
> >   create mode 100644 arch/arm/boot/dts/stih407-clock.dtsi
> >   create mode 100644 arch/arm/boot/dts/stih407-pinctrl.dtsi
> >   create mode 100644 arch/arm/boot/dts/stih407.dtsi
> >
> > diff --git a/arch/arm/boot/dts/stih407-clock.dtsi b/arch/arm/boot/dts/stih407-clock.dtsi
> > new file mode 100644
> > index 0000000..ae8068c
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/stih407-clock.dtsi
> > @@ -0,0 +1,40 @@
> > +/*
> > + * Copyright (C) 2014 STMicroelectronics R&D Limited
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + */
> > +/ {
> > +	clocks {
> > +		/*
> > +		 * Fixed 30MHz oscillator inputs to SoC
> > +		 */
> > +		CLK_SYSIN: CLK_SYSIN {
>
> I thought we weren't using upper case clk names anymore?
>
> > +			#clock-cells = <0>;
> > +			compatible = "fixed-clock";
> > +			clock-frequency = <30000000>;
> > +			clock-output-names = "CLK_SYSIN";
> > +		};
> > +
> > +		/*
> > +		 * ARM Peripheral clock for timers
> > +		 */
> > +		arm_periph_clk: arm_periph_clk {
> > +			#clock-cells = <0>;
> > +			compatible = "fixed-clock";
> > +			clock-frequency = <600000000>;
> > +		};
> > +
> > +		/*
> > +		 * Bootloader initialized system infrastructure clock for
> > +		 * serial devices.
> > +		 */
> > +		CLK_EXT2F_A9: clockgenC0@13 {
>
> Or camel?

Good catch, v7 is in the pipe.

Thanks,
Maxime

> [...]
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/