All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Chen-Yu Tsai <wens@csie.org>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH v3 3/6] dt-bindings: pinctrl: Deprecate sunxi pinctrl bindings
Date: Tue, 25 Oct 2016 15:41:50 +0200	[thread overview]
Message-ID: <20161025134150.dct42swdbwxzgbgb@lukather> (raw)
In-Reply-To: <CACRpkdYGYPhPZQmgeLA-K-RQKZMLy1qHY6P7LS6SKsFC=d=MHg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1403 bytes --]

Hi Linus,

On Tue, Oct 25, 2016 at 02:07:27PM +0200, Linus Walleij wrote:
> On Mon, Oct 24, 2016 at 9:49 PM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> 
> > However, it looks like the first patch from this serie is missing from
> > your tree, is there a reason for that?
> 
> No can you point it out?

Sure:
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-October/462500.html

> > Also, in order to preserve bisectability, could you create an
> > immutable branch for those sunxi patches so that I can merge the DT
> > bits?
> 
> It's too late because they are already in the devel branch
> and mixed up with merged of *other* immutable stuff.

Hmmmm, ok.

> However I think it is plain wrong to try to keep any bisectability
> between the kernel at large and arch/*/boot/dts/*, because
> the DTS stuff is supposed to at some point be maintained outside
> of the kernel and for all OSes, they simply shouldn't be sync:ed.

Yes, in the case of a new driver that needs to be introduced, I
definitely get your point.

However, during a conversion like we're doing here, this is not ideal,
because it essentially means that you will not have a branch that
works, at all.

I'll hold off on those patches until 4.11 then.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 3/6] dt-bindings: pinctrl: Deprecate sunxi pinctrl bindings
Date: Tue, 25 Oct 2016 15:41:50 +0200	[thread overview]
Message-ID: <20161025134150.dct42swdbwxzgbgb@lukather> (raw)
In-Reply-To: <CACRpkdYGYPhPZQmgeLA-K-RQKZMLy1qHY6P7LS6SKsFC=d=MHg@mail.gmail.com>

Hi Linus,

On Tue, Oct 25, 2016 at 02:07:27PM +0200, Linus Walleij wrote:
> On Mon, Oct 24, 2016 at 9:49 PM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> 
> > However, it looks like the first patch from this serie is missing from
> > your tree, is there a reason for that?
> 
> No can you point it out?

Sure:
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-October/462500.html

> > Also, in order to preserve bisectability, could you create an
> > immutable branch for those sunxi patches so that I can merge the DT
> > bits?
> 
> It's too late because they are already in the devel branch
> and mixed up with merged of *other* immutable stuff.

Hmmmm, ok.

> However I think it is plain wrong to try to keep any bisectability
> between the kernel at large and arch/*/boot/dts/*, because
> the DTS stuff is supposed to at some point be maintained outside
> of the kernel and for all OSes, they simply shouldn't be sync:ed.

Yes, in the case of a new driver that needs to be introduced, I
definitely get your point.

However, during a conversion like we're doing here, this is not ideal,
because it essentially means that you will not have a branch that
works, at all.

I'll hold off on those patches until 4.11 then.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161025/b74e47e6/attachment-0001.sig>

  reply	other threads:[~2016-10-25 13:41 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-20 13:49 [PATCH v3 0/6] pinctrl: sunxi: Generic bindings rework Maxime Ripard
2016-10-20 13:49 ` Maxime Ripard
2016-10-20 13:49 ` Maxime Ripard
2016-10-20 13:49 ` [PATCH v3 1/6] pinctrl: sunxi: Deal with configless pins Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-29  9:08   ` Linus Walleij
2016-10-29  9:08     ` Linus Walleij
2016-10-29  9:08     ` Linus Walleij
2016-10-20 13:49 ` [PATCH v3 2/6] pinctrl: sunxi: Support generic binding Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-24  0:02   ` Linus Walleij
2016-10-24  0:02     ` Linus Walleij
2016-10-24  0:02     ` Linus Walleij
2016-10-20 13:49 ` [PATCH v3 3/6] dt-bindings: pinctrl: Deprecate sunxi pinctrl bindings Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-24  0:03   ` Linus Walleij
2016-10-24  0:03     ` Linus Walleij
2016-10-24  0:03     ` Linus Walleij
     [not found]     ` <CACRpkdb7zs7Nhs3i1o=WUt787vJCuePre3Tj3wLu++qwpO0atQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-10-24 19:49       ` Maxime Ripard
2016-10-24 19:49         ` Maxime Ripard
2016-10-24 19:49         ` Maxime Ripard
2016-10-25 12:07         ` Linus Walleij
2016-10-25 12:07           ` Linus Walleij
2016-10-25 12:07           ` Linus Walleij
2016-10-25 13:41           ` Maxime Ripard [this message]
2016-10-25 13:41             ` Maxime Ripard
2016-10-25 13:41             ` Maxime Ripard
2016-10-20 13:49 ` [PATCH v3 4/6] ARM: sunxi: Remove useless allwinner,drive property Maxime Ripard
2016-10-20 13:49 ` [PATCH v3 5/6] ARM: sunxi: Remove useless allwinner,pull property Maxime Ripard
2016-10-20 13:49   ` Maxime Ripard
2016-10-20 15:38   ` Jean-Francois Moine
2016-10-20 15:38     ` Jean-Francois Moine
     [not found]     ` <20161020173848.be9cf97854f8933b3ce90d55-GANU6spQydw@public.gmane.org>
2016-10-21 10:40       ` Maxime Ripard
2016-10-21 10:40         ` [PATCH v3 5/6] ARM: sunxi: Remove useless allwinner, pull property Maxime Ripard
2016-10-21 10:40         ` [PATCH v3 5/6] ARM: sunxi: Remove useless allwinner,pull property Maxime Ripard
2016-10-20 13:49 ` [PATCH v3 6/6] ARM: sunxi: Convert pinctrl nodes to generic bindings Maxime Ripard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161025134150.dct42swdbwxzgbgb@lukather \
    --to=maxime.ripard@free-electrons.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=wens@csie.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.