linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 08/25] gpio/mpc8xxx: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-29 16:13   ` Linus Walleij
  2012-07-23  9:13 ` [PATCH 10/25] i2c/mpc: " Uwe Kleine-König
                   ` (8 subsequent siblings)
  9 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Linus Walleij
  Cc: linuxppc-dev, linux-arm-kernel, kernel

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	drivers/gpio/gpio-mpc8xxx.c: In function 'mpc8xxx_add_controller':
	drivers/gpio/gpio-mpc8xxx.c:360:30: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/gpio/gpio-mpc8xxx.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpio/gpio-mpc8xxx.c b/drivers/gpio/gpio-mpc8xxx.c
index 5a1817e..9ae29cc 100644
--- a/drivers/gpio/gpio-mpc8xxx.c
+++ b/drivers/gpio/gpio-mpc8xxx.c
@@ -38,7 +38,7 @@ struct mpc8xxx_gpio_chip {
 	 */
 	u32 data;
 	struct irq_domain *irq;
-	void *of_dev_id_data;
+	const void *of_dev_id_data;
 };
 
 static inline u32 mpc8xxx_gpio2mask(unsigned int gpio)
-- 
1.7.10.4

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

* [PATCH 10/25] i2c/mpc: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
  2012-07-23  9:13 ` [PATCH 08/25] gpio/mpc8xxx: add a const qualifier Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 13/25] macintosh/mediabay: " Uwe Kleine-König
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Ben Dooks, Wolfram Sang
  Cc: linuxppc-dev, linux-arm-kernel, kernel, linux-i2c

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	drivers/i2c/busses/i2c-mpc.c: In function 'fsl_i2c_probe':
	drivers/i2c/busses/i2c-mpc.c:650:31: warning: initialization discards 'const' qualifier from pointer target type [enabled by default]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/i2c/busses/i2c-mpc.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index b76731e..775062c 100644
