From: Michael Opdenacker <michael.opdenacker@free-electrons.com>
To: ben-linux@fluff.org, kgene.kim@samsung.com, linux@arm.linux.org.uk
Cc: linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
Michael Opdenacker <michael.opdenacker@free-electrons.com>
Subject: Re: [PATCH] ARM: S3C24XX: remove unused Kconfig parameter
Date: Mon, 04 Nov 2013 10:15:40 +0100 [thread overview]
Message-ID: <527765BC.5010904@free-electrons.com> (raw)
In-Reply-To: <1383463529-3113-1-git-send-email-michael.opdenacker@free-electrons.com>
On 11/03/2013 08:25 AM, Michael Opdenacker wrote:
> This removes the MACH_S3C2413 Kconfig parameter,
> which was no longer used anywhere in the source code
> and Makefiles.
>
> Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com>
> ---
> arch/arm/mach-s3c24xx/Kconfig | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/arch/arm/mach-s3c24xx/Kconfig b/arch/arm/mach-s3c24xx/Kconfig
> index dba2173e70f3..9d90dd06cf2f 100644
> --- a/arch/arm/mach-s3c24xx/Kconfig
> +++ b/arch/arm/mach-s3c24xx/Kconfig
> @@ -415,13 +415,6 @@ config MACH_JIVE_SHOW_BOOTLOADER
> bool "Allow access to bootloader partitions in MTD"
> depends on MACH_JIVE
>
> -config MACH_S3C2413
> - bool
> - help
> - Internal node for S3C2413 version of SMDK2413, so that
> - machine_is_s3c2413() will work when MACH_SMDK2413 is
> - selected
> -
> config MACH_SMDK2412
> bool "SMDK2412"
> select MACH_SMDK2413
Actually, in another thread, Sascha Hauer made me realize that this
symbol is actually used to make machine_is_s3c2413() become true when
the kernel is started with the corresponding machine id.
However, this patch doesn't hurt as machine_is_s3c2413() isn't used in
the code.
You can either keep it or wait for s3c2413 support to be removed or to
be converted to the device tree.
Cheers,
Michael.
--
Michael Opdenacker, CEO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
+33 484 258 098
WARNING: multiple messages have this Message-ID (diff)
From: michael.opdenacker@free-electrons.com (Michael Opdenacker)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: S3C24XX: remove unused Kconfig parameter
Date: Mon, 04 Nov 2013 10:15:40 +0100 [thread overview]
Message-ID: <527765BC.5010904@free-electrons.com> (raw)
In-Reply-To: <1383463529-3113-1-git-send-email-michael.opdenacker@free-electrons.com>
On 11/03/2013 08:25 AM, Michael Opdenacker wrote:
> This removes the MACH_S3C2413 Kconfig parameter,
> which was no longer used anywhere in the source code
> and Makefiles.
>
> Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com>
> ---
> arch/arm/mach-s3c24xx/Kconfig | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/arch/arm/mach-s3c24xx/Kconfig b/arch/arm/mach-s3c24xx/Kconfig
> index dba2173e70f3..9d90dd06cf2f 100644
> --- a/arch/arm/mach-s3c24xx/Kconfig
> +++ b/arch/arm/mach-s3c24xx/Kconfig
> @@ -415,13 +415,6 @@ config MACH_JIVE_SHOW_BOOTLOADER
> bool "Allow access to bootloader partitions in MTD"
> depends on MACH_JIVE
>
> -config MACH_S3C2413
> - bool
> - help
> - Internal node for S3C2413 version of SMDK2413, so that
> - machine_is_s3c2413() will work when MACH_SMDK2413 is
> - selected
> -
> config MACH_SMDK2412
> bool "SMDK2412"
> select MACH_SMDK2413
Actually, in another thread, Sascha Hauer made me realize that this
symbol is actually used to make machine_is_s3c2413() become true when
the kernel is started with the corresponding machine id.
However, this patch doesn't hurt as machine_is_s3c2413() isn't used in
the code.
You can either keep it or wait for s3c2413 support to be removed or to
be converted to the device tree.
Cheers,
Michael.
--
Michael Opdenacker, CEO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
+33 484 258 098
next prev parent reply other threads:[~2013-11-04 9:15 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-03 7:25 [PATCH] ARM: S3C24XX: remove unused Kconfig parameter Michael Opdenacker
2013-11-03 7:25 ` Michael Opdenacker
2013-11-04 9:15 ` Michael Opdenacker [this message]
2013-11-04 9:15 ` Michael Opdenacker
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=527765BC.5010904@free-electrons.com \
--to=michael.opdenacker@free-electrons.com \
--cc=ben-linux@fluff.org \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
/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.