diff mbox

arm/omap: use const char properly

Message ID 1370525079-22195-1-git-send-email-bigeasy@linutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Sebastian Andrzej Siewior June 6, 2013, 1:24 p.m. UTC
The itention was probably to make both pointers const but as it is now,
it is just const used twice.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
 arch/arm/mach-omap2/id.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ruslan Bilovol June 6, 2013, 1:42 p.m. UTC | #1
Hi Sebastian,

On Thu, Jun 6, 2013 at 4:24 PM, Sebastian Andrzej Siewior
<bigeasy@linutronix.de> wrote:
> The itention was probably to make both pointers const but as it is now,
> it is just const used twice.

Yes, it's a typo I did accidentally. Good catch!

Regards,
Ruslan

>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
>  arch/arm/mach-omap2/id.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c
> index 1272c41..54e4c16 100644
> --- a/arch/arm/mach-omap2/id.c
> +++ b/arch/arm/mach-omap2/id.c
> @@ -601,7 +601,7 @@ void __init omap2_set_globals_tap(u32 class, void __iomem *tap)
>
>  #ifdef CONFIG_SOC_BUS
>
> -static const char const *omap_types[] = {
> +static const char * const omap_types[] = {
>         [OMAP2_DEVICE_TYPE_TEST]        = "TST",
>         [OMAP2_DEVICE_TYPE_EMU]         = "EMU",
>         [OMAP2_DEVICE_TYPE_SEC]         = "HS",
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tony Lindgren June 12, 2013, 2:17 p.m. UTC | #2
* Ruslan Bilovol <ruslan.bilovol@ti.com> [130606 06:48]:
> Hi Sebastian,
> 
> On Thu, Jun 6, 2013 at 4:24 PM, Sebastian Andrzej Siewior
> <bigeasy@linutronix.de> wrote:
> > The itention was probably to make both pointers const but as it is now,
> > it is just const used twice.
> 
> Yes, it's a typo I did accidentally. Good catch!

Thanks applying into omap-for-v3.11/fixes-non-critical.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c
index 1272c41..54e4c16 100644
--- a/arch/arm/mach-omap2/id.c
+++ b/arch/arm/mach-omap2/id.c
@@ -601,7 +601,7 @@  void __init omap2_set_globals_tap(u32 class, void __iomem *tap)
 
 #ifdef CONFIG_SOC_BUS
 
-static const char const *omap_types[] = {
+static const char * const omap_types[] = {
 	[OMAP2_DEVICE_TYPE_TEST]	= "TST",
 	[OMAP2_DEVICE_TYPE_EMU]		= "EMU",
 	[OMAP2_DEVICE_TYPE_SEC]		= "HS",