* [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
@ 2015-09-06 23:58 Boqun Feng
2015-09-08 17:04 ` Geoff Levand
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Boqun Feng @ 2015-09-06 23:58 UTC (permalink / raw)
To: linuxppc-dev
Cc: linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman, Olof Johansson, Geoff Levand,
Boqun Feng
Currently, little endian is only supported on powernv and pseries,
however, Kconfigs still allow us to include other platforms in a LE
kernel, this may result in space wasting or even build error if some
BE-only platforms always assume they are built for a BE kernel. So just
modify the Kconfigs of BE-only platforms to remove them from being built
for a LE kernel.
For 32bit only platforms, nothing needs to be done, because
CPU_LITTLE_ENDIAN depends on PPC64. For 64bit supported platforms, add
CPU_BIG_ENDIAN to dependencies explicitly, so that these platforms will
be disabled for LE [Suggested-by: Cédric Le Goater <clg@fr.ibm.com>].
Signed-off-by: Boqun Feng <boqun.feng@gmail.com>
---
arch/powerpc/platforms/cell/Kconfig | 4 ++--
arch/powerpc/platforms/maple/Kconfig | 2 +-
arch/powerpc/platforms/pasemi/Kconfig | 2 +-
arch/powerpc/platforms/powermac/Kconfig | 2 +-
arch/powerpc/platforms/ps3/Kconfig | 2 +-
5 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/powerpc/platforms/cell/Kconfig b/arch/powerpc/platforms/cell/Kconfig
index 2f23133..808a904 100644
--- a/arch/powerpc/platforms/cell/Kconfig
+++ b/arch/powerpc/platforms/cell/Kconfig
@@ -25,7 +25,7 @@ config PPC_CELL_NATIVE
config PPC_IBM_CELL_BLADE
bool "IBM Cell Blade"
- depends on PPC64 && PPC_BOOK3S
+ depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
select PPC_CELL_NATIVE
select PPC_OF_PLATFORM_PCI
select PCI
@@ -35,7 +35,7 @@ config PPC_IBM_CELL_BLADE
config PPC_CELL_QPACE
bool "IBM Cell - QPACE"
- depends on PPC64 && PPC_BOOK3S
+ depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
select PPC_CELL_COMMON
config AXON_MSI
diff --git a/arch/powerpc/platforms/maple/Kconfig b/arch/powerpc/platforms/maple/Kconfig
index 1ea621a..e359d0d 100644
--- a/arch/powerpc/platforms/maple/Kconfig
+++ b/arch/powerpc/platforms/maple/Kconfig
@@ -1,5 +1,5 @@
config PPC_MAPLE
- depends on PPC64 && PPC_BOOK3S
+ depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
bool "Maple 970FX Evaluation Board"
select PCI
select MPIC
diff --git a/arch/powerpc/platforms/pasemi/Kconfig b/arch/powerpc/platforms/pasemi/Kconfig
index a2aeb32..00d4b28 100644
--- a/arch/powerpc/platforms/pasemi/Kconfig
+++ b/arch/powerpc/platforms/pasemi/Kconfig
@@ -1,5 +1,5 @@
config PPC_PASEMI
- depends on PPC64 && PPC_BOOK3S
+ depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
bool "PA Semi SoC-based platforms"
default n
select MPIC
diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig
index 607124b..43c6062 100644
--- a/arch/powerpc/platforms/powermac/Kconfig
+++ b/arch/powerpc/platforms/powermac/Kconfig
@@ -1,6 +1,6 @@
config PPC_PMAC
bool "Apple PowerMac based machines"
- depends on PPC_BOOK3S
+ depends on PPC_BOOK3S && CPU_BIG_ENDIAN
select MPIC
select PCI
select PPC_INDIRECT_PCI if PPC32
diff --git a/arch/powerpc/platforms/ps3/Kconfig b/arch/powerpc/platforms/ps3/Kconfig
index 56f2740..b27f40f 100644
--- a/arch/powerpc/platforms/ps3/Kconfig
+++ b/arch/powerpc/platforms/ps3/Kconfig
@@ -1,6 +1,6 @@
config PPC_PS3
bool "Sony PS3"
- depends on PPC64 && PPC_BOOK3S
+ depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
select PPC_CELL
select USB_OHCI_LITTLE_ENDIAN
select USB_OHCI_BIG_ENDIAN_MMIO
--
2.5.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-06 23:58 [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build Boqun Feng
@ 2015-09-08 17:04 ` Geoff Levand
2015-09-09 2:26 ` Michael Ellerman
` (2 subsequent siblings)
3 siblings, 0 replies; 9+ messages in thread
From: Geoff Levand @ 2015-09-08 17:04 UTC (permalink / raw)
To: Boqun Feng
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman, Olof Johansson
On Mon, 2015-09-07 at 07:58 +0800, Boqun Feng wrote:
> arch/powerpc/platforms/ps3/Kconfig | 2 +-
Looks OK for PS3.
Acked-by: Geoff Levand <geoff@infradead.org>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-06 23:58 [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build Boqun Feng
2015-09-08 17:04 ` Geoff Levand
@ 2015-09-09 2:26 ` Michael Ellerman
2015-09-09 3:25 ` Boqun Feng
2015-09-18 0:22 ` Boqun Feng
2015-10-02 7:47 ` Michael Ellerman
3 siblings, 1 reply; 9+ messages in thread
From: Michael Ellerman @ 2015-09-09 2:26 UTC (permalink / raw)
To: Boqun Feng
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Olof Johansson, Geoff Levand
On Mon, 2015-09-07 at 07:58 +0800, Boqun Feng wrote:
> diff --git a/arch/powerpc/platforms/cell/Kconfig b/arch/powerpc/platforms/cell/Kconfig
> index 2f23133..808a904 100644
> --- a/arch/powerpc/platforms/cell/Kconfig
> +++ b/arch/powerpc/platforms/cell/Kconfig
> @@ -25,7 +25,7 @@ config PPC_CELL_NATIVE
>
> config PPC_IBM_CELL_BLADE
> bool "IBM Cell Blade"
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
We end up saying this five times.
We already have PPC_BOOK3S_64 which captures the first two conditions, should
we add a PPC_BOOK3S_64_BE which expresses it all?
cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-09 2:26 ` Michael Ellerman
@ 2015-09-09 3:25 ` Boqun Feng
2015-09-09 5:48 ` Michael Ellerman
0 siblings, 1 reply; 9+ messages in thread
From: Boqun Feng @ 2015-09-09 3:25 UTC (permalink / raw)
To: Michael Ellerman
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Olof Johansson, Geoff Levand
[-- Attachment #1: Type: text/plain, Size: 1426 bytes --]
Hi Michael,
On Wed, Sep 09, 2015 at 12:26:44PM +1000, Michael Ellerman wrote:
> On Mon, 2015-09-07 at 07:58 +0800, Boqun Feng wrote:
> > diff --git a/arch/powerpc/platforms/cell/Kconfig b/arch/powerpc/platforms/cell/Kconfig
> > index 2f23133..808a904 100644
> > --- a/arch/powerpc/platforms/cell/Kconfig
> > +++ b/arch/powerpc/platforms/cell/Kconfig
> > @@ -25,7 +25,7 @@ config PPC_CELL_NATIVE
> >
> > config PPC_IBM_CELL_BLADE
> > bool "IBM Cell Blade"
> > - depends on PPC64 && PPC_BOOK3S
> > + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
>
> We end up saying this five times.
>
> We already have PPC_BOOK3S_64 which captures the first two conditions, should
> we add a PPC_BOOK3S_64_BE which expresses it all?
>
I'm not sure whether this is worth.. IMO, we add a config option only if
1. we can use this config somewhere in the code, for example,
"#ifdef CONFIG_XXX"
or
2. we want to offer a option for users to choose.
PPC_BOOK3S_64_BE satisfies neither condition. Further more,
CONFIG_PPC64, CONFIG_PPC_BOOK3S and CONFIG_CPU_BIG_ENDIAN all are used
in some #ifdefs in current code. As a newbie of kernel, I'm happy to get
some knowledge like: "If I'm hacking PS3, then #ifndef CONFIG_PPC64 is
guaranteed to be false, so I can just ignore the code guarded by them".
But if we add a PPC_BOOK3S_64_BE, it will take a little more effort to
see this.
Regards,
Boqun
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-09 3:25 ` Boqun Feng
@ 2015-09-09 5:48 ` Michael Ellerman
0 siblings, 0 replies; 9+ messages in thread
From: Michael Ellerman @ 2015-09-09 5:48 UTC (permalink / raw)
To: Boqun Feng
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Olof Johansson, Geoff Levand
On Wed, 2015-09-09 at 11:25 +0800, Boqun Feng wrote:
> Hi Michael,
>
> On Wed, Sep 09, 2015 at 12:26:44PM +1000, Michael Ellerman wrote:
> > On Mon, 2015-09-07 at 07:58 +0800, Boqun Feng wrote:
> > > diff --git a/arch/powerpc/platforms/cell/Kconfig b/arch/powerpc/platforms/cell/Kconfig
> > > index 2f23133..808a904 100644
> > > --- a/arch/powerpc/platforms/cell/Kconfig
> > > +++ b/arch/powerpc/platforms/cell/Kconfig
> > > @@ -25,7 +25,7 @@ config PPC_CELL_NATIVE
> > >
> > > config PPC_IBM_CELL_BLADE
> > > bool "IBM Cell Blade"
> > > - depends on PPC64 && PPC_BOOK3S
> > > + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> >
> > We end up saying this five times.
> >
> > We already have PPC_BOOK3S_64 which captures the first two conditions, should
> > we add a PPC_BOOK3S_64_BE which expresses it all?
>
> I'm not sure whether this is worth.. IMO, we add a config option only if
>
> 1. we can use this config somewhere in the code, for example,
> "#ifdef CONFIG_XXX"
> or
>
> 2. we want to offer a option for users to choose.
>
> PPC_BOOK3S_64_BE satisfies neither condition. Further more,
> CONFIG_PPC64, CONFIG_PPC_BOOK3S and CONFIG_CPU_BIG_ENDIAN all are used
> in some #ifdefs in current code. As a newbie of kernel, I'm happy to get
> some knowledge like: "If I'm hacking PS3, then #ifndef CONFIG_PPC64 is
> guaranteed to be false, so I can just ignore the code guarded by them".
> But if we add a PPC_BOOK3S_64_BE, it will take a little more effort to
> see this.
Yeah those are good points, so I'll take it as is. Thanks for thinking about it :)
cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-06 23:58 [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build Boqun Feng
2015-09-08 17:04 ` Geoff Levand
2015-09-09 2:26 ` Michael Ellerman
@ 2015-09-18 0:22 ` Boqun Feng
2015-09-18 9:49 ` Michael Ellerman
2015-10-02 7:47 ` Michael Ellerman
3 siblings, 1 reply; 9+ messages in thread
From: Boqun Feng @ 2015-09-18 0:22 UTC (permalink / raw)
To: linuxppc-dev
Cc: linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Michael Ellerman, Olof Johansson, Geoff Levand
[-- Attachment #1: Type: text/plain, Size: 3646 bytes --]
Ping ;-)
Regards,
Boqun
On Mon, Sep 07, 2015 at 07:58:00AM +0800, Boqun Feng wrote:
> Currently, little endian is only supported on powernv and pseries,
> however, Kconfigs still allow us to include other platforms in a LE
> kernel, this may result in space wasting or even build error if some
> BE-only platforms always assume they are built for a BE kernel. So just
> modify the Kconfigs of BE-only platforms to remove them from being built
> for a LE kernel.
>
> For 32bit only platforms, nothing needs to be done, because
> CPU_LITTLE_ENDIAN depends on PPC64. For 64bit supported platforms, add
> CPU_BIG_ENDIAN to dependencies explicitly, so that these platforms will
> be disabled for LE [Suggested-by: Cédric Le Goater <clg@fr.ibm.com>].
>
> Signed-off-by: Boqun Feng <boqun.feng@gmail.com>
> ---
> arch/powerpc/platforms/cell/Kconfig | 4 ++--
> arch/powerpc/platforms/maple/Kconfig | 2 +-
> arch/powerpc/platforms/pasemi/Kconfig | 2 +-
> arch/powerpc/platforms/powermac/Kconfig | 2 +-
> arch/powerpc/platforms/ps3/Kconfig | 2 +-
> 5 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/powerpc/platforms/cell/Kconfig b/arch/powerpc/platforms/cell/Kconfig
> index 2f23133..808a904 100644
> --- a/arch/powerpc/platforms/cell/Kconfig
> +++ b/arch/powerpc/platforms/cell/Kconfig
> @@ -25,7 +25,7 @@ config PPC_CELL_NATIVE
>
> config PPC_IBM_CELL_BLADE
> bool "IBM Cell Blade"
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> select PPC_CELL_NATIVE
> select PPC_OF_PLATFORM_PCI
> select PCI
> @@ -35,7 +35,7 @@ config PPC_IBM_CELL_BLADE
>
> config PPC_CELL_QPACE
> bool "IBM Cell - QPACE"
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> select PPC_CELL_COMMON
>
> config AXON_MSI
> diff --git a/arch/powerpc/platforms/maple/Kconfig b/arch/powerpc/platforms/maple/Kconfig
> index 1ea621a..e359d0d 100644
> --- a/arch/powerpc/platforms/maple/Kconfig
> +++ b/arch/powerpc/platforms/maple/Kconfig
> @@ -1,5 +1,5 @@
> config PPC_MAPLE
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> bool "Maple 970FX Evaluation Board"
> select PCI
> select MPIC
> diff --git a/arch/powerpc/platforms/pasemi/Kconfig b/arch/powerpc/platforms/pasemi/Kconfig
> index a2aeb32..00d4b28 100644
> --- a/arch/powerpc/platforms/pasemi/Kconfig
> +++ b/arch/powerpc/platforms/pasemi/Kconfig
> @@ -1,5 +1,5 @@
> config PPC_PASEMI
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> bool "PA Semi SoC-based platforms"
> default n
> select MPIC
> diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig
> index 607124b..43c6062 100644
> --- a/arch/powerpc/platforms/powermac/Kconfig
> +++ b/arch/powerpc/platforms/powermac/Kconfig
> @@ -1,6 +1,6 @@
> config PPC_PMAC
> bool "Apple PowerMac based machines"
> - depends on PPC_BOOK3S
> + depends on PPC_BOOK3S && CPU_BIG_ENDIAN
> select MPIC
> select PCI
> select PPC_INDIRECT_PCI if PPC32
> diff --git a/arch/powerpc/platforms/ps3/Kconfig b/arch/powerpc/platforms/ps3/Kconfig
> index 56f2740..b27f40f 100644
> --- a/arch/powerpc/platforms/ps3/Kconfig
> +++ b/arch/powerpc/platforms/ps3/Kconfig
> @@ -1,6 +1,6 @@
> config PPC_PS3
> bool "Sony PS3"
> - depends on PPC64 && PPC_BOOK3S
> + depends on PPC64 && PPC_BOOK3S && CPU_BIG_ENDIAN
> select PPC_CELL
> select USB_OHCI_LITTLE_ENDIAN
> select USB_OHCI_BIG_ENDIAN_MMIO
> --
> 2.5.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-18 0:22 ` Boqun Feng
@ 2015-09-18 9:49 ` Michael Ellerman
2015-09-18 9:59 ` Boqun Feng
0 siblings, 1 reply; 9+ messages in thread
From: Michael Ellerman @ 2015-09-18 9:49 UTC (permalink / raw)
To: Boqun Feng
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Olof Johansson, Geoff Levand
On Fri, 2015-09-18 at 08:22 +0800, Boqun Feng wrote:
> Ping ;-)
Hi Boqun,
We keep track of patches in patchwork:
http://patchwork.ozlabs.org/project/linuxppc-dev/list/?submitter=67102
So your patch won't be forgotten.
I tend to start taking patches for next after rc2, which should come out on
Sunday. So I'll probably merge this patch next week, when I do you'll get an
email.
cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-18 9:49 ` Michael Ellerman
@ 2015-09-18 9:59 ` Boqun Feng
0 siblings, 0 replies; 9+ messages in thread
From: Boqun Feng @ 2015-09-18 9:59 UTC (permalink / raw)
To: Michael Ellerman
Cc: linuxppc-dev, linux-kernel, Arnd Bergmann, Benjamin Herrenschmidt,
Paul Mackerras, Olof Johansson, Geoff Levand
[-- Attachment #1: Type: text/plain, Size: 577 bytes --]
On Fri, Sep 18, 2015 at 07:49:56PM +1000, Michael Ellerman wrote:
> On Fri, 2015-09-18 at 08:22 +0800, Boqun Feng wrote:
> > Ping ;-)
>
> Hi Boqun,
>
Hello,
> We keep track of patches in patchwork:
>
> http://patchwork.ozlabs.org/project/linuxppc-dev/list/?submitter=67102
>
Good to know ;-)
> So your patch won't be forgotten.
>
> I tend to start taking patches for next after rc2, which should come out on
> Sunday. So I'll probably merge this patch next week, when I do you'll get an
> email.
>
Understood. Thank you!
Regards,
Boqun
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: powerpc: Kconfig: remove BE-only platforms from LE kernel build
2015-09-06 23:58 [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build Boqun Feng
` (2 preceding siblings ...)
2015-09-18 0:22 ` Boqun Feng
@ 2015-10-02 7:47 ` Michael Ellerman
3 siblings, 0 replies; 9+ messages in thread
From: Michael Ellerman @ 2015-10-02 7:47 UTC (permalink / raw)
To: Boqun Feng, linuxppc-dev
Cc: Arnd Bergmann, Geoff Levand, Boqun Feng, linux-kernel,
Paul Mackerras, Olof Johansson
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 919 bytes --]
On Sun, 2015-06-09 at 23:58:00 UTC, Boqun Feng wrote:
> Currently, little endian is only supported on powernv and pseries,
> however, Kconfigs still allow us to include other platforms in a LE
> kernel, this may result in space wasting or even build error if some
> BE-only platforms always assume they are built for a BE kernel. So just
> modify the Kconfigs of BE-only platforms to remove them from being built
> for a LE kernel.
>
> For 32bit only platforms, nothing needs to be done, because
> CPU_LITTLE_ENDIAN depends on PPC64. For 64bit supported platforms, add
> CPU_BIG_ENDIAN to dependencies explicitly, so that these platforms will
> be disabled for LE [Suggested-by: Cédric Le Goater <clg@fr.ibm.com>].
>
> Signed-off-by: Boqun Feng <boqun.feng@gmail.com>
> Acked-by: Geoff Levand <geoff@infradead.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/e5e16d8f3ec6973af2068897
cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-10-02 7:47 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-06 23:58 [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build Boqun Feng
2015-09-08 17:04 ` Geoff Levand
2015-09-09 2:26 ` Michael Ellerman
2015-09-09 3:25 ` Boqun Feng
2015-09-09 5:48 ` Michael Ellerman
2015-09-18 0:22 ` Boqun Feng
2015-09-18 9:49 ` Michael Ellerman
2015-09-18 9:59 ` Boqun Feng
2015-10-02 7:47 ` Michael Ellerman
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).