linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/7] gpio: grgpio: Make of_device_id array const
@ 2014-05-07  9:03 Jingoo Han
  2014-05-07  9:05 ` [PATCH 2/7] gpio: mcp23s08: " Jingoo Han
                   ` (7 more replies)
  0 siblings, 8 replies; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:03 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Andreas Larsson'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-grgpio.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-grgpio.c b/drivers/gpio/gpio-grgpio.c
index 84d2478..3c3f515 100644
--- a/drivers/gpio/gpio-grgpio.c
+++ b/drivers/gpio/gpio-grgpio.c
@@ -481,7 +481,7 @@ out:
 	return ret;
 }
 
-static struct of_device_id grgpio_match[] = {
+static const struct of_device_id grgpio_match[] = {
 	{.name = "GAISLER_GPIO"},
 	{.name = "01_01a"},
 	{},
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 2/7] gpio: mcp23s08: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
@ 2014-05-07  9:05 ` Jingoo Han
  2014-05-09  8:59   ` Linus Walleij
  2014-05-07  9:06 ` [PATCH 3/7] gpio: mvebu: " Jingoo Han
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:05 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-mcp23s08.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpio-mcp23s08.c b/drivers/gpio/gpio-mcp23s08.c
index 3d53fd6..4d677f1 100644
--- a/drivers/gpio/gpio-mcp23s08.c
+++ b/drivers/gpio/gpio-mcp23s08.c
@@ -714,7 +714,7 @@ fail:
 
 #ifdef CONFIG_OF
 #ifdef CONFIG_SPI_MASTER
-static struct of_device_id mcp23s08_spi_of_match[] = {
+static const struct of_device_id mcp23s08_spi_of_match[] = {
 	{
 		.compatible = "microchip,mcp23s08",
 		.data = (void *) MCP_TYPE_S08,
@@ -738,7 +738,7 @@ MODULE_DEVICE_TABLE(of, mcp23s08_spi_of_match);
 #endif
 
 #if IS_ENABLED(CONFIG_I2C)
-static struct of_device_id mcp23s08_i2c_of_match[] = {
+static const struct of_device_id mcp23s08_i2c_of_match[] = {
 	{
 		.compatible = "microchip,mcp23008",
 		.data = (void *) MCP_TYPE_008,
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 3/7] gpio: mvebu: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
  2014-05-07  9:05 ` [PATCH 2/7] gpio: mcp23s08: " Jingoo Han
@ 2014-05-07  9:06 ` Jingoo Han
  2014-05-09  9:01   ` Linus Walleij
  2014-05-07  9:06 ` [PATCH 4/7] gpio: palmas: " Jingoo Han
                   ` (5 subsequent siblings)
  7 siblings, 1 reply; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:06 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Jason Cooper', 'Thomas Petazzoni'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-mvebu.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-mvebu.c b/drivers/gpio/gpio-mvebu.c
index d425094..b5d8645 100644
--- a/drivers/gpio/gpio-mvebu.c
+++ b/drivers/gpio/gpio-mvebu.c
@@ -535,7 +535,7 @@ static void mvebu_gpio_dbg_show(struct seq_file *s, struct gpio_chip *chip)
 #define mvebu_gpio_dbg_show NULL
 #endif
 
-static struct of_device_id mvebu_gpio_of_match[] = {
+static const struct of_device_id mvebu_gpio_of_match[] = {
 	{
 		.compatible = "marvell,orion-gpio",
 		.data       = (void *) MVEBU_GPIO_SOC_VARIANT_ORION,
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 4/7] gpio: palmas: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
  2014-05-07  9:05 ` [PATCH 2/7] gpio: mcp23s08: " Jingoo Han
  2014-05-07  9:06 ` [PATCH 3/7] gpio: mvebu: " Jingoo Han
@ 2014-05-07  9:06 ` Jingoo Han
  2014-05-07  9:07 ` [PATCH 5/7] gpio: tegra: " Jingoo Han
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:06 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Laxman Dewangan'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-palmas.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-palmas.c b/drivers/gpio/gpio-palmas.c
index da9d332..0860b74 100644
--- a/drivers/gpio/gpio-palmas.c
+++ b/drivers/gpio/gpio-palmas.c
@@ -148,7 +148,7 @@ static const struct palmas_device_data tps80036_dev_data = {
 	.ngpio = 16,
 };
 
-static struct of_device_id of_palmas_gpio_match[] = {
+static const struct of_device_id of_palmas_gpio_match[] = {
 	{ .compatible = "ti,palmas-gpio", .data = &palmas_dev_data,},
 	{ .compatible = "ti,tps65913-gpio", .data = &palmas_dev_data,},
 	{ .compatible = "ti,tps65914-gpio", .data = &palmas_dev_data,},
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 5/7] gpio: tegra: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
                   ` (2 preceding siblings ...)
  2014-05-07  9:06 ` [PATCH 4/7] gpio: palmas: " Jingoo Han
@ 2014-05-07  9:07 ` Jingoo Han
  2014-05-09  9:02   ` Linus Walleij
  2014-05-07  9:08 ` [PATCH 6/7] gpio: xilinx: " Jingoo Han
                   ` (3 subsequent siblings)
  7 siblings, 1 reply; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:07 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Stephen Warren'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-tegra.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-tegra.c b/drivers/gpio/gpio-tegra.c
index 2b49f87..c2ea6f4 100644
--- a/drivers/gpio/gpio-tegra.c
+++ b/drivers/gpio/gpio-tegra.c
@@ -408,7 +408,7 @@ static struct tegra_gpio_soc_config tegra30_gpio_config = {
 	.upper_offset = 0x80,
 };
 
-static struct of_device_id tegra_gpio_of_match[] = {
+static const struct of_device_id tegra_gpio_of_match[] = {
 	{ .compatible = "nvidia,tegra30-gpio", .data = &tegra30_gpio_config },
 	{ .compatible = "nvidia,tegra20-gpio", .data = &tegra20_gpio_config },
 	{ },
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 6/7] gpio: xilinx: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
                   ` (3 preceding siblings ...)
  2014-05-07  9:07 ` [PATCH 5/7] gpio: tegra: " Jingoo Han
@ 2014-05-07  9:08 ` Jingoo Han
  2014-05-07 11:25   ` Michal Simek
  2014-05-09  9:03   ` Linus Walleij
  2014-05-07  9:09 ` [PATCH 7/7] gpio: zevio: " Jingoo Han
                   ` (2 subsequent siblings)
  7 siblings, 2 replies; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:08 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Michal Simek'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-xilinx.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-xilinx.c b/drivers/gpio/gpio-xilinx.c
index 792a05a..1248186 100644
--- a/drivers/gpio/gpio-xilinx.c
+++ b/drivers/gpio/gpio-xilinx.c
@@ -289,7 +289,7 @@ static int xgpio_of_probe(struct device_node *np)
 	return 0;
 }
 
-static struct of_device_id xgpio_of_match[] = {
+static const struct of_device_id xgpio_of_match[] = {
 	{ .compatible = "xlnx,xps-gpio-1.00.a", },
 	{ /* end of list */ },
 };
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [PATCH 7/7] gpio: zevio: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
                   ` (4 preceding siblings ...)
  2014-05-07  9:08 ` [PATCH 6/7] gpio: xilinx: " Jingoo Han
@ 2014-05-07  9:09 ` Jingoo Han
  2014-05-09  9:04   ` Linus Walleij
  2014-05-07 12:01 ` [PATCH 1/7] gpio: grgpio: " Andreas Larsson
  2014-05-09  8:58 ` Linus Walleij
  7 siblings, 1 reply; 15+ messages in thread
From: Jingoo Han @ 2014-05-07  9:09 UTC (permalink / raw)
  To: 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio, 'Jingoo Han',
	'Fabian Vogt'

Make of_device_id array const, because all OF functions
handle it as const.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
 drivers/gpio/gpio-zevio.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-zevio.c b/drivers/gpio/gpio-zevio.c
index f20941c..23a7669 100644
--- a/drivers/gpio/gpio-zevio.c
+++ b/drivers/gpio/gpio-zevio.c
@@ -208,7 +208,7 @@ static int zevio_gpio_probe(struct platform_device *pdev)
 	return 0;
 }
 
-static struct of_device_id zevio_gpio_of_match[] = {
+static const struct of_device_id zevio_gpio_of_match[] = {
 	{ .compatible = "lsi,zevio-gpio", },
 	{ },
 };
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: [PATCH 6/7] gpio: xilinx: Make of_device_id array const
  2014-05-07  9:08 ` [PATCH 6/7] gpio: xilinx: " Jingoo Han
@ 2014-05-07 11:25   ` Michal Simek
  2014-05-09  9:03   ` Linus Walleij
  1 sibling, 0 replies; 15+ messages in thread
From: Michal Simek @ 2014-05-07 11:25 UTC (permalink / raw)
  To: Jingoo Han, 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio

[-- Attachment #1: Type: text/plain, Size: 1134 bytes --]

On 05/07/2014 11:08 AM, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
> 
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>
> ---
>  drivers/gpio/gpio-xilinx.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpio/gpio-xilinx.c b/drivers/gpio/gpio-xilinx.c
> index 792a05a..1248186 100644
> --- a/drivers/gpio/gpio-xilinx.c
> +++ b/drivers/gpio/gpio-xilinx.c
> @@ -289,7 +289,7 @@ static int xgpio_of_probe(struct device_node *np)
>  	return 0;
>  }
>  
> -static struct of_device_id xgpio_of_match[] = {
> +static const struct of_device_id xgpio_of_match[] = {
>  	{ .compatible = "xlnx,xps-gpio-1.00.a", },
>  	{ /* end of list */ },
>  };
> 

Reviewed-by: Michal Simek <monstr@monstr.eu>

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 263 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 1/7] gpio: grgpio: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
                   ` (5 preceding siblings ...)
  2014-05-07  9:09 ` [PATCH 7/7] gpio: zevio: " Jingoo Han
@ 2014-05-07 12:01 ` Andreas Larsson
  2014-05-09  8:58 ` Linus Walleij
  7 siblings, 0 replies; 15+ messages in thread
From: Andreas Larsson @ 2014-05-07 12:01 UTC (permalink / raw)
  To: Jingoo Han, 'Linus Walleij'
  Cc: 'Alexandre Courbot', linux-gpio

On 2014-05-07 11:03, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Acked-by: Andreas Larsson <andreas@gaisler.com>

> ---
>   drivers/gpio/gpio-grgpio.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpio/gpio-grgpio.c b/drivers/gpio/gpio-grgpio.c
> index 84d2478..3c3f515 100644
> --- a/drivers/gpio/gpio-grgpio.c
> +++ b/drivers/gpio/gpio-grgpio.c
> @@ -481,7 +481,7 @@ out:
>   	return ret;
>   }
>
> -static struct of_device_id grgpio_match[] = {
> +static const struct of_device_id grgpio_match[] = {
>   	{.name = "GAISLER_GPIO"},
>   	{.name = "01_01a"},
>   	{},
>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 1/7] gpio: grgpio: Make of_device_id array const
  2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
                   ` (6 preceding siblings ...)
  2014-05-07 12:01 ` [PATCH 1/7] gpio: grgpio: " Andreas Larsson
@ 2014-05-09  8:58 ` Linus Walleij
  7 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  8:58 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Alexandre Courbot, linux-gpio@vger.kernel.org, Andreas Larsson

On Wed, May 7, 2014 at 11:03 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied with Andreas' ACK.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 2/7] gpio: mcp23s08: Make of_device_id array const
  2014-05-07  9:05 ` [PATCH 2/7] gpio: mcp23s08: " Jingoo Han
@ 2014-05-09  8:59   ` Linus Walleij
  0 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  8:59 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Alexandre Courbot, linux-gpio@vger.kernel.org

On Wed, May 7, 2014 at 11:05 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 3/7] gpio: mvebu: Make of_device_id array const
  2014-05-07  9:06 ` [PATCH 3/7] gpio: mvebu: " Jingoo Han
@ 2014-05-09  9:01   ` Linus Walleij
  0 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  9:01 UTC (permalink / raw)
  To: Jingoo Han
  Cc: Alexandre Courbot, linux-gpio@vger.kernel.org, Jason Cooper,
	Thomas Petazzoni

On Wed, May 7, 2014 at 11:06 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 5/7] gpio: tegra: Make of_device_id array const
  2014-05-07  9:07 ` [PATCH 5/7] gpio: tegra: " Jingoo Han
@ 2014-05-09  9:02   ` Linus Walleij
  0 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  9:02 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Alexandre Courbot, linux-gpio@vger.kernel.org, Stephen Warren

On Wed, May 7, 2014 at 11:07 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 6/7] gpio: xilinx: Make of_device_id array const
  2014-05-07  9:08 ` [PATCH 6/7] gpio: xilinx: " Jingoo Han
  2014-05-07 11:25   ` Michal Simek
@ 2014-05-09  9:03   ` Linus Walleij
  1 sibling, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  9:03 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Alexandre Courbot, linux-gpio@vger.kernel.org, Michal Simek

On Wed, May 7, 2014 at 11:08 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied with Michal's Review tag.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH 7/7] gpio: zevio: Make of_device_id array const
  2014-05-07  9:09 ` [PATCH 7/7] gpio: zevio: " Jingoo Han
@ 2014-05-09  9:04   ` Linus Walleij
  0 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2014-05-09  9:04 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Alexandre Courbot, linux-gpio@vger.kernel.org, Fabian Vogt

On Wed, May 7, 2014 at 11:09 AM, Jingoo Han <jg1.han@samsung.com> wrote:

> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han@samsung.com>

Patch applied.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2014-05-09  9:04 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-07  9:03 [PATCH 1/7] gpio: grgpio: Make of_device_id array const Jingoo Han
2014-05-07  9:05 ` [PATCH 2/7] gpio: mcp23s08: " Jingoo Han
2014-05-09  8:59   ` Linus Walleij
2014-05-07  9:06 ` [PATCH 3/7] gpio: mvebu: " Jingoo Han
2014-05-09  9:01   ` Linus Walleij
2014-05-07  9:06 ` [PATCH 4/7] gpio: palmas: " Jingoo Han
2014-05-07  9:07 ` [PATCH 5/7] gpio: tegra: " Jingoo Han
2014-05-09  9:02   ` Linus Walleij
2014-05-07  9:08 ` [PATCH 6/7] gpio: xilinx: " Jingoo Han
2014-05-07 11:25   ` Michal Simek
2014-05-09  9:03   ` Linus Walleij
2014-05-07  9:09 ` [PATCH 7/7] gpio: zevio: " Jingoo Han
2014-05-09  9:04   ` Linus Walleij
2014-05-07 12:01 ` [PATCH 1/7] gpio: grgpio: " Andreas Larsson
2014-05-09  8:58 ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).