[PATCH 1/5] i2c: ocores: Make of_device

linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH 1/5] i2c: ocores: Make of_device_id array const
@ 2014-05-07  8:54 Jingoo Han
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
  0 siblings, 1 reply; 9+ messages in thread
From: Jingoo Han @ 2014-05-07  8:54 UTC (permalink / raw)
  To: 'Wolfram Sang'
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Jingoo Han',
	'Peter Korsgaard', 'Andreas Larsson'

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

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/i2c/busses/i2c-ocores.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c
index 1f6369f..0e10cc6 100644
--- a/drivers/i2c/busses/i2c-ocores.c
+++ b/drivers/i2c/busses/i2c-ocores.c
@@ -250,7 +250,7 @@ static struct i2c_adapter ocores_adapter = {
 	.algo		= &ocores_algorithm,
 };
 
-static struct of_device_id ocores_i2c_match[] = {
+static const struct of_device_id ocores_i2c_match[] = {
 	{
 		.compatible = "opencores,i2c-ocores",
 		.data = (void *)TYPE_OCORES,
-- 
1.7.10.4

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

[parent not found: <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>]

* [PATCH 2/5] i2c: pxa: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2014-05-07  8:57   ` Jingoo Han
       [not found]     ` <022201cf69d2$5fcc0ad0$1f642070$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
  2014-05-07  8:58   ` [PATCH 3/5] i2c: riic: " Jingoo Han
                     ` (4 subsequent siblings)
  5 siblings, 1 reply; 9+ messages in thread
From: Jingoo Han @ 2014-05-07  8:57 UTC (permalink / raw)
  To: 'Wolfram Sang'
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Jingoo Han',
	'Leilei Shang', 'Haojian Zhuang'

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

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/i2c/busses/i2c-pxa.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
index bbe6dfb..be671f7 100644
--- a/drivers/i2c/busses/i2c-pxa.c
+++ b/drivers/i2c/busses/i2c-pxa.c
@@ -1084,7 +1084,7 @@ static const struct i2c_algorithm i2c_pxa_pio_algorithm = {
 	.functionality	= i2c_pxa_functionality,
 };
 
-static struct of_device_id i2c_pxa_dt_ids[] = {
+static const struct of_device_id i2c_pxa_dt_ids[] = {
 	{ .compatible = "mrvl,pxa-i2c", .data = (void *)REGS_PXA2XX },
 	{ .compatible = "mrvl,pwri2c", .data = (void *)REGS_PXA3XX },
 	{ .compatible = "mrvl,mmp-twsi", .data = (void *)REGS_PXA2XX },
-- 
1.7.10.4

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

[parent not found: <022201cf69d2$5fcc0ad0$1f642070$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>]

* Re: [PATCH 2/5] i2c: pxa: Make of_device_id array const
       [not found]     ` <022201cf69d2$5fcc0ad0$1f642070$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2014-07-09 23:52       ` Haojian Zhuang
  0 siblings, 0 replies; 9+ messages in thread
From: Haojian Zhuang @ 2014-07-09 23:52 UTC (permalink / raw)
  To: Jingoo Han; +Cc: Wolfram Sang, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Leilei Shang

On Wed, May 7, 2014 at 4:57 PM, Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>
> Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> ---
>  drivers/i2c/busses/i2c-pxa.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
> index bbe6dfb..be671f7 100644
> --- a/drivers/i2c/busses/i2c-pxa.c
> +++ b/drivers/i2c/busses/i2c-pxa.c
> @@ -1084,7 +1084,7 @@ static const struct i2c_algorithm i2c_pxa_pio_algorithm = {
>         .functionality  = i2c_pxa_functionality,
>  };
>
> -static struct of_device_id i2c_pxa_dt_ids[] = {
> +static const struct of_device_id i2c_pxa_dt_ids[] = {
>         { .compatible = "mrvl,pxa-i2c", .data = (void *)REGS_PXA2XX },
>         { .compatible = "mrvl,pwri2c", .data = (void *)REGS_PXA3XX },
>         { .compatible = "mrvl,mmp-twsi", .data = (void *)REGS_PXA2XX },
> --
> 1.7.10.4
>
>

Acked-by: Haojian Zhuang <haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

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

* [PATCH 3/5] i2c: riic: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
  2014-05-07  8:57   ` [PATCH 2/5] i2c: pxa: " Jingoo Han
@ 2014-05-07  8:58   ` Jingoo Han
  2014-05-07  9:00   ` [PATCH 5/5] i2c: wmt: " Jingoo Han
                     ` (3 subsequent siblings)
  5 siblings, 0 replies; 9+ messages in thread
From: Jingoo Han @ 2014-05-07  8:58 UTC (permalink / raw)
  To: 'Wolfram Sang'
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Jingoo Han'

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

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/i2c/busses/i2c-riic.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
index 9e1f8ba..af3b3d0 100644
--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c
@@ -404,7 +404,7 @@ static int riic_i2c_remove(struct platform_device *pdev)
 	return 0;
 }
 
-static struct of_device_id riic_i2c_dt_ids[] = {
+static const struct of_device_id riic_i2c_dt_ids[] = {
 	{ .compatible = "renesas,riic-rz" },
 	{ /* Sentinel */ },
 };
-- 
1.7.10.4

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

* [PATCH 5/5] i2c: wmt: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
  2014-05-07  8:57   ` [PATCH 2/5] i2c: pxa: " Jingoo Han
  2014-05-07  8:58   ` [PATCH 3/5] i2c: riic: " Jingoo Han
@ 2014-05-07  9:00   ` Jingoo Han
  2014-05-07  9:01   ` [PATCH 4/5] i2c: st: " Jingoo Han
                     ` (2 subsequent siblings)
  5 siblings, 0 replies; 9+ messages in thread
From: Jingoo Han @ 2014-05-07  9:00 UTC (permalink / raw)
  To: 'Wolfram Sang'
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Jingoo Han',
	'Tony Prisk'

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

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/i2c/busses/i2c-wmt.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-wmt.c b/drivers/i2c/busses/i2c-wmt.c
index 2c8a3e4..924b501 100644
--- a/drivers/i2c/busses/i2c-wmt.c
+++ b/drivers/i2c/busses/i2c-wmt.c
@@ -454,7 +454,7 @@ static int wmt_i2c_remove(struct platform_device *pdev)
 	return 0;
 }
 
-static struct of_device_id wmt_i2c_dt_ids[] = {
+static const struct of_device_id wmt_i2c_dt_ids[] = {
 	{ .compatible = "wm,wm8505-i2c" },
 	{ /* Sentinel */ },
 };
-- 
1.7.10.4

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

* [PATCH 4/5] i2c: st: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
                     ` (2 preceding siblings ...)
  2014-05-07  9:00   ` [PATCH 5/5] i2c: wmt: " Jingoo Han
@ 2014-05-07  9:01   ` Jingoo Han
       [not found]     ` <022601cf69d2$ef8ba9f0$cea2fdd0$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
  2014-05-07  9:30   ` [PATCH 1/5] i2c: ocores: " Peter Korsgaard
  2014-05-14 16:38   ` Wolfram Sang
  5 siblings, 1 reply; 9+ messages in thread
From: Jingoo Han @ 2014-05-07  9:01 UTC (permalink / raw)
  To: 'Wolfram Sang'
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Jingoo Han',
	'Maxime COQUELIN'

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

Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
 drivers/i2c/busses/i2c-st.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c
index 8720161..95b94767 100644
--- a/drivers/i2c/busses/i2c-st.c
+++ b/drivers/i2c/busses/i2c-st.c
@@ -847,7 +847,7 @@ static int st_i2c_remove(struct platform_device *pdev)
 	return 0;
 }
 
-static struct of_device_id st_i2c_match[] = {
+static const struct of_device_id st_i2c_match[] = {
 	{ .compatible = "st,comms-ssc-i2c", },
 	{ .compatible = "st,comms-ssc4-i2c", },
 	{},
-- 
1.7.10.4

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

[parent not found: <022601cf69d2$ef8ba9f0$cea2fdd0$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>]

* Re: [PATCH 4/5] i2c: st: Make of_device_id array const
       [not found]     ` <022601cf69d2$ef8ba9f0$cea2fdd0$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2014-05-07  9:05       ` Maxime Coquelin
  0 siblings, 0 replies; 9+ messages in thread
From: Maxime Coquelin @ 2014-05-07  9:05 UTC (permalink / raw)
  To: Jingoo Han, 'Wolfram Sang'; +Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA

Hi Jingoo,

On 05/07/2014 11:01 AM, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
>

Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>

> Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> ---
>   drivers/i2c/busses/i2c-st.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c
> index 8720161..95b94767 100644
> --- a/drivers/i2c/busses/i2c-st.c
> +++ b/drivers/i2c/busses/i2c-st.c
> @@ -847,7 +847,7 @@ static int st_i2c_remove(struct platform_device *pdev)
>   	return 0;
>   }
>
> -static struct of_device_id st_i2c_match[] = {
> +static const struct of_device_id st_i2c_match[] = {
>   	{ .compatible = "st,comms-ssc-i2c", },
>   	{ .compatible = "st,comms-ssc4-i2c", },
>   	{},
>

Thanks!
Maxime

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

* Re: [PATCH 1/5] i2c: ocores: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
                     ` (3 preceding siblings ...)
  2014-05-07  9:01   ` [PATCH 4/5] i2c: st: " Jingoo Han
@ 2014-05-07  9:30   ` Peter Korsgaard
  2014-05-14 16:38   ` Wolfram Sang
  5 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2014-05-07  9:30 UTC (permalink / raw)
  To: Jingoo Han
  Cc: 'Wolfram Sang', linux-i2c-u79uwXL29TY76Z2rM5mHXA,
	'Peter Korsgaard', 'Andreas Larsson'

>>>>> "Jingoo" == Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> writes:

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

 > Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>

Acked-by: Peter Korsgaard <peter-+2lRwdCCLRT2eFz/2MeuCQ@public.gmane.org>


 > ---
 >  drivers/i2c/busses/i2c-ocores.c |    2 +-
 >  1 file changed, 1 insertion(+), 1 deletion(-)

 > diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c
 > index 1f6369f..0e10cc6 100644
 > --- a/drivers/i2c/busses/i2c-ocores.c
 > +++ b/drivers/i2c/busses/i2c-ocores.c
 > @@ -250,7 +250,7 @@ static struct i2c_adapter ocores_adapter = {
 >  	.algo		= &ocores_algorithm,
 >  };
 
 > -static struct of_device_id ocores_i2c_match[] = {
 > +static const struct of_device_id ocores_i2c_match[] = {
 >  	{
 >  		.compatible = "opencores,i2c-ocores",
 >  		.data = (void *)TYPE_OCORES,
 > -- 
 > 1.7.10.4


 > --
 > To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
 > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
 > More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Bye, Peter Korsgaard

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

* Re: [PATCH 1/5] i2c: ocores: Make of_device_id array const
       [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
                     ` (4 preceding siblings ...)
  2014-05-07  9:30   ` [PATCH 1/5] i2c: ocores: " Peter Korsgaard
@ 2014-05-14 16:38   ` Wolfram Sang
  5 siblings, 0 replies; 9+ messages in thread
From: Wolfram Sang @ 2014-05-14 16:38 UTC (permalink / raw)
  To: Jingoo Han
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, 'Peter Korsgaard',
	'Andreas Larsson'

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

On Wed, May 07, 2014 at 05:54:23PM +0900, Jingoo Han wrote:
> Make of_device_id array const, because all OF functions
> handle it as const.
> 
> Signed-off-by: Jingoo Han <jg1.han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>

Ditto, one patch please.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

end of thread, other threads:[~2014-07-09 23:52 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-07  8:54 [PATCH 1/5] i2c: ocores: Make of_device_id array const Jingoo Han
     [not found] ` <022101cf69d1$f1311200$d3933600$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-05-07  8:57   ` [PATCH 2/5] i2c: pxa: " Jingoo Han
     [not found]     ` <022201cf69d2$5fcc0ad0$1f642070$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-07-09 23:52       ` Haojian Zhuang
2014-05-07  8:58   ` [PATCH 3/5] i2c: riic: " Jingoo Han
2014-05-07  9:00   ` [PATCH 5/5] i2c: wmt: " Jingoo Han
2014-05-07  9:01   ` [PATCH 4/5] i2c: st: " Jingoo Han
     [not found]     ` <022601cf69d2$ef8ba9f0$cea2fdd0$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-05-07  9:05       ` Maxime Coquelin
2014-05-07  9:30   ` [PATCH 1/5] i2c: ocores: " Peter Korsgaard
2014-05-14 16:38   ` Wolfram Sang

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).