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 14:42:03
Also in: linux-devicetree, linux-gpio, lkml

Hi Michael,
 
 On mer., mai 31 2017, Michael Turquette [off-list ref] wrote:
Quoting Rob Herring (2017-05-31 07:23:58)
quoted
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.
Stephen and I have started making a -dt branch for each new clk driver
which. This stable branch has only the dt bindings/headers in it (no
driver code).

I'll make such a branch for ap806 this week and everyone can merge that
in as a dep.
So please use the version 2 I've just submitted. I split each patch to
allow you to have a dt binding only branch. See
http://lists.infradead.org/pipermail/linux-arm-kernel/2017-May/509832.html

I did the same for the cp110:
http://lists.infradead.org/pipermail/linux-arm-kernel/2017-May/509816.html

Gregory
Regards,
Mike
quoted
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