[2/4] ARM: shmobile: Set CONFIG_LOCALVERSION in defconfigs

Message ID	1388764128-27686-2-git-send-email-geert@linux-m68k.org (mailing list archive)
State	New, archived
Headers	show Return-Path: <linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org> From: Geert Uytterhoeven <geert@linux-m68k.org> To: Simon Horman <horms@verge.net.au>, Magnus Damm <magnus.damm@gmail.com> Subject: [PATCH 2/4] ARM: shmobile: Set CONFIG_LOCALVERSION in defconfigs Date: Fri, 3 Jan 2014 16:48:46 +0100 Message-Id: <1388764128-27686-2-git-send-email-geert@linux-m68k.org> In-Reply-To: <1388764128-27686-1-git-send-email-geert@linux-m68k.org> References: <1388764128-27686-1-git-send-email-geert@linux-m68k.org> Cc: Geert Uytterhoeven <geert+renesas@linux-m68k.org>, linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org> Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org

Message ID

1388764128-27686-2-git-send-email-geert@linux-m68k.org (mailing list archive)

State

New, archived

Headers

From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Simon Horman <horms@verge.net.au>,
	Magnus Damm <magnus.damm@gmail.com>
Subject: [PATCH 2/4] ARM: shmobile: Set CONFIG_LOCALVERSION in defconfigs
Date: Fri,  3 Jan 2014 16:48:46 +0100
Message-Id: <1388764128-27686-2-git-send-email-geert@linux-m68k.org>
In-Reply-To: <1388764128-27686-1-git-send-email-geert@linux-m68k.org>
References: <1388764128-27686-1-git-send-email-geert@linux-m68k.org>
Cc: Geert Uytterhoeven <geert+renesas@linux-m68k.org>,
	linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org>
Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org

Commit Message

Geert Uytterhoeven Jan. 3, 2014, 3:48 p.m. UTC

From: Geert Uytterhoeven <geert+renesas@linux-m68k.org>

This allows to derive the kernel flavor from the kernel version.

Signed-off-by: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
---
 arch/arm/configs/ape6evm_defconfig         |    1 +
 arch/arm/configs/armadillo800eva_defconfig |    1 +
 arch/arm/configs/bockw_defconfig           |    1 +
 arch/arm/configs/genmai_defconfig          |    1 +
 arch/arm/configs/koelsch_defconfig         |    1 +
 arch/arm/configs/kzm9d_defconfig           |    1 +
 arch/arm/configs/kzm9g_defconfig           |    1 +
 arch/arm/configs/lager_defconfig           |    1 +
 arch/arm/configs/mackerel_defconfig        |    1 +
 arch/arm/configs/marzen_defconfig          |    1 +
 10 files changed, 10 insertions(+)

Comments

Simon Horman Jan. 6, 2014, 2:27 a.m. UTC | #1

On Fri, Jan 03, 2014 at 04:48:46PM +0100, Geert Uytterhoeven wrote:
> From: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
> 
> This allows to derive the kernel flavor from the kernel version.

Is this a common practice for ARM?

I'm a little hesitant to take this if its not.
Particularly as we are planing to deprecate these defconfigs
in favour of a shared multiplatform defconfig.

> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
> ---
>  arch/arm/configs/ape6evm_defconfig         |    1 +
>  arch/arm/configs/armadillo800eva_defconfig |    1 +
>  arch/arm/configs/bockw_defconfig           |    1 +
>  arch/arm/configs/genmai_defconfig          |    1 +
>  arch/arm/configs/koelsch_defconfig         |    1 +
>  arch/arm/configs/kzm9d_defconfig           |    1 +
>  arch/arm/configs/kzm9g_defconfig           |    1 +
>  arch/arm/configs/lager_defconfig           |    1 +
>  arch/arm/configs/mackerel_defconfig        |    1 +
>  arch/arm/configs/marzen_defconfig          |    1 +
>  10 files changed, 10 insertions(+)
> 
> diff --git a/arch/arm/configs/ape6evm_defconfig b/arch/arm/configs/ape6evm_defconfig
> index c13f560f07db..554611c82d04 100644
> --- a/arch/arm/configs/ape6evm_defconfig
> +++ b/arch/arm/configs/ape6evm_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-ape6evm"
>  CONFIG_SYSVIPC=y
>  CONFIG_POSIX_MQUEUE=y
>  CONFIG_NO_HZ=y
> diff --git a/arch/arm/configs/armadillo800eva_defconfig b/arch/arm/configs/armadillo800eva_defconfig
> index 1edc2ce32063..2e8212d4c51e 100644
> --- a/arch/arm/configs/armadillo800eva_defconfig
> +++ b/arch/arm/configs/armadillo800eva_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-armadillo800eva"
>  CONFIG_EXPERIMENTAL=y
>  CONFIG_SYSVIPC=y
>  CONFIG_IKCONFIG=y
> diff --git a/arch/arm/configs/bockw_defconfig b/arch/arm/configs/bockw_defconfig
> index 564eab338f02..0f583fecfc69 100644
> --- a/arch/arm/configs/bockw_defconfig
> +++ b/arch/arm/configs/bockw_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-bockw"
>  # CONFIG_ARM_PATCH_PHYS_VIRT is not set
>  CONFIG_KERNEL_LZMA=y
>  CONFIG_NO_HZ=y
> diff --git a/arch/arm/configs/genmai_defconfig b/arch/arm/configs/genmai_defconfig
> index 5fa44cb6e915..ede35f400df2 100644
> --- a/arch/arm/configs/genmai_defconfig
> +++ b/arch/arm/configs/genmai_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-genmai"
>  CONFIG_SYSVIPC=y
>  CONFIG_NO_HZ=y
>  CONFIG_IKCONFIG=y
> diff --git a/arch/arm/configs/koelsch_defconfig b/arch/arm/configs/koelsch_defconfig
> index c8ebeb1f0497..f795ea42a8fc 100644
> --- a/arch/arm/configs/koelsch_defconfig
> +++ b/arch/arm/configs/koelsch_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-koelsch"
>  CONFIG_SYSVIPC=y
>  CONFIG_NO_HZ=y
>  CONFIG_IKCONFIG=y
> diff --git a/arch/arm/configs/kzm9d_defconfig b/arch/arm/configs/kzm9d_defconfig
> index 3ee3858939d9..2d1b9b3740dd 100644
> --- a/arch/arm/configs/kzm9d_defconfig
> +++ b/arch/arm/configs/kzm9d_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-kzm9d"
>  # CONFIG_ARM_PATCH_PHYS_VIRT is not set
>  CONFIG_EXPERIMENTAL=y
>  CONFIG_SYSVIPC=y
> diff --git a/arch/arm/configs/kzm9g_defconfig b/arch/arm/configs/kzm9g_defconfig
> index 800faff9df1d..26a3303bcdf7 100644
> --- a/arch/arm/configs/kzm9g_defconfig
> +++ b/arch/arm/configs/kzm9g_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-kzm9g"
>  # CONFIG_ARM_PATCH_PHYS_VIRT is not set
>  CONFIG_EXPERIMENTAL=y
>  # CONFIG_LOCALVERSION_AUTO is not set
> diff --git a/arch/arm/configs/lager_defconfig b/arch/arm/configs/lager_defconfig
> index da8fa5e8719b..18b9dd96250a 100644
> --- a/arch/arm/configs/lager_defconfig
> +++ b/arch/arm/configs/lager_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-lager"
>  CONFIG_SYSVIPC=y
>  CONFIG_NO_HZ=y
>  CONFIG_IKCONFIG=y
> diff --git a/arch/arm/configs/mackerel_defconfig b/arch/arm/configs/mackerel_defconfig
> index 9d406022d72b..f303b8c5b7d2 100644
> --- a/arch/arm/configs/mackerel_defconfig
> +++ b/arch/arm/configs/mackerel_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-mackerel"
>  CONFIG_EXPERIMENTAL=y
>  CONFIG_SYSVIPC=y
>  CONFIG_IKCONFIG=y
> diff --git a/arch/arm/configs/marzen_defconfig b/arch/arm/configs/marzen_defconfig
> index fe3d8f7c0a2c..b89105b12ae6 100644
> --- a/arch/arm/configs/marzen_defconfig
> +++ b/arch/arm/configs/marzen_defconfig
> @@ -1,3 +1,4 @@
> +CONFIG_LOCALVERSION="-marzen"
>  # CONFIG_ARM_PATCH_PHYS_VIRT is not set
>  CONFIG_EXPERIMENTAL=y
>  CONFIG_KERNEL_LZMA=y
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Geert Uytterhoeven Jan. 6, 2014, 8:43 a.m. UTC | #2

