public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] Allwinner DT changes for 4.4, round 3
Date: Mon, 26 Oct 2015 11:34:09 +0100	[thread overview]
Message-ID: <20151026103409.GX10947@lukather> (raw)
In-Reply-To: <20151026054828.GD4216@localhost>

On Mon, Oct 26, 2015 at 02:48:28PM +0900, Olof Johansson wrote:
> On Mon, Oct 26, 2015 at 09:46:24AM +0800, Chen-Yu Tsai wrote:
> > Hi Olof,
> > 
> > On Mon, Oct 26, 2015 at 9:07 AM, Olof Johansson <olof@lixom.net> wrote:
> > > On Sun, Oct 25, 2015 at 09:13:08PM +0100, Maxime Ripard wrote:
> > >> On Fri, Oct 23, 2015 at 10:55:13PM +0200, Arnd Bergmann wrote:
> > >> > On Friday 23 October 2015 22:41:04 Arnd Bergmann wrote:
> > >> > > On Friday 23 October 2015 20:07:11 Maxime Ripard wrote:
> > >> > > > Allwinner DT changes for 4.4, round 3
> > >> > > >
> > >> > > > A bunch of new boards, but mostly enable the support for the Audio Codec
> > >> > > > on the older SoCs (A10, A10s, A13, A20, R8) and enable it on various
> > >> > > > boards.
> > >> > > >
> > >> > > >
> > >> > >
> > >> > > Pulled into next/dt, thanks!
> > >> >
> > >> > And dropped again:
> > >> >
> > >> > In file included from /git/arm-soc/arch/arm/boot/dts/sun4i-a10-a1000.dts:46:0:
> > >> > /git/arm-soc/arch/arm/boot/dts/sun4i-a10.dtsi:48:46: fatal error: dt-bindings/clock/sun4i-a10-pll2.h: No such file or directory
> > >> >
> > >> > Please make sure branches you send actually work by themselves.
> > >>
> > >> Damn, you're right... Sorry about that...
> > >>
> > >> You need to pull the tag sunxi-clk-for-4.4 that has been merged in the
> > >> clk tree in 38ce30e29dc.
> > >>
> > >> Mike, Stephen, will that tag be stable?
> > >
> > > I don't see that tag in your tree, Maxime? I also don't see 38ce30e29dc in
> > > the clk tree on kernel.org.
> > >
> > > I'm a little confused right now...
> > 
> > The tag is "sunxi-clocks-for-4.4":
> >     https://git.kernel.org/cgit/linux/kernel/git/mripard/linux.git/tag/?h=sunxi-clocks-for-4.4
> > 
> > The merge commit in the clk tree is 938ce30e29dc:
> >     https://git.kernel.org/cgit/linux/kernel/git/clk/linux.git/commit/?h=clk-next&id=938ce30e29dc
> 
> Thanks, so both the tag and the hash were missing a few characters. :-)

Gaah, sorry...

> I've merged this in now. Note that this has resulted in a tree that won't
> misect cleanly, since having the clk contents merged instead of used as a base
> for the dt branch means that you could end up in a bisect state that has the DT
> branch but not the clk branch.

Even if it has been merged before the DT patches have been applied?
That's not really what I'd expect from bisect :/

> These are some of the reasons why I'm not overly excited about these
> header file dependencies. :(

Yeah, I see why now. Your suggestion of using the raw values first and
then switch to using the headers make sense I guess, we'll try to
stick to that in the future.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151026/9df02597/attachment.sig>

  reply	other threads:[~2015-10-26 10:34 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-23 18:07 [GIT PULL] Allwinner DT changes for 4.4, round 3 Maxime Ripard
2015-10-23 20:41 ` Arnd Bergmann
2015-10-23 20:55   ` Arnd Bergmann
2015-10-25 20:13     ` Maxime Ripard
2015-10-26  1:07       ` Olof Johansson
2015-10-26  1:46         ` Chen-Yu Tsai
2015-10-26  5:48           ` Olof Johansson
2015-10-26 10:34             ` Maxime Ripard [this message]
2015-10-31  9:42               ` Olof Johansson
2015-11-02 19:19                 ` Stephen Boyd
2015-11-02 19:35                   ` Olof Johansson
2015-11-02 19:41                     ` Stephen Boyd
2015-11-09  0:04                   ` Maxime Ripard
2015-11-09 21:14                     ` Stephen Boyd
2015-10-26 10:26       ` Michael Turquette
2015-10-26  0:40     ` Olof Johansson

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=20151026103409.GX10947@lukather \
    --to=maxime.ripard@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox