* [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K @ 2017-06-19 15:36 Gregory CLEMENT 2017-06-20 0:18 ` Stephen Boyd 0 siblings, 1 reply; 3+ messages in thread From: Gregory CLEMENT @ 2017-06-19 15:36 UTC (permalink / raw) To: linux-arm-kernel Hi Mike, Stephen, This time I turned the series in a pull request so I removed the device tree binding part which will be in an other series for Rob Herring and I also removed the dt part that I will take care of once you will have pulled this one. I hope this will be easier for you to apply this way. Thanks! Gregory The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6: Linux 4.12-rc1 (Sat, 13 May 2017 13:19:49 -0700) are available in the git repository at: git://git.infradead.org/linux-mvebu.git refs/heads/clk-cp110 for you to fetch changes up to a45af6d3a98b4d8a2746de13a8db52fb4123bb56: clk: mvebu: cp110: add sdio clock to cp-110 system controller (Wed, 31 May 2017 15:19:15 +0200) ---------------------------------------------------------------- Improve cp110 clk support on Marvell Armada 7K/8K 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 clk driver is modified accordingly from this new information. ---------------------------------------------------------------- Gregory CLEMENT (3): clk: mvebu: cp110: make failure labels more meaningful clk: mvebu: cp110: do not depend anymore of the *-clock-output-names clk: mvebu: cp110: introduce a new binding Konstantin Porotchkin (1): clk: mvebu: cp110: add sdio clock to cp-110 system controller drivers/clk/mvebu/cp110-system-controller.c | 200 ++++++++++++++------- 1 file changed, 138 insertions(+), 62 deletions(-) -- git-series 0.9.1 ^ permalink raw reply [flat|nested] 3+ messages in thread
* [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K 2017-06-19 15:36 [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K Gregory CLEMENT @ 2017-06-20 0:18 ` Stephen Boyd 2017-06-20 7:32 ` Gregory CLEMENT 0 siblings, 1 reply; 3+ messages in thread From: Stephen Boyd @ 2017-06-20 0:18 UTC (permalink / raw) To: linux-arm-kernel On 06/19, Gregory CLEMENT wrote: > > Hi Mike, Stephen, > > This time I turned the series in a pull request so I removed the > device tree binding part which will be in an other series for Rob > Herring and I also removed the dt part that I will take care of once > you will have pulled this one. > > I hope this will be easier for you to apply this way. > Ok. It wasn't clear what was going on and I forgot to ask in the lastest patch series. Pulled into clk-next, but I had to apply this patch on top: ---8<---- From: Stephen Boyd <sboyd@codeaurora.org> Subject: [PATCH] clk: mvebu: cp110: Minor cleanups Mark an array of strings static const and remove the dereference of a function pointer when assigning to the platform driver probe struct member. drivers/clk/mvebu/cp110-system-controller.c:89:12: warning: symbol 'gate_base_names' was not declared. Should it be static? drivers/clk/mvebu/cp110-system-controller.c:447:18: error: cannot dereference this type Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- drivers/clk/mvebu/cp110-system-controller.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/clk/mvebu/cp110-system-controller.c b/drivers/clk/mvebu/cp110-system-controller.c index b034b79345ec..ca9a0a536174 100644 --- a/drivers/clk/mvebu/cp110-system-controller.c +++ b/drivers/clk/mvebu/cp110-system-controller.c @@ -86,7 +86,7 @@ enum { #define CP110_GATE_EIP150 25 #define CP110_GATE_EIP197 26 -const char *gate_base_names[] = { +static const char * const gate_base_names[] = { [CP110_GATE_AUDIO] = "audio", [CP110_GATE_COMM_UNIT] = "communit", [CP110_GATE_NAND] = "nand", @@ -437,14 +437,13 @@ static int cp110_clk_probe(struct platform_device *pdev) return cp110_syscon_common_probe(pdev, pdev->dev.of_node->parent); } - static const struct of_device_id cp110_syscon_legacy_of_match[] = { { .compatible = "marvell,cp110-system-controller0", }, { } }; static struct platform_driver cp110_syscon_legacy_driver = { - .probe = *cp110_syscon_legacy_clk_probe, + .probe = cp110_syscon_legacy_clk_probe, .driver = { .name = "marvell-cp110-system-controller0", .of_match_table = cp110_syscon_legacy_of_match, -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K 2017-06-20 0:18 ` Stephen Boyd @ 2017-06-20 7:32 ` Gregory CLEMENT 0 siblings, 0 replies; 3+ messages in thread From: Gregory CLEMENT @ 2017-06-20 7:32 UTC (permalink / raw) To: linux-arm-kernel Hi Stephen, On mar., juin 20 2017, Stephen Boyd <sboyd@codeaurora.org> wrote: > On 06/19, Gregory CLEMENT wrote: >> >> Hi Mike, Stephen, >> >> This time I turned the series in a pull request so I removed the >> device tree binding part which will be in an other series for Rob >> Herring and I also removed the dt part that I will take care of once >> you will have pulled this one. >> >> I hope this will be easier for you to apply this way. >> > > Ok. It wasn't clear what was going on and I forgot to ask in the > lastest patch series. Pulled into clk-next, but I had to apply > this patch on top: Thanks for the pull and the fix patch! Gregory > > ---8<---- > From: Stephen Boyd <sboyd@codeaurora.org> > Subject: [PATCH] clk: mvebu: cp110: Minor cleanups > > Mark an array of strings static const and remove the dereference > of a function pointer when assigning to the platform driver probe > struct member. > > drivers/clk/mvebu/cp110-system-controller.c:89:12: > warning: symbol 'gate_base_names' was not declared. Should it be static? > drivers/clk/mvebu/cp110-system-controller.c:447:18: > error: cannot dereference this type > > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > drivers/clk/mvebu/cp110-system-controller.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/clk/mvebu/cp110-system-controller.c b/drivers/clk/mvebu/cp110-system-controller.c > index b034b79345ec..ca9a0a536174 100644 > --- a/drivers/clk/mvebu/cp110-system-controller.c > +++ b/drivers/clk/mvebu/cp110-system-controller.c > @@ -86,7 +86,7 @@ enum { > #define CP110_GATE_EIP150 25 > #define CP110_GATE_EIP197 26 > > -const char *gate_base_names[] = { > +static const char * const gate_base_names[] = { > [CP110_GATE_AUDIO] = "audio", > [CP110_GATE_COMM_UNIT] = "communit", > [CP110_GATE_NAND] = "nand", > @@ -437,14 +437,13 @@ static int cp110_clk_probe(struct platform_device *pdev) > return cp110_syscon_common_probe(pdev, pdev->dev.of_node->parent); > } > > - > static const struct of_device_id cp110_syscon_legacy_of_match[] = { > { .compatible = "marvell,cp110-system-controller0", }, > { } > }; > > static struct platform_driver cp110_syscon_legacy_driver = { > - .probe = *cp110_syscon_legacy_clk_probe, > + .probe = cp110_syscon_legacy_clk_probe, > .driver = { > .name = "marvell-cp110-system-controller0", > .of_match_table = cp110_syscon_legacy_of_match, > -- > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > a Linux Foundation Collaborative Project -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-06-20 7:32 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-06-19 15:36 [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K Gregory CLEMENT 2017-06-20 0:18 ` Stephen Boyd 2017-06-20 7:32 ` Gregory CLEMENT
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).