Hi , Please do not apply this version. I was just pointed that I misread the documentation and that the GOP clock are really also used by the SD/eMMC. So I am about sending a new version of the series removing the 2 first patchs and fixing the patch 5. Gregory On mer., mai 31 2017, Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > We got more information about the clock controllers and the clock tree > of the CP110 part that we find in the Marvell Armada 7K/8K SoCs. > > The main change in this second version is about the binding > documentation: all the documentation related changes are now move in > their own patches. It allows to provide a stable -dt branch for theses > changes. I also added the acked-by from Rob Herring. > > However the last two patches modifying the device tree _must_ be > merged through the mvebu tree to avoid future conflict (especially > with the recent ICU series sent by Thomas Petazzoni). > > For the record: > > The clock tree needs to be fixed, indeed the GOP clock is only the > clock for the GOIP and not for the SDIO, it is fixed in the first > patch. > > The third patch does a small improve of the code itself. > > The purpose of the fifth patch is to have an easier way to add new > clocks without breaking the backward compatibility. Indeed, we expect > to continue to have new update on the datasheet and this patch will > allow integrating this change easier and faster. > > The patch 7 is needed to ease the integration of the pinctrl node > in the device tree. > > The patch 9 adds a new clock for the SDIO, it also fixes the clock > tree. > > In this series, even if there is some change in the device tree > binding, we paid attention to the backward compatibility, and the > driver can still work with the old device tree. > > Thanks, > > Gregory > > Gregory CLEMENT (10): > dt-bindings: cp110: fix name of the GOP gate clock > clk: mvebu: cp110 fix name of the GOP gate clock > clk: mvebu: cp110: make failure labels more meaningful > dt-bindings: cp110: do not depend anymore of the *-clock-output-names > clk: mvebu: cp110: do not depend anymore of the *-clock-output-names > dt-bindings: cp110: introduce a new binding > clk: mvebu: cp110: introduce a new binding > dt-bindings: cp110: add sdio clock to cp-110 system controller > arm64: dts: marvell: remove *-clock-output-names on cp110 > arm64: dts: marvell: use new binding for the system controller on cp110 > > Konstantin Porotchkin (1): > clk: mvebu: cp110: add sdio clock to cp-110 system controller > > Documentation/devicetree/bindings/arm/marvell/cp110-system-controller0.txt | 35 +++++------- > arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 54 +++++++------------ > arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 48 ++++++----------- > drivers/clk/mvebu/cp110-system-controller.c | 202 +++++++++++++++++++++++++++++++++++++++++++++++++----------------------- > 4 files changed, 196 insertions(+), 143 deletions(-) > > base-commit: 2ea659a9ef488125eb46da6eb571de5eae5c43f6 > -- > git-series 0.9.1 -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html