--- a/drivers/i2c/busses/i2c-mpc.c
+++ b/drivers/i2c/busses/i2c-mpc.c
@@ -647,7 +647,7 @@ static int __devinit fsl_i2c_probe(struct platform_device *op)
 	}
 
 	if (match->data) {
-		struct mpc_i2c_data *data = match->data;
+		const struct mpc_i2c_data *data = match->data;
 		data->setup(op->dev.of_node, i2c, clock, data->prescaler);
 	} else {
 		/* Backwards compatibility */
-- 
1.7.10.4

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

* [PATCH 13/25] macintosh/mediabay: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
  2012-07-23  9:13 ` [PATCH 08/25] gpio/mpc8xxx: add a const qualifier Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 10/25] i2c/mpc: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-09-05  2:40   ` Benjamin Herrenschmidt
  2012-07-23  9:13 ` [PATCH 14/25] powerpc/83xx: " Uwe Kleine-König
                   ` (6 subsequent siblings)
  9 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Benjamin Herrenschmidt
  Cc: linuxppc-dev, linux-arm-kernel, kernel

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	drivers/macintosh/mediabay.c: In function 'media_bay_attach':
	drivers/macintosh/mediabay.c:589:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/macintosh/mediabay.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/macintosh/mediabay.c b/drivers/macintosh/mediabay.c
index 831d751..54bf584 100644
--- a/drivers/macintosh/mediabay.c
+++ b/drivers/macintosh/mediabay.c
@@ -63,7 +63,7 @@ struct media_bay_info {
 	int				value_count;
 	int				timer;
 	struct macio_dev		*mdev;
-	struct mb_ops*			ops;
+	const struct mb_ops*		ops;
 	int				index;
 	int				cached_gpio;
 	int				sleeping;
-- 
1.7.10.4

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

* [PATCH 14/25] powerpc/83xx: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (2 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 13/25] macintosh/mediabay: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 15/25] powerpc/fsl_msi: " Uwe Kleine-König
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Kumar Gala,
	Benjamin Herrenschmidt, Paul Mackerras
  Cc: linuxppc-dev, linux-arm-kernel, kernel

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	arch/powerpc/platforms/83xx/suspend.c: In function 'pmc_probe':
	arch/powerpc/platforms/83xx/suspend.c:336:7: error: assignment discards 'const' qualifier from pointer target type [-Werror]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/powerpc/platforms/83xx/suspend.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/83xx/suspend.c b/arch/powerpc/platforms/83xx/suspend.c
index 1a04671..1d769a2 100644
--- a/arch/powerpc/platforms/83xx/suspend.c
+++ b/arch/powerpc/platforms/83xx/suspend.c
@@ -326,7 +326,7 @@ static int pmc_probe(struct platform_device *ofdev)
 	const struct of_device_id *match;
 	struct device_node *np = ofdev->dev.of_node;
 	struct resource res;
-	struct pmc_type *type;
+	const struct pmc_type *type;
 	int ret = 0;
 
 	match = of_match_device(pmc_match, &ofdev->dev);
-- 
1.7.10.4

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

* [PATCH 15/25] powerpc/fsl_msi: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (3 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 14/25] powerpc/83xx: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-10-15 16:14   ` Tabi Timur-B04825
  2012-07-23  9:13 ` [PATCH 16/25] powerpc/celleb_pci: " Uwe Kleine-König
                   ` (4 subsequent siblings)
  9 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Benjamin Herrenschmidt,
	Paul Mackerras, Timur Tabi, Kumar Gala
  Cc: linuxppc-dev, linux-arm-kernel, kernel

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	arch/powerpc/sysdev/fsl_msi.c: In function 'fsl_of_msi_probe':
	arch/powerpc/sysdev/fsl_msi.c:379:11: error: assignment discards 'const' qualifier from pointer target type [-Werror]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/powerpc/sysdev/fsl_msi.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c
index 6e097de..a40acd6 100644
--- a/arch/powerpc/sysdev/fsl_msi.c
+++ b/arch/powerpc/sysdev/fsl_msi.c
@@ -368,7 +368,7 @@ static int __devinit fsl_of_msi_probe(struct platform_device *dev)
 	int err, i, j, irq_index, count;
 	int rc;
 	const u32 *p;
-	struct fsl_msi_feature *features;
+	const struct fsl_msi_feature *features;
 	int len;
 	u32 offset;
 	static const u32 all_avail[] = { 0, NR_MSI_IRQS };
-- 
1.7.10.4

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

* [PATCH 16/25] powerpc/celleb_pci: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (4 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 15/25] powerpc/fsl_msi: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 17/25] watchdog/mpc8xxx: " Uwe Kleine-König
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Benjamin Herrenschmidt,
	Paul Mackerras, cbe-oss-dev
  Cc: linuxppc-dev, linux-arm-kernel, kernel

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	arch/powerpc/platforms/cell/celleb_pci.c: In function 'celleb_setup_phb':
	arch/powerpc/platforms/cell/celleb_pci.c:485:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/powerpc/platforms/cell/celleb_pci.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/cell/celleb_pci.c b/arch/powerpc/platforms/cell/celleb_pci.c
index 5822141..abc8af4 100644
--- a/arch/powerpc/platforms/cell/celleb_pci.c
+++ b/arch/powerpc/platforms/cell/celleb_pci.c
@@ -472,7 +472,7 @@ int __init celleb_setup_phb(struct pci_controller *phb)
 {
 	struct device_node *dev = phb->dn;
 	const struct of_device_id *match;
-	struct celleb_phb_spec *phb_spec;
+	const struct celleb_phb_spec *phb_spec;
 	int rc;
 
 	match = of_match_node(celleb_phb_match, dev);
-- 
1.7.10.4

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

* [PATCH 17/25] watchdog/mpc8xxx: add a const qualifier
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (5 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 16/25] powerpc/celleb_pci: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23 10:37   ` Wim Van Sebroeck
  2012-07-23  9:13 ` [PATCH 21/25] powerpc/fsl_msi: drop unneeded cast to non-const pointer Uwe Kleine-König
                   ` (2 subsequent siblings)
  9 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Wim Van Sebroeck
  Cc: linuxppc-dev, linux-arm-kernel, kernel, linux-watchdog

From: Arnd Bergmann <arnd@arndb.de>

This prepares *of_device_id.data becoming const. Without this change
the following warning would occur:

	drivers/watchdog/mpc8xxx_wdt.c: In function 'mpc8xxx_wdt_probe':
	drivers/watchdog/mpc8xxx_wdt.c:203:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
[ukl: split Arnd's patch by driver and add changelog]
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/watchdog/mpc8xxx_wdt.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/watchdog/mpc8xxx_wdt.c b/drivers/watchdog/mpc8xxx_wdt.c
index 40f7bf1..e6a038a 100644
--- a/drivers/watchdog/mpc8xxx_wdt.c
+++ b/drivers/watchdog/mpc8xxx_wdt.c
@@ -193,7 +193,7 @@ static int __devinit mpc8xxx_wdt_probe(struct platform_device *ofdev)
 	int ret;
 	const struct of_device_id *match;
 	struct device_node *np = ofdev->dev.of_node;
-	struct mpc8xxx_wdt_type *wdt_type;
+	const struct mpc8xxx_wdt_type *wdt_type;
 	u32 freq = fsl_get_sys_freq();
 	bool enabled;
 
-- 
1.7.10.4

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

* [PATCH 21/25] powerpc/fsl_msi: drop unneeded cast to non-const pointer
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (6 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 17/25] watchdog/mpc8xxx: " Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 23/25] i2c/mpc: make data used as *of_device_id.data const Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 24/25] macintosh/mediabay: " Uwe Kleine-König
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Benjamin Herrenschmidt,
	Paul Mackerras, Timur Tabi, Kumar Gala
  Cc: linuxppc-dev, linux-arm-kernel, kernel

From: Arnd Bergmann <arnd@arndb.de>

This cast is unneeded since *of_device_id.data became const.

[ukl: split Arnd's patch by driver and add changelog]

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/powerpc/sysdev/fsl_msi.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c
index a40acd6..51ffafa 100644
--- a/arch/powerpc/sysdev/fsl_msi.c
+++ b/arch/powerpc/sysdev/fsl_msi.c
@@ -502,15 +502,15 @@ static const struct fsl_msi_feature vmpic_msi_feature = {
 static const struct of_device_id fsl_of_msi_ids[] = {
 	{
 		.compatible = "fsl,mpic-msi",
-		.data = (void *)&mpic_msi_feature,
+		.data = &mpic_msi_feature,
 	},
 	{
 		.compatible = "fsl,ipic-msi",
-		.data = (void *)&ipic_msi_feature,
+		.data = &ipic_msi_feature,
 	},
 	{
 		.compatible = "fsl,vmpic-msi",
-		.data = (void *)&vmpic_msi_feature,
+		.data = &vmpic_msi_feature,
 	},
 	{}
 };
-- 
1.7.10.4

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

* [PATCH 23/25] i2c/mpc: make data used as *of_device_id.data const
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (7 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 21/25] powerpc/fsl_msi: drop unneeded cast to non-const pointer Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  2012-07-23  9:13 ` [PATCH 24/25] macintosh/mediabay: " Uwe Kleine-König
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Ben Dooks, Wolfram Sang
  Cc: linuxppc-dev, linux-arm-kernel, kernel, linux-i2c

From: Arnd Bergmann <arnd@arndb.de>

[ukl: split Arnd's patch by driver]

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/i2c/busses/i2c-mpc.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index 775062c..57f7703 100644
--- a/drivers/i2c/busses/i2c-mpc.c
+++ b/drivers/i2c/busses/i2c-mpc.c
@@ -730,24 +730,24 @@ static int mpc_i2c_resume(struct device *dev)
 SIMPLE_DEV_PM_OPS(mpc_i2c_pm_ops, mpc_i2c_suspend, mpc_i2c_resume);
 #endif
 
-static struct mpc_i2c_data mpc_i2c_data_512x __devinitdata = {
+static const struct mpc_i2c_data mpc_i2c_data_512x __devinitdata = {
 	.setup = mpc_i2c_setup_512x,
 };
 
-static struct mpc_i2c_data mpc_i2c_data_52xx __devinitdata = {
+static const struct mpc_i2c_data mpc_i2c_data_52xx __devinitdata = {
 	.setup = mpc_i2c_setup_52xx,
 };
 
-static struct mpc_i2c_data mpc_i2c_data_8313 __devinitdata = {
+static const struct mpc_i2c_data mpc_i2c_data_8313 __devinitdata = {
 	.setup = mpc_i2c_setup_8xxx,
 };
 
-static struct mpc_i2c_data mpc_i2c_data_8543 __devinitdata = {
+static const struct mpc_i2c_data mpc_i2c_data_8543 __devinitdata = {
 	.setup = mpc_i2c_setup_8xxx,
 	.prescaler = 2,
 };
 
-static struct mpc_i2c_data mpc_i2c_data_8544 __devinitdata = {
+static const struct mpc_i2c_data mpc_i2c_data_8544 __devinitdata = {
 	.setup = mpc_i2c_setup_8xxx,
 	.prescaler = 3,
 };
-- 
1.7.10.4

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

* [PATCH 24/25] macintosh/mediabay: make data used as *of_device_id.data const
       [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
                   ` (8 preceding siblings ...)
  2012-07-23  9:13 ` [PATCH 23/25] i2c/mpc: make data used as *of_device_id.data const Uwe Kleine-König
@ 2012-07-23  9:13 ` Uwe Kleine-König
  9 siblings, 0 replies; 18+ messages in thread
From: Uwe Kleine-König @ 2012-07-23  9:13 UTC (permalink / raw)
  To: Grant Likely, Arnd Bergmann, Rob Herring, Benjamin Herrenschmidt
  Cc: linuxppc-dev, linux-arm-kernel, kernel

From: Arnd Bergmann <arnd@arndb.de>

[ukl: split Arnd's patch by driver]

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/macintosh/mediabay.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/macintosh/mediabay.c b/drivers/macintosh/mediabay.c
index 54bf584..3f8d032 100644
--- a/drivers/macintosh/mediabay.c
+++ b/drivers/macintosh/mediabay.c
@@ -669,7 +669,7 @@ static int media_bay_resume(struct macio_dev *mdev)
 
 /* Definitions of "ops" structures.
  */
-static struct mb_ops ohare_mb_ops = {
+static const struct mb_ops ohare_mb_ops = {
 	.name		= "Ohare",
 	.content	= ohare_mb_content,
 	.power		= ohare_mb_power,
@@ -678,7 +678,7 @@ static struct mb_ops ohare_mb_ops = {
 	.un_reset_ide	= ohare_mb_un_reset_ide,
 };
 
-static struct mb_ops heathrow_mb_ops = {
+static const struct mb_ops heathrow_mb_ops = {
 	.name		= "Heathrow",
 	.content	= heathrow_mb_content,
 	.power		= heathrow_mb_power,
@@ -687,7 +687,7 @@ static struct mb_ops heathrow_mb_ops = {
 	.un_reset_ide	= heathrow_mb_un_reset_ide,
 };
 
-static struct mb_ops keylargo_mb_ops = {
+static const struct mb_ops keylargo_mb_ops = {
 	.name		= "KeyLargo",
 	.init		= keylargo_mb_init,
 	.content	= keylargo_mb_content,
-- 
1.7.10.4

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

* Re: [PATCH 17/25] watchdog/mpc8xxx: add a const qualifier
  2012-07-23  9:13 ` [PATCH 17/25] watchdog/mpc8xxx: " Uwe Kleine-König
@ 2012-07-23 10:37   ` Wim Van Sebroeck
  0 siblings, 0 replies; 18+ messages in thread
From: Wim Van Sebroeck @ 2012-07-23 10:37 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: linux-watchdog, Arnd Bergmann, Rob Herring, kernel, linuxppc-dev,
	linux-arm-kernel

Hi Uwe,

> From: Arnd Bergmann <arnd@arndb.de>
> 
> This prepares *of_device_id.data becoming const. Without this change
> the following warning would occur:
> 
> 	drivers/watchdog/mpc8xxx_wdt.c: In function 'mpc8xxx_wdt_probe':
> 	drivers/watchdog/mpc8xxx_wdt.c:203:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> [ukl: split Arnd's patch by driver and add changelog]
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Acked-by: Wim Van Sebroeck <wim@iguana.be>

Kind regards,
Wim.

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

* Re: [PATCH 08/25] gpio/mpc8xxx: add a const qualifier
  2012-07-23  9:13 ` [PATCH 08/25] gpio/mpc8xxx: add a const qualifier Uwe Kleine-König
@ 2012-07-29 16:13   ` Linus Walleij
  0 siblings, 0 replies; 18+ messages in thread
From: Linus Walleij @ 2012-07-29 16:13 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Linus Walleij, Arnd Bergmann, Rob Herring, kernel, linuxppc-dev,
	linux-arm-kernel

On Mon, Jul 23, 2012 at 11:13 AM, Uwe Kleine-K=F6nig
<u.kleine-koenig@pengutronix.de> wrote:

> This prepares *of_device_id.data becoming const. Without this change
> the following warning would occur:
>
>         drivers/gpio/gpio-mpc8xxx.c: In function 'mpc8xxx_add_controller'=
:
>         drivers/gpio/gpio-mpc8xxx.c:360:30: warning: assignment discards =
'const' qualifier from pointer target type [enabled by default]
>
> Signed-off-by: Uwe Kleine-K=F6nig <u.kleine-koenig@pengutronix.de>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij

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

* Re: [PATCH 13/25] macintosh/mediabay: add a const qualifier
  2012-07-23  9:13 ` [PATCH 13/25] macintosh/mediabay: " Uwe Kleine-König
@ 2012-09-05  2:40   ` Benjamin Herrenschmidt
  2012-09-05  8:02     ` Uwe Kleine-König
  0 siblings, 1 reply; 18+ messages in thread
From: Benjamin Herrenschmidt @ 2012-09-05  2:40 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Arnd Bergmann, Rob Herring, kernel, linuxppc-dev, linux-arm-kernel

On Mon, 2012-07-23 at 11:13 +0200, Uwe Kleine-König wrote:
> This prepares *of_device_id.data becoming const. Without this change
> the following warning would occur:
> 
> 	drivers/macintosh/mediabay.c: In function 'media_bay_attach':
> 	drivers/macintosh/mediabay.c:589:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---

Ack all of these assuming you test built (I didn't). Do you need me to
carry any of this via the powerpc tree ?

Cheers,
Ben.

>  drivers/macintosh/mediabay.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/macintosh/mediabay.c b/drivers/macintosh/mediabay.c
> index 831d751..54bf584 100644
> --- a/drivers/macintosh/mediabay.c
> +++ b/drivers/macintosh/mediabay.c
> @@ -63,7 +63,7 @@ struct media_bay_info {
>  	int				value_count;
>  	int				timer;
>  	struct macio_dev		*mdev;
> -	struct mb_ops*			ops;
> +	const struct mb_ops*		ops;
>  	int				index;
>  	int				cached_gpio;
>  	int				sleeping;

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

* Re: [PATCH 13/25] macintosh/mediabay: add a const qualifier
  2012-09-05  2:40   ` Benjamin Herrenschmidt
@ 2012-09-05  8:02     ` Uwe Kleine-König
  2012-09-05  9:23       ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-09-05  8:02 UTC (permalink / raw)
  To: Benjamin Herrenschmidt
  Cc: Arnd Bergmann, Rob Herring, kernel, linuxppc-dev, linux-arm-kernel

Hello,

On Wed, Sep 05, 2012 at 12:40:17PM +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2012-07-23 at 11:13 +0200, Uwe Kleine-König wrote:
> > This prepares *of_device_id.data becoming const. Without this change
> > the following warning would occur:
> > 
> > 	drivers/macintosh/mediabay.c: In function 'media_bay_attach':
> > 	drivers/macintosh/mediabay.c:589:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]
> > 
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > ---
> 
> Ack all of these assuming you test built (I didn't).
"all" means the two mediabay patches? And yes, they are all built
tested.

>                                                      Do you need me to
> carry any of this via the powerpc tree ?
The patch that adds the const to of_device_id depends[1] on this patch.
And the other mediabay is only valid after that const is added. So the
easiest is if the series is pulled in one go. Arnd intended to let Linus
pull it for 3.6-rc1 but it was missed because of a communication
problem. So I intend to get it in during the 3.7 merge window.

Best regards
Uwe

[1] it's only a soft depend, because the result is "only" a warning, but
still.

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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

* Re: [PATCH 13/25] macintosh/mediabay: add a const qualifier
  2012-09-05  8:02     ` Uwe Kleine-König
@ 2012-09-05  9:23       ` Benjamin Herrenschmidt
  2012-09-11  6:56         ` Uwe Kleine-König
  0 siblings, 1 reply; 18+ messages in thread
From: Benjamin Herrenschmidt @ 2012-09-05  9:23 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Arnd Bergmann, Rob Herring, kernel, linuxppc-dev, linux-arm-kernel

On Wed, 2012-09-05 at 10:02 +0200, Uwe Kleine-König wrote:
> Hello,
> 
> On Wed, Sep 05, 2012 at 12:40:17PM +1000, Benjamin Herrenschmidt wrote:
> > On Mon, 2012-07-23 at 11:13 +0200, Uwe Kleine-König wrote:
> > > This prepares *of_device_id.data becoming const. Without this change
> > > the following warning would occur:
> > > 
> > > 	drivers/macintosh/mediabay.c: In function 'media_bay_attach':
> > > 	drivers/macintosh/mediabay.c:589:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]
> > > 
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > ---
> > 
> > Ack all of these assuming you test built (I didn't).
> "all" means the two mediabay patches? And yes, they are all built
> tested.

Yeah, anything powerpc you had in your series, but the two mediabay ones
are what I spotted when digging through stuff today :-)

> >                                                      Do you need me to
> > carry any of this via the powerpc tree ?
> The patch that adds the const to of_device_id depends[1] on this patch.
> And the other mediabay is only valid after that const is added. So the
> easiest is if the series is pulled in one go. Arnd intended to let Linus
> pull it for 3.6-rc1 but it was missed because of a communication
> problem. So I intend to get it in during the 3.7 merge window.

Ok, I'm happy for Arnd to carry all of these, I trust his
judgement/review for that stuff including for ppc specific bits.

Cheers,
Ben.

> Best regards
> Uwe
> 
> [1] it's only a soft depend, because the result is "only" a warning, but
> still.
> 

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

* Re: [PATCH 13/25] macintosh/mediabay: add a const qualifier
  2012-09-05  9:23       ` Benjamin Herrenschmidt
@ 2012-09-11  6:56         ` Uwe Kleine-König
  2012-09-11  7:08           ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 18+ messages in thread
From: Uwe Kleine-König @ 2012-09-11  6:56 UTC (permalink / raw)
  To: Benjamin Herrenschmidt
  Cc: Arnd Bergmann, Rob Herring, kernel, linuxppc-dev, linux-arm-kernel

On Wed, Sep 05, 2012 at 07:23:43PM +1000, Benjamin Herrenschmidt wrote:
> On Wed, 2012-09-05 at 10:02 +0200, Uwe Kleine-König wrote:
> > Hello,
> > 
> > On Wed, Sep 05, 2012 at 12:40:17PM +1000, Benjamin Herrenschmidt wrote:
> > > On Mon, 2012-07-23 at 11:13 +0200, Uwe Kleine-König wrote:
> > > > This prepares *of_device_id.data becoming const. Without this change
> > > > the following warning would occur:
> > > > 
> > > > 	drivers/macintosh/mediabay.c: In function 'media_bay_attach':
> > > > 	drivers/macintosh/mediabay.c:589:11: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]
> > > > 
> > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > > ---
> > > 
> > > Ack all of these assuming you test built (I didn't).
> > "all" means the two mediabay patches? And yes, they are all built
> > tested.
> 
> Yeah, anything powerpc you had in your series, but the two mediabay ones
> are what I spotted when digging through stuff today :-)
I added that to

 serial/mpc52xx_uart: add a const qualifier
 gpio/mpc8xxx: add a const qualifier
 i2c/mpc: add a const qualifier
 macintosh/mediabay: add a const qualifier
 powerpc/83xx: add a const qualifier
 powerpc/fsl_msi: add a const qualifier
 powerpc/celleb_pci: add a const qualifier
 watchdog/mpc8xxx: add a const qualifier
 powerpc/fsl_msi: drop unneeded cast to non-const pointer
 i2c/mpc: make data used as *of_device_id.data const
 macintosh/mediabay: make data used as *of_device_id.data const
 can: mpc5xxx_can: make data used as *of_device_id.data const

in the hope I didn't interpret "anything powerpc" too wide. Please tell
me if I did.

Thanks
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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

* Re: [PATCH 13/25] macintosh/mediabay: add a const qualifier
  2012-09-11  6:56         ` Uwe Kleine-König
@ 2012-09-11  7:08           ` Benjamin Herrenschmidt
  0 siblings, 0 replies; 18+ messages in thread
From: Benjamin Herrenschmidt @ 2012-09-11  7:08 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Arnd Bergmann, Rob Herring, kernel, linuxppc-dev, linux-arm-kernel

On Tue, 2012-09-11 at 08:56 +0200, Uwe Kleine-König wrote:
> 
> in the hope I didn't interpret "anything powerpc" too wide. Please
> tell me if I did. 

No, it's fine with me, give me a git URL if you want me to run that
through my various config test builds.

Cheers,
Ben.

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

* Re: [PATCH 15/25] powerpc/fsl_msi: add a const qualifier
  2012-07-23  9:13 ` [PATCH 15/25] powerpc/fsl_msi: " Uwe Kleine-König
@ 2012-10-15 16:14   ` Tabi Timur-B04825
  0 siblings, 0 replies; 18+ messages in thread
From: Tabi Timur-B04825 @ 2012-10-15 16:14 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Arnd Bergmann, Tabi Timur-B04825, Rob Herring, Paul Mackerras,
	kernel, linuxppc-dev, linux-arm-kernel

On Mon, Jul 23, 2012 at 4:13 AM, Uwe Kleine-K=F6nig
<u.kleine-koenig@pengutronix.de> wrote:
> This prepares *of_device_id.data becoming const. Without this change
> the following warning would occur:
>
>         arch/powerpc/sysdev/fsl_msi.c: In function 'fsl_of_msi_probe':
>         arch/powerpc/sysdev/fsl_msi.c:379:11: error: assignment discards =
'const' qualifier from pointer target type [-Werror]
>
> Signed-off-by: Uwe Kleine-K=F6nig <u.kleine-koenig@pengutronix.de>

Acked-by: Timur Tabi <timur@freescale.com>

--=20
Timur Tabi
Linux kernel developer at Freescale=

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

end of thread, other threads:[~2012-10-15 16:14 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1343034810-3386-1-git-send-email-u.kleine-koenig@pengutronix.de>
2012-07-23  9:13 ` [PATCH 08/25] gpio/mpc8xxx: add a const qualifier Uwe Kleine-König
2012-07-29 16:13   ` Linus Walleij
2012-07-23  9:13 ` [PATCH 10/25] i2c/mpc: " Uwe Kleine-König
2012-07-23  9:13 ` [PATCH 13/25] macintosh/mediabay: " Uwe Kleine-König
2012-09-05  2:40   ` Benjamin Herrenschmidt
2012-09-05  8:02     ` Uwe Kleine-König
2012-09-05  9:23       ` Benjamin Herrenschmidt
2012-09-11  6:56         ` Uwe Kleine-König
2012-09-11  7:08           ` Benjamin Herrenschmidt
2012-07-23  9:13 ` [PATCH 14/25] powerpc/83xx: " Uwe Kleine-König
2012-07-23  9:13 ` [PATCH 15/25] powerpc/fsl_msi: " Uwe Kleine-König
2012-10-15 16:14   ` Tabi Timur-B04825
2012-07-23  9:13 ` [PATCH 16/25] powerpc/celleb_pci: " Uwe Kleine-König
2012-07-23  9:13 ` [PATCH 17/25] watchdog/mpc8xxx: " Uwe Kleine-König
2012-07-23 10:37   ` Wim Van Sebroeck
2012-07-23  9:13 ` [PATCH 21/25] powerpc/fsl_msi: drop unneeded cast to non-const pointer Uwe Kleine-König
2012-07-23  9:13 ` [PATCH 23/25] i2c/mpc: make data used as *of_device_id.data const Uwe Kleine-König
2012-07-23  9:13 ` [PATCH 24/25] macintosh/mediabay: " Uwe Kleine-König

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