On Mon, Jan 6, 2014 at 3:27 AM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jan 03, 2014 at 04:48:46PM +0100, Geert Uytterhoeven wrote:
>> From: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
>>
>> This allows to derive the kernel flavor from the kernel version.
>
> Is this a common practice for ARM?

Not really, only 5 ARM deconfigs do that.
Globally, ca. 10% of the Linux defconfigs do it.

> I'm a little hesitant to take this if its not.
> Particularly as we are planing to deprecate these defconfigs
> in favour of a shared multiplatform defconfig.

A shared multiplatform defconfig can still have
CONFIG_LOCALVERSION="-shmobile", so you'll know it's a multiplatform
kernel when you see the dmesg.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Simon Horman Jan. 6, 2014, 8:47 a.m. UTC | #3

On Mon, Jan 06, 2014 at 09:43:11AM +0100, Geert Uytterhoeven wrote:
> On Mon, Jan 6, 2014 at 3:27 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Jan 03, 2014 at 04:48:46PM +0100, Geert Uytterhoeven wrote:
> >> From: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
> >>
> >> This allows to derive the kernel flavor from the kernel version.
> >
> > Is this a common practice for ARM?
> 
> Not really, only 5 ARM deconfigs do that.
> Globally, ca. 10% of the Linux defconfigs do it.
> 
> > I'm a little hesitant to take this if its not.
> > Particularly as we are planing to deprecate these defconfigs
> > in favour of a shared multiplatform defconfig.
> 
> A shared multiplatform defconfig can still have
> CONFIG_LOCALVERSION="-shmobile", so you'll know it's a multiplatform
> kernel when you see the dmesg.

Not if it is shared more widely :^)

Simon Horman Jan. 7, 2014, 1:04 a.m. UTC | #4

On Mon, Jan 06, 2014 at 05:47:40PM +0900, Simon Horman wrote:
> On Mon, Jan 06, 2014 at 09:43:11AM +0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 6, 2014 at 3:27 AM, Simon Horman <horms@verge.net.au> wrote:
> > > On Fri, Jan 03, 2014 at 04:48:46PM +0100, Geert Uytterhoeven wrote:
> > >> From: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
> > >>
> > >> This allows to derive the kernel flavor from the kernel version.
> > >
> > > Is this a common practice for ARM?
> > 
> > Not really, only 5 ARM deconfigs do that.
> > Globally, ca. 10% of the Linux defconfigs do it.
> > 
> > > I'm a little hesitant to take this if its not.
> > > Particularly as we are planing to deprecate these defconfigs
> > > in favour of a shared multiplatform defconfig.
> > 
> > A shared multiplatform defconfig can still have
> > CONFIG_LOCALVERSION="-shmobile", so you'll know it's a multiplatform
> > kernel when you see the dmesg.
> 
> Not if it is shared more widely :^)

I'm not really sold on this patch as I don't truly understand
its motivation and I am reluctant to move towards an
uncommon practice for ARM. I am not planning to take this
patch at this time.

diff --git a/arch/arm/configs/ape6evm_defconfig b/arch/arm/configs/ape6evm_defconfig
index c13f560f07db..554611c82d04 100644
--- a/arch/arm/configs/ape6evm_defconfig
+++ b/arch/arm/configs/ape6evm_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-ape6evm"
 CONFIG_SYSVIPC=y
 CONFIG_POSIX_MQUEUE=y
 CONFIG_NO_HZ=y
