[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