[PATCH] ep93xx: gpio.c: local functions should be static
Ryan Mallon
ryan at bluewatersys.com
Tue Apr 6 17:40:23 EDT 2010
H Hartley Sweeten wrote:
> Ryan,
>
> Any comments on this?
Looks fine.
Acked-by: Ryan Mallon <ryan at bluewatersys.com>
> Regards,
> Hartley
>
> -----Original Message-----
> From: linux-arm-kernel-bounces at lists.infradead.org [mailto:linux-arm-kernel-bounces at lists.infradead.org] On Behalf Of H Hartley Sweeten
> Sent: Tuesday, March 09, 2010 1:58 PM
> To: ARM Kernel
> Cc: ryan at bluewatersys.com
> Subject: [PATCH] ep93xx: gpio.c: local functions should be static
>
> The functions ep93xx_gpio_update_int_params and ep93xx_gpio_int_mask
> are not exported and should be static. This was overlooked when
> moving the code from core.c.
>
> Also, change the comment to better indicate what this code is for.
>
> Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
> Cc: Ryan Mallon <ryan at bluewatersys.com>
>
> ---
>
> diff --git a/arch/arm/mach-ep93xx/gpio.c b/arch/arm/mach-ep93xx/gpio.c
> index cc377ae..cf547ad 100644
> --- a/arch/arm/mach-ep93xx/gpio.c
> +++ b/arch/arm/mach-ep93xx/gpio.c
> @@ -25,7 +25,7 @@
> #include <mach/hardware.h>
>
> /*************************************************************************
> - * GPIO handling for EP93xx
> + * Interrupt handling for EP93xx on-chip GPIOs
> *************************************************************************/
> static unsigned char gpio_int_unmasked[3];
> static unsigned char gpio_int_enabled[3];
> @@ -40,7 +40,7 @@ static const u8 eoi_register_offset[3] = { 0x98, 0xb4, 0x54 };
> static const u8 int_en_register_offset[3] = { 0x9c, 0xb8, 0x58 };
> static const u8 int_debounce_register_offset[3] = { 0xa8, 0xc4, 0x64 };
>
> -void ep93xx_gpio_update_int_params(unsigned port)
> +static void ep93xx_gpio_update_int_params(unsigned port)
> {
> BUG_ON(port > 2);
>
> @@ -56,7 +56,7 @@ void ep93xx_gpio_update_int_params(unsigned port)
> EP93XX_GPIO_REG(int_en_register_offset[port]));
> }
>
> -void ep93xx_gpio_int_mask(unsigned line)
> +static inline void ep93xx_gpio_int_mask(unsigned line)
> {
> gpio_int_unmasked[line >> 3] &= ~(1 << (line & 7));
> }
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Bluewater Systems Ltd - ARM Technology Solution Centre
Ryan Mallon 5 Amuri Park, 404 Barbadoes St
ryan at bluewatersys.com PO Box 13 889, Christchurch 8013
http://www.bluewatersys.com New Zealand
Phone: +64 3 3779127 Freecall: Australia 1800 148 751
Fax: +64 3 3779135 USA 1800 261 2934
More information about the linux-arm-kernel
mailing list