diff --git a/arch/arm/configs/armadillo800eva_defconfig b/arch/arm/configs/armadillo800eva_defconfig
index 1edc2ce32063..2e8212d4c51e 100644
--- a/arch/arm/configs/armadillo800eva_defconfig
+++ b/arch/arm/configs/armadillo800eva_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-armadillo800eva"
 CONFIG_EXPERIMENTAL=y
 CONFIG_SYSVIPC=y
 CONFIG_IKCONFIG=y
diff --git a/arch/arm/configs/bockw_defconfig b/arch/arm/configs/bockw_defconfig
index 564eab338f02..0f583fecfc69 100644
--- a/arch/arm/configs/bockw_defconfig
+++ b/arch/arm/configs/bockw_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-bockw"
 # CONFIG_ARM_PATCH_PHYS_VIRT is not set
 CONFIG_KERNEL_LZMA=y
 CONFIG_NO_HZ=y
diff --git a/arch/arm/configs/genmai_defconfig b/arch/arm/configs/genmai_defconfig
index 5fa44cb6e915..ede35f400df2 100644
--- a/arch/arm/configs/genmai_defconfig
+++ b/arch/arm/configs/genmai_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-genmai"
 CONFIG_SYSVIPC=y
 CONFIG_NO_HZ=y
 CONFIG_IKCONFIG=y
diff --git a/arch/arm/configs/koelsch_defconfig b/arch/arm/configs/koelsch_defconfig
index c8ebeb1f0497..f795ea42a8fc 100644
--- a/arch/arm/configs/koelsch_defconfig
+++ b/arch/arm/configs/koelsch_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-koelsch"
 CONFIG_SYSVIPC=y
 CONFIG_NO_HZ=y
 CONFIG_IKCONFIG=y
diff --git a/arch/arm/configs/kzm9d_defconfig b/arch/arm/configs/kzm9d_defconfig
index 3ee3858939d9..2d1b9b3740dd 100644
--- a/arch/arm/configs/kzm9d_defconfig
+++ b/arch/arm/configs/kzm9d_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-kzm9d"
 # CONFIG_ARM_PATCH_PHYS_VIRT is not set
 CONFIG_EXPERIMENTAL=y
 CONFIG_SYSVIPC=y
diff --git a/arch/arm/configs/kzm9g_defconfig b/arch/arm/configs/kzm9g_defconfig
index 800faff9df1d..26a3303bcdf7 100644
--- a/arch/arm/configs/kzm9g_defconfig
+++ b/arch/arm/configs/kzm9g_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-kzm9g"
 # CONFIG_ARM_PATCH_PHYS_VIRT is not set
 CONFIG_EXPERIMENTAL=y
 # CONFIG_LOCALVERSION_AUTO is not set
diff --git a/arch/arm/configs/lager_defconfig b/arch/arm/configs/lager_defconfig
index da8fa5e8719b..18b9dd96250a 100644
--- a/arch/arm/configs/lager_defconfig
+++ b/arch/arm/configs/lager_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-lager"
 CONFIG_SYSVIPC=y
 CONFIG_NO_HZ=y
 CONFIG_IKCONFIG=y
diff --git a/arch/arm/configs/mackerel_defconfig b/arch/arm/configs/mackerel_defconfig
index 9d406022d72b..f303b8c5b7d2 100644
--- a/arch/arm/configs/mackerel_defconfig
+++ b/arch/arm/configs/mackerel_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-mackerel"
 CONFIG_EXPERIMENTAL=y
 CONFIG_SYSVIPC=y
 CONFIG_IKCONFIG=y
diff --git a/arch/arm/configs/marzen_defconfig b/arch/arm/configs/marzen_defconfig
index fe3d8f7c0a2c..b89105b12ae6 100644
--- a/arch/arm/configs/marzen_defconfig
+++ b/arch/arm/configs/marzen_defconfig
@@ -1,3 +1,4 @@ 
+CONFIG_LOCALVERSION="-marzen"
 # CONFIG_ARM_PATCH_PHYS_VIRT is not set
 CONFIG_EXPERIMENTAL=y
 CONFIG_KERNEL_LZMA=y

[2/4] ARM: shmobile: Set CONFIG_LOCALVERSION in defconfigs

Commit Message

Comments

Patch