Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-24 Thread Florian Fainelli
On 21/03/15 16:45, Stefan Agner wrote: > The const declaration for char* is actually duplicated, however > the array of strings is currently not constant. However, typically > the dt_compat array is declared as const char *const. Follow > that convention and also add the __initconst macro for

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-24 Thread Uwe Kleine-König
Hello Stefan, On Sun, Mar 22, 2015 at 12:45:35AM +0100, Stefan Agner wrote: > The const declaration for char* is actually duplicated, however > the array of strings is currently not constant. However, typically > the dt_compat array is declared as const char *const. Follow > that convention and

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-24 Thread Florian Fainelli
On 21/03/15 16:45, Stefan Agner wrote: The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also add the __initconst macro for constant

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-24 Thread Uwe Kleine-König
Hello Stefan, On Sun, Mar 22, 2015 at 12:45:35AM +0100, Stefan Agner wrote: The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-23 Thread Ray Jui
On 3/23/2015 10:18 PM, Scott Branden wrote: > Hi Stefan, > > On 15-03-21 04:45 PM, Stefan Agner wrote: >> The const declaration for char* is actually duplicated, however >> the array of strings is currently not constant. However, typically >> the dt_compat array is declared as const char

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-23 Thread Scott Branden
Hi Stefan, On 15-03-21 04:45 PM, Stefan Agner wrote: The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also add the __initconst macro

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-23 Thread Scott Branden
Hi Stefan, On 15-03-21 04:45 PM, Stefan Agner wrote: The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also add the __initconst macro

Re: [PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-23 Thread Ray Jui
On 3/23/2015 10:18 PM, Scott Branden wrote: Hi Stefan, On 15-03-21 04:45 PM, Stefan Agner wrote: The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow

[PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-21 Thread Stefan Agner
The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also add the __initconst macro for constant initialization data. Signed-off-by:

[PATCH] ARM: cygnus: fix const declaration bcm_cygnus_dt_compat

2015-03-21 Thread Stefan Agner
The const declaration for char* is actually duplicated, however the array of strings is currently not constant. However, typically the dt_compat array is declared as const char *const. Follow that convention and also add the __initconst macro for constant initialization data. Signed-off-by: