* [PATCH] MIPS: IP22/IP32: Add missing ifdefs to Platform files
@ 2014-10-08 1:43 Joshua Kinard
2014-10-08 10:34 ` Markos Chandras
0 siblings, 1 reply; 4+ messages in thread
From: Joshua Kinard @ 2014-10-08 1:43 UTC (permalink / raw)
To: Ralf Baechle; +Cc: Linux MIPS List
In arch/mips/sgi-ip22/Platform and arch/mips/sgi-ip32/Platform, ifdefs for
CONFIG_SGI_IP22 and CONFIG_SGI_IP32 are missing, which can cause the
definitions for these platforms to get included in builds for other platforms.
This patch adds these missing ifdefs, which matches IP27's Platform file.
Signed-off-by: Joshua Kinard <kumba@gentoo.org>
---
arch/mips/sgi-ip22/Platform | 8 +++++---
arch/mips/sgi-ip32/Platform | 9 ++++++---
2 files changed, 11 insertions(+), 6 deletions(-)
diff --git a/arch/mips/sgi-ip22/Platform b/arch/mips/sgi-ip22/Platform
index b7a4b7e..5fa3c7a 100644
--- a/arch/mips/sgi-ip22/Platform
+++ b/arch/mips/sgi-ip22/Platform
@@ -7,7 +7,8 @@
# current variable will break so for 64-bit kernels we have to raise the start
# address by 8kb.
#
-platform-$(CONFIG_SGI_IP22) += sgi-ip22/
+ifdef CONFIG_SGI_IP22
+platform-$(CONFIG_SGI_IP22) += sgi-ip22/
cflags-$(CONFIG_SGI_IP22) += -I$(srctree)/arch/mips/include/asm/mach-ip22
ifdef CONFIG_32BIT
load-$(CONFIG_SGI_IP22) += 0xffffffff88002000
@@ -15,6 +16,7 @@ endif
ifdef CONFIG_64BIT
load-$(CONFIG_SGI_IP22) += 0xffffffff88004000
endif
+endif
#
# SGI IP28 (Indigo2 R10k)
@@ -28,7 +30,7 @@ ifdef CONFIG_SGI_IP28
ifeq ($(call cc-option-yn,-mr10k-cache-barrier=store), n)
$(error gcc doesn't support needed option -mr10k-cache-barrier=store)
endif
-endif
-platform-$(CONFIG_SGI_IP28) += sgi-ip22/
+platform-$(CONFIG_SGI_IP28) += sgi-ip22/
cflags-$(CONFIG_SGI_IP28) += -mr10k-cache-barrier=store -I$(srctree)/arch/mips/include/asm/mach-ip28
load-$(CONFIG_SGI_IP28) += 0xa800000020004000
+endif
diff --git a/arch/mips/sgi-ip32/Platform b/arch/mips/sgi-ip32/Platform
index 0fea556..ab95705 100644
--- a/arch/mips/sgi-ip32/Platform
+++ b/arch/mips/sgi-ip32/Platform
@@ -6,6 +6,9 @@
# a multiple of the kernel stack size or the handling of the current variable
# will break.
#
-platform-$(CONFIG_SGI_IP32) += sgi-ip32/
-cflags-$(CONFIG_SGI_IP32) += -I$(srctree)/arch/mips/include/asm/mach-ip32
-load-$(CONFIG_SGI_IP32) += 0xffffffff80004000
+ifdef CONFIG_SGI_IP32
+platform-$(CONFIG_SGI_IP32) += sgi-ip32/
+cflags-$(CONFIG_SGI_IP32) += -I$(srctree)/arch/mips/include/asm/mach-ip32
+load-$(CONFIG_SGI_IP32) += 0xffffffff80004000
+endif
+
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] MIPS: IP22/IP32: Add missing ifdefs to Platform files
@ 2014-10-08 10:34 ` Markos Chandras
0 siblings, 0 replies; 4+ messages in thread
From: Markos Chandras @ 2014-10-08 10:34 UTC (permalink / raw)
To: linux-mips
On 10/08/2014 02:43 AM, Joshua Kinard wrote:
> In arch/mips/sgi-ip22/Platform and arch/mips/sgi-ip32/Platform, ifdefs for
> CONFIG_SGI_IP22 and CONFIG_SGI_IP32 are missing, which can cause the
> definitions for these platforms to get included in builds for other platforms.
> This patch adds these missing ifdefs, which matches IP27's Platform file.
>
> Signed-off-by: Joshua Kinard <kumba@gentoo.org>
> ---
> arch/mips/sgi-ip22/Platform | 8 +++++---
> arch/mips/sgi-ip32/Platform | 9 ++++++---
> 2 files changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/arch/mips/sgi-ip22/Platform b/arch/mips/sgi-ip22/Platform
> index b7a4b7e..5fa3c7a 100644
> --- a/arch/mips/sgi-ip22/Platform
> +++ b/arch/mips/sgi-ip22/Platform
> @@ -7,7 +7,8 @@
> # current variable will break so for 64-bit kernels we have to raise the start
> # address by 8kb.
> #
> -platform-$(CONFIG_SGI_IP22) += sgi-ip22/
> +ifdef CONFIG_SGI_IP22
> +platform-$(CONFIG_SGI_IP22) += sgi-ip22/
> cflags-$(CONFIG_SGI_IP22) += -I$(srctree)/arch/mips/include/asm/mach-ip22
> ifdef CONFIG_32BIT
> load-$(CONFIG_SGI_IP22) += 0xffffffff88002000
> @@ -15,6 +16,7 @@ endif
> ifdef CONFIG_64BIT
> load-$(CONFIG_SGI_IP22) += 0xffffffff88004000
> endif
> +endif
>
I could be wrong but isn't that functionally the same thing? As in, if
CONFIG_SGI_IP22 is not enabled, the sgi-ip22 etc are not included.
That's the same thing as the original code was doing no? Why do you need
to hide all the $FOO-$(CONFIG_SGI_IP22) in a separate #ifdef block. What
problem are you trying to solve?
--
markos
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] MIPS: IP22/IP32: Add missing ifdefs to Platform files
@ 2014-10-08 10:34 ` Markos Chandras
0 siblings, 0 replies; 4+ messages in thread
From: Markos Chandras @ 2014-10-08 10:34 UTC (permalink / raw)
To: linux-mips
On 10/08/2014 02:43 AM, Joshua Kinard wrote:
> In arch/mips/sgi-ip22/Platform and arch/mips/sgi-ip32/Platform, ifdefs for
> CONFIG_SGI_IP22 and CONFIG_SGI_IP32 are missing, which can cause the
> definitions for these platforms to get included in builds for other platforms.
> This patch adds these missing ifdefs, which matches IP27's Platform file.
>
> Signed-off-by: Joshua Kinard <kumba@gentoo.org>
> ---
> arch/mips/sgi-ip22/Platform | 8 +++++---
> arch/mips/sgi-ip32/Platform | 9 ++++++---
> 2 files changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/arch/mips/sgi-ip22/Platform b/arch/mips/sgi-ip22/Platform
> index b7a4b7e..5fa3c7a 100644
> --- a/arch/mips/sgi-ip22/Platform
> +++ b/arch/mips/sgi-ip22/Platform
> @@ -7,7 +7,8 @@
> # current variable will break so for 64-bit kernels we have to raise the start
> # address by 8kb.
> #
> -platform-$(CONFIG_SGI_IP22) += sgi-ip22/
> +ifdef CONFIG_SGI_IP22
> +platform-$(CONFIG_SGI_IP22) += sgi-ip22/
> cflags-$(CONFIG_SGI_IP22) += -I$(srctree)/arch/mips/include/asm/mach-ip22
> ifdef CONFIG_32BIT
> load-$(CONFIG_SGI_IP22) += 0xffffffff88002000
> @@ -15,6 +16,7 @@ endif
> ifdef CONFIG_64BIT
> load-$(CONFIG_SGI_IP22) += 0xffffffff88004000
> endif
> +endif
>
I could be wrong but isn't that functionally the same thing? As in, if
CONFIG_SGI_IP22 is not enabled, the sgi-ip22 etc are not included.
That's the same thing as the original code was doing no? Why do you need
to hide all the $FOO-$(CONFIG_SGI_IP22) in a separate #ifdef block. What
problem are you trying to solve?
--
markos
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] MIPS: IP22/IP32: Add missing ifdefs to Platform files
2014-10-08 10:34 ` Markos Chandras
(?)
@ 2014-10-08 11:30 ` Joshua Kinard
-1 siblings, 0 replies; 4+ messages in thread
From: Joshua Kinard @ 2014-10-08 11:30 UTC (permalink / raw)
To: linux-mips
On 10/08/2014 06:34, Markos Chandras wrote:
> On 10/08/2014 02:43 AM, Joshua Kinard wrote:
>> In arch/mips/sgi-ip22/Platform and arch/mips/sgi-ip32/Platform, ifdefs for
>> CONFIG_SGI_IP22 and CONFIG_SGI_IP32 are missing, which can cause the
>> definitions for these platforms to get included in builds for other platforms.
>> This patch adds these missing ifdefs, which matches IP27's Platform file.
>>
>> Signed-off-by: Joshua Kinard <kumba@gentoo.org>
>> ---
>> arch/mips/sgi-ip22/Platform | 8 +++++---
>> arch/mips/sgi-ip32/Platform | 9 ++++++---
>> 2 files changed, 11 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/mips/sgi-ip22/Platform b/arch/mips/sgi-ip22/Platform
>> index b7a4b7e..5fa3c7a 100644
>> --- a/arch/mips/sgi-ip22/Platform
>> +++ b/arch/mips/sgi-ip22/Platform
>> @@ -7,7 +7,8 @@
>> # current variable will break so for 64-bit kernels we have to raise the start
>> # address by 8kb.
>> #
>> -platform-$(CONFIG_SGI_IP22) += sgi-ip22/
>> +ifdef CONFIG_SGI_IP22
>> +platform-$(CONFIG_SGI_IP22) += sgi-ip22/
>> cflags-$(CONFIG_SGI_IP22) += -I$(srctree)/arch/mips/include/asm/mach-ip22
>> ifdef CONFIG_32BIT
>> load-$(CONFIG_SGI_IP22) += 0xffffffff88002000
>> @@ -15,6 +16,7 @@ endif
>> ifdef CONFIG_64BIT
>> load-$(CONFIG_SGI_IP22) += 0xffffffff88004000
>> endif
>> +endif
>>
>
> I could be wrong but isn't that functionally the same thing? As in, if
> CONFIG_SGI_IP22 is not enabled, the sgi-ip22 etc are not included.
> That's the same thing as the original code was doing no? Why do you need
> to hide all the $FOO-$(CONFIG_SGI_IP22) in a separate #ifdef block. What
> problem are you trying to solve?
That's what I thought, but one of my builds in 3.15 for Octane (IP30) was
pulling in IP32's Platform file for some reason, which caused a build error. I
could not figure it out. I played with building an IP27 kernel some, and
figured out it wasn't affected, so after looking at its Platform file, I
noticed the ifdef. Looking at IP32, that wasn't there, so I added it, and the
build problem went away. I did the same for IP22 just for consistency.
I figure kbuild or make was just getting confused somewhere and globbing IP3*.
IP22 wouldn't be affected because IP27 has an ifdef present, but IP30 was
somehow affected because IP32 lacked the ifdef. No one else has probably run
into it because IP30 isn't officially in the tree.
--
Joshua Kinard
Gentoo/MIPS
kumba@gentoo.org
4096R/D25D95E3 2011-03-28
"The past tempts us, the present confuses us, the future frightens us. And our
lives slip away, moment by moment, lost in that vast, terrible in-between."
--Emperor Turhan, Centauri Republic
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-10-08 11:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-08 1:43 [PATCH] MIPS: IP22/IP32: Add missing ifdefs to Platform files Joshua Kinard
2014-10-08 10:34 ` Markos Chandras
2014-10-08 10:34 ` Markos Chandras
2014-10-08 11:30 ` Joshua Kinard
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.