linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: imx: fix mc9s08dz60_gpiochip_match parameter type
@ 2012-03-31  8:46 Shawn Guo
  2012-03-31 13:24 ` Fabio Estevam
  0 siblings, 1 reply; 3+ messages in thread
From: Shawn Guo @ 2012-03-31  8:46 UTC (permalink / raw)
  To: linux-arm-kernel

Due to the change made by commit 6e2cf65 (gpio: constify the data
parameter to gpiochip_find()), the following compile warning is seen
on imx_v6_v7_defconfig build.

  CC      arch/arm/mach-imx/mach-mx35_3ds.o
arch/arm/mach-imx/mach-mx35_3ds.c: In function ?mx35_3ds_lcd_set_power?:
arch/arm/mach-imx/mach-mx35_3ds.c:112:5: warning: passing argument 2 of ?gpiochip_find? from incompatible pointer type [enabled by default]
include/asm-generic/gpio.h:145:26: note: expected ?int (*)(struct gpio_chip *, const void *)? but argument is of type ?int (*)(struct gpio_chip *, void *)?

Change mc9s08dz60_gpiochip_match parameter type accordingly to fix
the warning.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
 arch/arm/mach-imx/mach-mx35_3ds.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-imx/mach-mx35_3ds.c b/arch/arm/mach-imx/mach-mx35_3ds.c
index e14291d..6ae51c6 100644
--- a/arch/arm/mach-imx/mach-mx35_3ds.c
+++ b/arch/arm/mach-imx/mach-mx35_3ds.c
@@ -97,7 +97,7 @@ static struct i2c_board_info __initdata i2c_devices_3ds[] = {
 static int lcd_power_gpio = -ENXIO;
 
 static int mc9s08dz60_gpiochip_match(struct gpio_chip *chip,
-						     void *data)
+						     const void *data)
 {
 	return !strcmp(chip->label, data);
 }
-- 
1.7.5.4

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

* [PATCH] ARM: imx: fix mc9s08dz60_gpiochip_match parameter type
  2012-03-31  8:46 [PATCH] ARM: imx: fix mc9s08dz60_gpiochip_match parameter type Shawn Guo
@ 2012-03-31 13:24 ` Fabio Estevam
  2012-03-31 13:28   ` Shawn Guo
  0 siblings, 1 reply; 3+ messages in thread
From: Fabio Estevam @ 2012-03-31 13:24 UTC (permalink / raw)
  To: linux-arm-kernel

Shawn,

On Sat, Mar 31, 2012 at 5:46 AM, Shawn Guo <shawn.guo@linaro.org> wrote:
> Due to the change made by commit 6e2cf65 (gpio: constify the data
> parameter to gpiochip_find()), the following compile warning is seen
> on imx_v6_v7_defconfig build.
>
> ?CC ? ? ?arch/arm/mach-imx/mach-mx35_3ds.o
> arch/arm/mach-imx/mach-mx35_3ds.c: In function ?mx35_3ds_lcd_set_power?:
> arch/arm/mach-imx/mach-mx35_3ds.c:112:5: warning: passing argument 2 of ?gpiochip_find? from incompatible pointer type [enabled by default]
> include/asm-generic/gpio.h:145:26: note: expected ?int (*)(struct gpio_chip *, const void *)? but argument is of type ?int (*)(struct gpio_chip *, void *)?
>
> Change mc9s08dz60_gpiochip_match parameter type accordingly to fix
> the warning.
>
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>

I have already sent this fix:
http://comments.gmane.org/gmane.linux.ports.arm.kernel/158438

Thanks,

Fabio Estevam

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

* [PATCH] ARM: imx: fix mc9s08dz60_gpiochip_match parameter type
  2012-03-31 13:24 ` Fabio Estevam
@ 2012-03-31 13:28   ` Shawn Guo
  0 siblings, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2012-03-31 13:28 UTC (permalink / raw)
  To: linux-arm-kernel

On 31 March 2012 21:24, Fabio Estevam <festevam@gmail.com> wrote:
....
> I have already sent this fix:
> http://comments.gmane.org/gmane.linux.ports.arm.kernel/158438
>
Okay, ignore mine then.

Regards,
Shawn

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

end of thread, other threads:[~2012-03-31 13:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-31  8:46 [PATCH] ARM: imx: fix mc9s08dz60_gpiochip_match parameter type Shawn Guo
2012-03-31 13:24 ` Fabio Estevam
2012-03-31 13:28   ` Shawn Guo

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