From: Stephen Boyd <sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Cc: Sascha Hauer <s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@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>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [GIT PULL] ARM: mediatek: arm64 updates for v4.2
Date: Fri, 29 May 2015 09:46:05 -0700 [thread overview]
Message-ID: <20150529164605.GA3022@codeaurora.org> (raw)
In-Reply-To: <2560864.dSIGJOm4yU@wuerfel>
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.
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
WARNING: multiple messages have this Message-ID (diff)
From: sboyd@codeaurora.org (Stephen Boyd)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] ARM: mediatek: arm64 updates for v4.2
Date: Fri, 29 May 2015 09:46:05 -0700 [thread overview]
Message-ID: <20150529164605.GA3022@codeaurora.org> (raw)
In-Reply-To: <2560864.dSIGJOm4yU@wuerfel>
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.
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
next prev parent reply other threads:[~2015-05-29 16:46 UTC|newest]
Thread overview: 30+ 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
2015-05-27 15:14 ` Matthias Brugger
[not found] ` <CABuKBeLhFR=94cF301wL7J8hJVTtyseoD7g45PLE6znQkPAFGQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-29 12:04 ` Arnd Bergmann
2015-05-29 12:04 ` Arnd Bergmann
2015-05-29 12:07 ` Arnd Bergmann
2015-05-29 12:07 ` Arnd Bergmann
2015-05-29 12:58 ` Sascha Hauer
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 14:28 ` Arnd Bergmann
2015-05-29 16:46 ` Stephen Boyd [this message]
2015-05-29 16:46 ` Stephen Boyd
[not found] ` <20150529164605.GA3022-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2015-06-01 8:07 ` Matthias Brugger
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
2015-06-01 8:11 ` Arnd Bergmann
2015-06-01 8:56 ` Matthias Brugger
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
2015-06-01 14:32 ` Arnd Bergmann
-- strict thread matches above, loose matches on Subject: below --
2015-06-01 10:30 Matthias Brugger
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-06-01 16:00 ` Arnd Bergmann
2015-07-28 12:06 Matthias Brugger
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-28 16:15 ` Olof Johansson
2015-07-29 8:21 ` Matthias Brugger
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=20150529164605.GA3022@codeaurora.org \
--to=sboyd-sgv2jx0feol9jmxxk+q4oq@public.gmane.org \
--cc=Yingjoe.Chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=arnd-r2nGTMty4D4@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 \
/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.