Thread (1 message) 1 message, 1 author, 2017-05-31

[PATCH 3/6] gpio: dt-bindings: Add documentation for gpio controllers on Armada 7K/8K

From: Gregory CLEMENT <hidden>
Date: 2017-05-31 10:03:03
Also in: linux-devicetree, linux-gpio, lkml

Hi Rob,
 
 On mer., mai 31 2017, Rob Herring [off-list ref] wrote:
On Mon, May 29, 2017 at 10:08:16AM +0200, Linus Walleij wrote:
quoted
On Tue, May 23, 2017 at 3:06 PM, Gregory CLEMENT
[off-list ref] wrote:
quoted
 On mar., mai 23 2017, Linus Walleij [off-list ref] wrote:
quoted
quoted
Please rebase and resend the rest of the patches.
Actually I was wrong with my assumption that there was no dependency.
For the binding documentation there is dependecy accross the series I
sent.

This patch depends on "pinctrl: dt-bindings: add documentation for AP806
pin controllers" [1] and "pinctrl: dt-bindings: add documentation for
CP110 pin controllers" [3] from the series "Add support for the pin
controllers on the Marvell Armada 7K/8K".

As you are maintainer of both gpio and pinctrl subsystem it will be easy
to resolv.
Please send all patches in one big series in that case, so I can apply
them all to a branch and merge that branch into both trees.
quoted
But these last patches depend also on "clk: mvebu: ap806: introduce a
new binding" [3] in the clock series "Improve ap806 clk support on
Marvell Armada 7K/8K" and on "clk: mvebu: cp110: introduce a new
binding" [4] in the clock series "Improve cp110 clk support on Marvell
Armada 7K/8K".
That sounds messy.
quoted
So for these particular patches, either I rebase them on the v4.12-rc2
and there will be a small merge conflict during the merge window for
v4.13, or we can ask a stable branch with only these few patches (I am
adding the clock maintainer to this email).

If we chose this last option I can split the clock patches to have patch
modifying only the binding documentation.
I think you can make a patch just modifying the binding documentation
and then apply that patch to *both* trees, just make sure the patch
is *final* and not applied in different versions in clk and pinctrl.
I thought we try to avoid doing that.
quoted
I think cross-tree branches for documentation clashes is too much
bureaucracy.
I can just take all the doc patches separately. Or you can take the 
clock one too.
If you can take all the doc patches I think we will managed to solve the
issue in a efficient way. So I am going to send a new version of the
clock series by splitting the "clk: mvebu: *: introduce a new binding"
patches so you will be able to only pick the doc part.

Thanks!

Gregory

quoted
Make sure your clock patch is finalized and preferably also applied
to the clk tree then put it in the bottom of the patch set you send
to me as well and I will proceed like above.

Yours,
Linus Walleij
-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help