diff mbox

[1/1] ARM: EXYNOS: Fix compilation warning

Message ID 1399282008-9738-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat May 5, 2014, 9:26 a.m. UTC
of_get_flat_dt_prop return type is now const.
Fixes the following compilation warning introduced by commit 9d0c4dfedd96
("of/fdt: update of_get_flat_dt_prop in prep for libfdt")

arch/arm/mach-exynos/exynos.c:259:6: warning:
assignment discards ‘const’ qualifier from pointer target type [enabled by default]

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Cc: Rob Herring <robh@kernel.org>
---
Based on linux-next (20140505)
---
 arch/arm/mach-exynos/exynos.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sachin Kamat May 21, 2014, 11:26 a.m. UTC | #1
On 5 May 2014 14:56, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> of_get_flat_dt_prop return type is now const.
> Fixes the following compilation warning introduced by commit 9d0c4dfedd96
> ("of/fdt: update of_get_flat_dt_prop in prep for libfdt")
>
> arch/arm/mach-exynos/exynos.c:259:6: warning:
> assignment discards ‘const’ qualifier from pointer target type [enabled by default]
>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> Cc: Rob Herring <robh@kernel.org>
> ---
> Based on linux-next (20140505)
> ---
>  arch/arm/mach-exynos/exynos.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-exynos/exynos.c b/arch/arm/mach-exynos/exynos.c
> index 77293d39dfc9..f356c67a1fa9 100644
> --- a/arch/arm/mach-exynos/exynos.c
> +++ b/arch/arm/mach-exynos/exynos.c
> @@ -249,7 +249,7 @@ static int __init exynos_fdt_map_chipid(unsigned long node, const char *uname,
>                                         int depth, void *data)
>  {
>         struct map_desc iodesc;
> -       __be32 *reg;
> +       const __be32 *reg;
>         int len;
>
>         if (!of_flat_dt_is_compatible(node, "samsung,exynos4210-chipid") &&
> --
> 1.7.9.5
>

Ping. This warning is still seen in latest linux-next.
Tushar Behera May 22, 2014, 3:29 a.m. UTC | #2
On 05/21/2014 04:56 PM, Sachin Kamat wrote:
> On 5 May 2014 14:56, Sachin Kamat <sachin.kamat@linaro.org> wrote:
>> of_get_flat_dt_prop return type is now const.
>> Fixes the following compilation warning introduced by commit 9d0c4dfedd96
>> ("of/fdt: update of_get_flat_dt_prop in prep for libfdt")
>>
>> arch/arm/mach-exynos/exynos.c:259:6: warning:
>> assignment discards ‘const’ qualifier from pointer target type [enabled by default]
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>> Cc: Rob Herring <robh@kernel.org>
>> ---

Change looks good.

Reviewed-by: Tushar Behera <tushar.behera@linaro.org>

>> Based on linux-next (20140505)
>> ---
>>  arch/arm/mach-exynos/exynos.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-exynos/exynos.c b/arch/arm/mach-exynos/exynos.c
>> index 77293d39dfc9..f356c67a1fa9 100644
>> --- a/arch/arm/mach-exynos/exynos.c
>> +++ b/arch/arm/mach-exynos/exynos.c
>> @@ -249,7 +249,7 @@ static int __init exynos_fdt_map_chipid(unsigned long node, const char *uname,
>>                                         int depth, void *data)
>>  {
>>         struct map_desc iodesc;
>> -       __be32 *reg;
>> +       const __be32 *reg;
>>         int len;
>>
>>         if (!of_flat_dt_is_compatible(node, "samsung,exynos4210-chipid") &&
>> --
>> 1.7.9.5
>>
> 
> Ping. This warning is still seen in latest linux-next.
>
Arnd Bergmann June 13, 2014, 2:52 p.m. UTC | #3
On Thursday 22 May 2014, Tushar Behera wrote:
> On 05/21/2014 04:56 PM, Sachin Kamat wrote:
> > On 5 May 2014 14:56, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> >> of_get_flat_dt_prop return type is now const.
> >> Fixes the following compilation warning introduced by commit 9d0c4dfedd96
> >> ("of/fdt: update of_get_flat_dt_prop in prep for libfdt")
> >>
> >> arch/arm/mach-exynos/exynos.c:259:6: warning:
> >> assignment discards ‘const’ qualifier from pointer target type [enabled by default]
> >>
> >> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> >> Cc: Rob Herring <robh@kernel.org>
> >> ---
> 
> Change looks good.
> 
> Reviewed-by: Tushar Behera <tushar.behera@linaro.org>

This hasn't shown up in linux-next yet, and hte bug is still there, so I'm applying
it directly to the fixes branch in arm-soc before we get more people sending
the same patch. Thanks!

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" 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-exynos/exynos.c b/arch/arm/mach-exynos/exynos.c
index 77293d39dfc9..f356c67a1fa9 100644
--- a/arch/arm/mach-exynos/exynos.c
+++ b/arch/arm/mach-exynos/exynos.c
@@ -249,7 +249,7 @@  static int __init exynos_fdt_map_chipid(unsigned long node, const char *uname,
 					int depth, void *data)
 {
 	struct map_desc iodesc;
-	__be32 *reg;
+	const __be32 *reg;
 	int len;
 
 	if (!of_flat_dt_is_compatible(node, "samsung,exynos4210-chipid") &&