From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: Sascha Hauer <s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
Stephen Boyd <sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
"moderated list:ARM/Mediatek SoC..."
<linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
Matthias Brugger
<matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Yingjoe Chen
<Yingjoe.Chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
Subject: Re: [GIT PULL] ARM: mediatek: arm64 updates for v4.2
Date: Mon, 01 Jun 2015 10:11:07 +0200 [thread overview]
Message-ID: <2507882.daayE1Zgb4@wuerfel> (raw)
In-Reply-To: <CABuKBeJzTXrRRoAAL7m1=HAP8pR5vrp-tK7vj2oym8KYwTA+Cg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On Monday 01 June 2015 10:07:40 Matthias Brugger wrote:
> 2015-05-29 18:46 GMT+02:00 Stephen Boyd <sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>:
> > On 05/29, Arnd Bergmann wrote:
> >> On Friday 29 May 2015 14:58:29 Sascha Hauer wrote:
> >> > On Fri, May 29, 2015 at 02:07:40PM +0200, Arnd Bergmann wrote:
> >> > > On Friday 29 May 2015 14:04:32 Arnd Bergmann wrote:
> >> > > > On Wednesday 27 May 2015 17:14:08 Matthias Brugger wrote:
> >> > > > > - dts: fixup pinctrl mt8173
> >> > > > > - dts: mt8173-evb fix model name
> >> > > > > - dts: mt8173 fix identation
> >> > > > > - dts: mt8173 add clock controller
> >> > > > > - dts: mt8173 change uart clocks
> >> > > > > - dts: mt8173 add pmic wrapper
> >> > > > >
> >> > > >
> >> > > > Pulled into next/dt, thanks!
> >> > > >
> >> > >
> >> > > Dropped it again, after getting a build error:
> >> > >
> >> > > DTC arch/arm64/boot/dts/mediatek/mt8173-evb.dtb
> >> > > In file included from ../arch/arm64/boot/dts/mediatek/mt8173-evb.dts:16:0:
> >> > > ../arch/arm64/boot/dts/mediatek/mt8173.dtsi:14:42: fatal error: dt-bindings/clock/mt8173-clk.h: No such file or directory
> >> > > #include <dt-bindings/clock/mt8173-clk.h>
> >> > > ^
> >> > >
> >> > > Please fix and resubmit.
> >> >
> >> > These come from the clk tree. My original suggestion was to pull in this
> >> > dts changes late after -rc1.
> >>
> >> Please don't do it that way. If you can't figure out a way to
> >> avoid the dependency, just stage it out over an extra merge window
> >> and add the driver first and the dt changes the next time.
> >>
> >> > Other possibility: Stephen, can you provide
> >> > a stable commit containing the Mediatek clk changes?
> >>
> >> That would work.
> >
> > Or another possibility, put the same header file into the branch
> > you send to arm-soc. Git will figure out that the files are
> > exactly the same and there's no cross tree dependency.
>
> OK, I will do that.
>
If you put the same file in two branches, please use the same commit
ID. While git can figure it out when everything goes right, it will
fail horribly if one of the branches later gets a patch on top that
modifies the file, before the two branches get merged.
Arnd
next prev parent reply other threads:[~2015-06-01 8:11 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-27 15:14 [GIT PULL] ARM: mediatek: arm64 updates for v4.2 Matthias Brugger
[not found] ` <CABuKBeLhFR=94cF301wL7J8hJVTtyseoD7g45PLE6znQkPAFGQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-29 12:04 ` Arnd Bergmann
2015-05-29 12:07 ` Arnd Bergmann
2015-05-29 12:58 ` Sascha Hauer
[not found] ` <20150529125829.GE6325-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2015-05-29 14:28 ` Arnd Bergmann
2015-05-29 16:46 ` Stephen Boyd
[not found] ` <20150529164605.GA3022-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2015-06-01 8:07 ` Matthias Brugger
[not found] ` <CABuKBeJzTXrRRoAAL7m1=HAP8pR5vrp-tK7vj2oym8KYwTA+Cg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-01 8:11 ` Arnd Bergmann [this message]
2015-06-01 8:56 ` Matthias Brugger
[not found] ` <CABuKBeK+LT4UDKe0P=vswEXTUevcSNTf2Gn0cdMQ7r=tj2JmOA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-01 14:32 ` Arnd Bergmann
-- strict thread matches above, loose matches on Subject: below --
2015-06-01 10:30 Matthias Brugger
[not found] ` <CABuKBeL1mq_Q3fxQtWeSGV-vHWqiVr1wTVKzj_KA1c27oKxMcQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-01 16:00 ` Arnd Bergmann
2015-07-28 12:06 Matthias Brugger
[not found] ` <CABuKBeK=TQ1CQx4Co2m5p9xri4oBGMKynJL+9evm3xaTjvr8FQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-07-28 16:15 ` Olof Johansson
2015-07-29 8:21 ` Matthias Brugger
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=2507882.daayE1Zgb4@wuerfel \
--to=arnd-r2ngtmty4d4@public.gmane.org \
--cc=Yingjoe.Chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
--cc=sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.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;
as well as URLs for NNTP newsgroup(s).