On 05/07/2013 05:52 PM, Valentin Longchamp wrote: > The kirkwood device found in the Prestera SoCs does not have all the > peripherals of its the usual kirkwood SoCs. There are hence missing > clocks in the SoCs. > > This patch registers another gate controller for the kirkwood that > registers only the available clocks of this kirkwood variant.
Hi Valentin, Overall the patch set looks OK, however I don't understand why you need to declare a new gate controller. The list you delcared seemed to be just a subset of the Kirkwood one, why can't you use this one? Note that I am not an expert for the kirkwood hardware, so maybe I missed something. Regards, > > Signed-off-by: Valentin Longchamp <[email protected]> > --- > .../devicetree/bindings/clock/mvebu-gated-clock.txt | 17 +++++++++++++++++ > drivers/clk/mvebu/clk-gating-ctrl.c | 19 > +++++++++++++++++++ > 2 files changed, 36 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/mvebu-gated-clock.txt > b/Documentation/devicetree/bindings/clock/mvebu-gated-clock.txt > index cffc93d..7f494e90 100644 > --- a/Documentation/devicetree/bindings/clock/mvebu-gated-clock.txt > +++ b/Documentation/devicetree/bindings/clock/mvebu-gated-clock.txt > @@ -92,10 +92,27 @@ ID Clock Peripheral > 19 ge1 Gigabit Ethernet 1 > 20 tdm Time Division Mplx > > +The following is a list of provided IDs for Kirkwood Prestera variant: > +ID Clock Peripheral > +----------------------------------- > +0 ge0 Gigabit Ethernet 0 > +2 pex0 PCIe Cntrl 0 > +3 usb0 USB Host 0 > +4 sdio SDIO Cntrl > +5 tsu Transp. Stream Unit > +6 dunit SDRAM Cntrl > +7 runit Runit > +8 xor0 XOR DMA 0 > +16 xor1 XOR DMA 1 > +17 crypto CESA engine > +19 ge1 Gigabit Ethernet 1 > + > + > Required properties: > - compatible : shall be one of the following: > "marvell,dove-gating-clock" - for Dove SoC clock gating > "marvell,kirkwood-gating-clock" - for Kirkwood SoC clock gating > + "marvell,prestera-kw-gating-clock" - for Preseta SoC clock gating > - reg : shall be the register address of the Clock Gating Control register > - #clock-cells : from common clock binding; shall be set to 1 > > diff --git a/drivers/clk/mvebu/clk-gating-ctrl.c > b/drivers/clk/mvebu/clk-gating-ctrl.c > index ebf141d..ba37802 100644 > --- a/drivers/clk/mvebu/clk-gating-ctrl.c > +++ b/drivers/clk/mvebu/clk-gating-ctrl.c > @@ -203,6 +203,21 @@ static const struct mvebu_soc_descr __initconst > kirkwood_gating_descr[] = { > { "tdm", NULL, 20 }, > { } > }; > + > +static const struct mvebu_soc_descr __initconst prestera_kw_gating_descr[] = > { > + { "ge0", NULL, 0 }, > + { "pex0", NULL, 2 }, > + { "usb0", NULL, 3 }, > + { "sdio", NULL, 4 }, > + { "tsu", NULL, 5 }, > + { "runit", NULL, 7 }, > + { "xor0", NULL, 8 }, > + { "powersave", "cpuclk", 11 }, > + { "xor1", NULL, 16 }, > + { "crypto", NULL, 17 }, > + { "ge1", NULL, 19 }, > + { } > +}; > #endif > > static const __initdata struct of_device_id clk_gating_match[] = { > @@ -232,6 +247,10 @@ static const __initdata struct of_device_id > clk_gating_match[] = { > .compatible = "marvell,kirkwood-gating-clock", > .data = kirkwood_gating_descr, > }, > + { > + .compatible = "marvell,prestera-kw-gating-clock", > + .data = prestera_kw_gating_descr, > + }, > #endif > > { } > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
