* [GIT PULL] ARM: mvebu: drivers for v4.9 (#1)
@ 2016-09-19 22:16 ` Arnd Bergmann
0 siblings, 0 replies; 9+ messages in thread
From: Arnd Bergmann @ 2016-09-19 22:16 UTC (permalink / raw)
To: linux-arm-kernel
On Wednesday, September 14, 2016 5:35:10 PM CEST Gregory CLEMENT wrote:
> mvebu drivers for 4.9 (part 1)
>
> - Add pinctrl and clk support for the Orion5x SoC mv88f5181 variant
>
> ----------------------------------------------------------------
> Jamie Lentin (2):
> clk: mvebu: Add clk support for the orion5x SoC mv88f5181
> pinctrl: mvebu: orion5x: Generalise mv88f5181l support for 88f5181
>
> .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 +
> .../bindings/pinctrl/marvell,orion-pinctrl.txt | 4 +-
> drivers/clk/mvebu/orion.c | 70 ++++++++++++++++++++++
> drivers/pinctrl/mvebu/pinctrl-orion.c | 23 +++----
> 4 files changed, 86 insertions(+), 12 deletions(-)
I see this touches pinctrl and clk drivers, but none of the maintainers are
on Cc for this pull request, nor have they provided an Ack for the patches
according to your git log.
Why did you not send the driver changes to them?
Not pulled for now, until this is resolved, sorry.
Arnd
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL] ARM: mvebu: drivers for v4.9 (#1)
2016-09-19 22:16 ` Arnd Bergmann
@ 2016-09-20 7:09 ` Gregory CLEMENT
-1 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2016-09-20 7:09 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Olof Johansson, arm, Andrew Lunn, Jason Cooper,
Sebastian Hesselbarth, linux-arm-kernel, linux-gpio,
Linus Walleij, Michael Turquette, Stephen Boyd, linux-clk
Hi Arnd,
On mar., sept. 20 2016, Arnd Bergmann <arnd@arndb.de> wrote:
> On Wednesday, September 14, 2016 5:35:10 PM CEST Gregory CLEMENT wrote:
>> mvebu drivers for 4.9 (part 1)
>>
>> - Add pinctrl and clk support for the Orion5x SoC mv88f5181 variant
>>
>> ----------------------------------------------------------------
>> Jamie Lentin (2):
>> clk: mvebu: Add clk support for the orion5x SoC mv88f5181
>> pinctrl: mvebu: orion5x: Generalise mv88f5181l support for 88f5181
>>
>> .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 +
>> .../bindings/pinctrl/marvell,orion-pinctrl.txt | 4 +-
>> drivers/clk/mvebu/orion.c | 70 ++++++++++++++++++++++
>> drivers/pinctrl/mvebu/pinctrl-orion.c | 23 +++----
>> 4 files changed, 86 insertions(+), 12 deletions(-)
>
> I see this touches pinctrl and clk drivers, but none of the maintainers are
> on Cc for this pull request, nor have they provided an Ack for the patches
> according to your git log.
>
> Why did you not send the driver changes to them?
My bad sorry. The changes only touch data (adding new varaiant of the
orion5x), are very trivial and won't cause any merge
conflict. Furthermore there is depency with the other pacthes in the
mvebu branches and we already did this in the past to avoid dependecy
between subsystem. _But_ I should have asked the acked-by from the
drivers maintainers, it was the way we wolved it in the past and I
forgot it.
>
> Not pulled for now, until this is resolved, sorry.
If the drivers maintainer for pinctrl and clk give their acked-by do you
agree to take it for 4.9?
Thanks,
Gregory
--
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] 9+ messages in thread
* [GIT PULL] ARM: mvebu: drivers for v4.9 (#1)
@ 2016-09-20 7:09 ` Gregory CLEMENT
0 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2016-09-20 7:09 UTC (permalink / raw)
To: linux-arm-kernel
Hi Arnd,
On mar., sept. 20 2016, Arnd Bergmann <arnd@arndb.de> wrote:
> On Wednesday, September 14, 2016 5:35:10 PM CEST Gregory CLEMENT wrote:
>> mvebu drivers for 4.9 (part 1)
>>
>> - Add pinctrl and clk support for the Orion5x SoC mv88f5181 variant
>>
>> ----------------------------------------------------------------
>> Jamie Lentin (2):
>> clk: mvebu: Add clk support for the orion5x SoC mv88f5181
>> pinctrl: mvebu: orion5x: Generalise mv88f5181l support for 88f5181
>>
>> .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 +
>> .../bindings/pinctrl/marvell,orion-pinctrl.txt | 4 +-
>> drivers/clk/mvebu/orion.c | 70 ++++++++++++++++++++++
>> drivers/pinctrl/mvebu/pinctrl-orion.c | 23 +++----
>> 4 files changed, 86 insertions(+), 12 deletions(-)
>
> I see this touches pinctrl and clk drivers, but none of the maintainers are
> on Cc for this pull request, nor have they provided an Ack for the patches
> according to your git log.
>
> Why did you not send the driver changes to them?
My bad sorry. The changes only touch data (adding new varaiant of the
orion5x), are very trivial and won't cause any merge
conflict. Furthermore there is depency with the other pacthes in the
mvebu branches and we already did this in the past to avoid dependecy
between subsystem. _But_ I should have asked the acked-by from the
drivers maintainers, it was the way we wolved it in the past and I
forgot it.
>
> Not pulled for now, until this is resolved, sorry.
If the drivers maintainer for pinctrl and clk give their acked-by do you
agree to take it for 4.9?
Thanks,
Gregory
--
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] 9+ messages in thread
* Re: [GIT PULL] ARM: mvebu: drivers for v4.9 (#1)
2016-09-19 22:16 ` Arnd Bergmann
@ 2016-09-20 21:51 ` Linus Walleij
-1 siblings, 0 replies; 9+ messages in thread
From: Linus Walleij @ 2016-09-20 21:51 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Gregory CLEMENT, Olof Johansson, arm@kernel.org, Andrew Lunn,
Jason Cooper, Sebastian Hesselbarth,
linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org,
Michael Turquette, Stephen Boyd, linux-clk
On Tue, Sep 20, 2016 at 12:16 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Wednesday, September 14, 2016 5:35:10 PM CEST Gregory CLEMENT wrote:
>> mvebu drivers for 4.9 (part 1)
>>
>> - Add pinctrl and clk support for the Orion5x SoC mv88f5181 variant
>>
>> ----------------------------------------------------------------
>> Jamie Lentin (2):
>> clk: mvebu: Add clk support for the orion5x SoC mv88f5181
>> pinctrl: mvebu: orion5x: Generalise mv88f5181l support for 88f5181
>>
>> .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 +
>> .../bindings/pinctrl/marvell,orion-pinctrl.txt | 4 +-
>> drivers/clk/mvebu/orion.c | 70 ++++++++++++++++++++++
>> drivers/pinctrl/mvebu/pinctrl-orion.c | 23 +++----
>> 4 files changed, 86 insertions(+), 12 deletions(-)
>
> I see this touches pinctrl and clk drivers, but none of the maintainers are
> on Cc for this pull request, nor have they provided an Ack for the patches
> according to your git log.
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread
* [GIT PULL] ARM: mvebu: drivers for v4.9 (#1)
@ 2016-09-20 21:51 ` Linus Walleij
0 siblings, 0 replies; 9+ messages in thread
From: Linus Walleij @ 2016-09-20 21:51 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 20, 2016 at 12:16 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Wednesday, September 14, 2016 5:35:10 PM CEST Gregory CLEMENT wrote:
>> mvebu drivers for 4.9 (part 1)
>>
>> - Add pinctrl and clk support for the Orion5x SoC mv88f5181 variant
>>
>> ----------------------------------------------------------------
>> Jamie Lentin (2):
>> clk: mvebu: Add clk support for the orion5x SoC mv88f5181
>> pinctrl: mvebu: orion5x: Generalise mv88f5181l support for 88f5181
>>
>> .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 +
>> .../bindings/pinctrl/marvell,orion-pinctrl.txt | 4 +-
>> drivers/clk/mvebu/orion.c | 70 ++++++++++++++++++++++
>> drivers/pinctrl/mvebu/pinctrl-orion.c | 23 +++----
>> 4 files changed, 86 insertions(+), 12 deletions(-)
>
> I see this touches pinctrl and clk drivers, but none of the maintainers are
> on Cc for this pull request, nor have they provided an Ack for the patches
> according to your git log.
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread