diff mbox

[V2] i2c: ocores: Make of_device_id array const

Message ID 005801cf6fca$af7fd0e0$0e7f72a0$%han@samsung.com
State Superseded
Headers show

Commit Message

Jingoo Han May 14, 2014, 11:17 p.m. UTC
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: Peter Korsgaard <peter@korsgaard.com>
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
---
Changes since v1:
- squashed all patches into a single patch
- added acked-by

 drivers/i2c/busses/i2c-ocores.c |    2 +-
 drivers/i2c/busses/i2c-pxa.c    |    2 +-
 drivers/i2c/busses/i2c-riic.c   |    2 +-
 drivers/i2c/busses/i2c-st.c     |    2 +-
 drivers/i2c/busses/i2c-wmt.c    |    2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

Comments

Peter Korsgaard May 15, 2014, 6:35 a.m. UTC | #1
>>>>> "Jingoo" == Jingoo Han <jg1.han@samsung.com> writes:

You seem to have forgotten to change subject, but I guess Wolfram can
fix that when committing (E.G. s/ocores/busses/).

 > 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: Peter Korsgaard <peter@korsgaard.com>
 > Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
 > ---
 > Changes since v1:
 > - squashed all patches into a single patch
 > - added acked-by
Jingoo Han May 15, 2014, 6:43 a.m. UTC | #2
On Thursday, May 15, 2014 3:36 PM, Peter Korsgaard wrote:
> 
> >>>>> "Jingoo" == Jingoo Han <jg1.han@samsung.com> writes:
> 
> You seem to have forgotten to change subject, but I guess Wolfram can
> fix that when committing (E.G. s/ocores/busses/).

Oh, it's my mistake! I will fix the subject and send
v3 patch, soon. Thank you for your notification. :-)

Best regards,
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>
>  > Acked-by: Peter Korsgaard <peter@korsgaard.com>
>  > Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>  > ---
>  > Changes since v1:
>  > - squashed all patches into a single patch
>  > - added acked-by
> 
> --
> By, Peter Korsgaard

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" 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/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,
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 },
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 */ },
 };
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", },
 	{},
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 */ },
 };