* [GIT PULL] ux500 <mach/id.h> removal
@ 2013-01-29 18:06 Linus Walleij
2013-01-30 0:12 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2013-01-29 18:06 UTC (permalink / raw)
To: linux-arm-kernel
Hi ARM SoC maintainers,
This patch set removes the use of <mach/id.h> from being broadcasted across
the kernel and blocks any further usage of cpu_is* outside of the machine.
Target is v3.9 cleanups.
It is part of our path toward single zImage.
Please pull it in! Detailed description in the tag, MFD hunk ACKed by
Samuel Ortiz.
Yours,
Linus Walleij
The following changes since commit d1c3ed669a2d452cacfb48c2d171a1f364dae2ed:
Linux 3.8-rc2 (2013-01-02 18:13:21 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-stericsson.git
tags/ux500-no-idh
for you to fetch changes up to 7a4f26097d389c16c9956bc03b81532698d97d64:
ARM: ux500: de-globalize <mach/id.h> (2013-01-29 18:47:37 +0100)
----------------------------------------------------------------
Removal of the <mach/id.h> include from ux500
- First an ACKed MFD patch deleting the only consumer
of these cpu_is* functions outside of mach-ux500
- Introduce a new local cpu_is_u8580() in this patch
set to avoid clashing with other patch sets.
- Finally de-globalize <mach/id.h>.
----------------------------------------------------------------
Linus Walleij (2):
mfd: prcmu: delete pin control helpers
ARM: ux500: de-globalize <mach/id.h>
Maxime Coquelin (1):
ARM: ux500: Introduce cpu_is_u8580()
arch/arm/mach-ux500/board-mop500-uib.c | 1 +
arch/arm/mach-ux500/cache-l2x0.c | 3 +-
arch/arm/mach-ux500/cpu-db8500.c | 2 +
arch/arm/mach-ux500/cpu.c | 1 +
arch/arm/mach-ux500/id.c | 2 +
arch/arm/mach-ux500/{include/mach => }/id.h | 21 +++++++-
arch/arm/mach-ux500/include/mach/hardware.h | 1 -
arch/arm/mach-ux500/platsmp.c | 3 ++
arch/arm/mach-ux500/timer.c | 2 +
drivers/cpufreq/db8500-cpufreq.c | 3 --
drivers/mfd/db8500-prcmu.c | 17 +++---
include/linux/mfd/db8500-prcmu.h | 6 ---
include/linux/mfd/dbx500-prcmu.h | 81 -----------------------------
13 files changed, 39 insertions(+), 104 deletions(-)
rename arch/arm/mach-ux500/{include/mach => }/id.h (85%)
^ permalink raw reply [flat|nested] 6+ messages in thread* [GIT PULL] ux500 <mach/id.h> removal
2013-01-29 18:06 [GIT PULL] ux500 <mach/id.h> removal Linus Walleij
@ 2013-01-30 0:12 ` Olof Johansson
2013-01-30 9:39 ` Linus Walleij
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2013-01-30 0:12 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Tue, Jan 29, 2013 at 10:06 AM, Linus Walleij
<linus.walleij@linaro.org> wrote:
> Hi ARM SoC maintainers,
>
> This patch set removes the use of <mach/id.h> from being broadcasted across
> the kernel and blocks any further usage of cpu_is* outside of the machine.
>
> Target is v3.9 cleanups.
>
> It is part of our path toward single zImage.
>
> Please pull it in! Detailed description in the tag, MFD hunk ACKed by
> Samuel Ortiz.
>
> Yours,
> Linus Walleij
>
> The following changes since commit d1c3ed669a2d452cacfb48c2d171a1f364dae2ed:
>
> Linux 3.8-rc2 (2013-01-02 18:13:21 -0800)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-stericsson.git
> tags/ux500-no-idh
Pulled in.
This has a somewhat annoying (but trivial) conflict against your own
code (the cpufreq driver changes).
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] ux500 <mach/id.h> removal
2013-01-30 0:12 ` Olof Johansson
@ 2013-01-30 9:39 ` Linus Walleij
2013-01-30 23:15 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2013-01-30 9:39 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 30, 2013 at 1:12 AM, Olof Johansson <olof@lixom.net> wrote:
> [Me]
>> git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-stericsson.git
>> tags/ux500-no-idh
>
> Pulled in.
>
> This has a somewhat annoying (but trivial) conflict against your own
> code (the cpufreq driver changes).
Arnd always told me to split stuff cleanly on branches, which of course
leads to conflicts like that, but I thought he wanted them, rather than
huge accumulated patch sets built on to of each othere ... a bit hard to
do the right thing here, what would have been the right way?
Does it help if I base stuff off ARM SoC tree branches, and which
ones can be relied upon in that case?
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] ux500 <mach/id.h> removal
2013-01-30 9:39 ` Linus Walleij
@ 2013-01-30 23:15 ` Arnd Bergmann
2013-01-31 14:40 ` Linus Walleij
0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2013-01-30 23:15 UTC (permalink / raw)
To: linux-arm-kernel
On Wednesday 30 January 2013, Linus Walleij wrote:
> On Wed, Jan 30, 2013 at 1:12 AM, Olof Johansson <olof@lixom.net> wrote:
> > [Me]
> >> git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-stericsson.git
> >> tags/ux500-no-idh
> >
> > Pulled in.
> >
> > This has a somewhat annoying (but trivial) conflict against your own
> > code (the cpufreq driver changes).
>
> Arnd always told me to split stuff cleanly on branches, which of course
> leads to conflicts like that, but I thought he wanted them, rather than
> huge accumulated patch sets built on to of each othere ... a bit hard to
> do the right thing here, what would have been the right way?
>
> Does it help if I base stuff off ARM SoC tree branches, and which
> ones can be relied upon in that case?
Not sure if it applies here, but in a lot of cases, one can extract the
parts that do conflict and put those patches first in the series, and
merge them together before the pull request.
The classic example of this is two independent features A and B that
both contain some cleanup. By moving the cleanup first, you can merge
the cleanup-A patches with the cleanup-B patches and submit them as one
branch, and then have feature A on top of cleanup-A merge cleanly with
both feature-B (because there are no conflicts) and with cleanup-A-B
(because cleanup-A-B has already resolved the conflict).
Unrelated to this: The mach/id.h removal seems to have caused a few
build errors in the for-next tree with u8500_defconfig. Can you have
a look at what went wrong there?
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] ux500 <mach/id.h> removal
2013-01-30 23:15 ` Arnd Bergmann
@ 2013-01-31 14:40 ` Linus Walleij
2013-02-05 22:22 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2013-01-31 14:40 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jan 31, 2013 at 12:15 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> Unrelated to this: The mach/id.h removal seems to have caused a few
> build errors in the for-next tree with u8500_defconfig. Can you have
> a look at what went wrong there?
Yes ... hm.
I have no clue how this happened, somehow new dependencues
have appeared and I've somehow had a config that builds without
some of the drivers or something :-(
Or I built the wrong branch and thought it was clear to go.
Shall we just drop it until I've fixed it up?
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] ux500 <mach/id.h> removal
2013-01-31 14:40 ` Linus Walleij
@ 2013-02-05 22:22 ` Olof Johansson
0 siblings, 0 replies; 6+ messages in thread
From: Olof Johansson @ 2013-02-05 22:22 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Thu, Jan 31, 2013 at 6:40 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Thu, Jan 31, 2013 at 12:15 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>
>> Unrelated to this: The mach/id.h removal seems to have caused a few
>> build errors in the for-next tree with u8500_defconfig. Can you have
>> a look at what went wrong there?
>
> Yes ... hm.
>
> I have no clue how this happened, somehow new dependencues
> have appeared and I've somehow had a config that builds without
> some of the drivers or something :-(
>
> Or I built the wrong branch and thought it was clear to go.
>
> Shall we just drop it until I've fixed it up?
What's the expected timeline for fixup? I can revert the patch in
for-next and we can take a fix on top of the branch that introduced
the breakage, if that's easier.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-02-05 22:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-29 18:06 [GIT PULL] ux500 <mach/id.h> removal Linus Walleij
2013-01-30 0:12 ` Olof Johansson
2013-01-30 9:39 ` Linus Walleij
2013-01-30 23:15 ` Arnd Bergmann
2013-01-31 14:40 ` Linus Walleij
2013-02-05 22:22 ` Olof Johansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox