Re: [PATCH] EP93xx: Remove ep93xx_gpio_dbg_show function

[Grant Likely]

On Thu, Mar 31, 2011 at 05:03:53PM +1300, Ryan Mallon wrote:
> On 03/31/2011 05:01 PM, Grant Likely wrote:
> > On Tue, Mar 29, 2011 at 12:03:06PM +1300, Ryan Mallon wrote:
> >> Hartley pointed out that the patch I posted to remove the irq internals
> >> fiddling from ep93xx_gpio_dbg_show does not function as expected. It
> >> will print [interrupt] beside all pins which are capable of being
> >> interrupts, not just those which are currently configured as interrupts.
> >>
> >> The best solution is just to remove the custom ep93xx gpio debugfs
> >> function all together. The generic gpiolib one is good enough.
> >>
> >> Signed-off-by: Ryan Mallon <ryan@xxxxxxxxxxxxxxxx>
> > 
> > Applied, thanks.
> > 
> > g.
> 
> This is presently sitting in rmk's patch tracking system. I take it I
> should drop it from there?

Actually, no.  I missed it was under arch/arm instead of drivers/gpio.
I'll drop it from my tree.

g.

> 
> ~Ryan
> 
> >> ---
> >>
> >> diff --git a/arch/arm/mach-ep93xx/gpio.c b/arch/arm/mach-ep93xx/gpio.c
> >> index 34e071d..c74adf9 100644
> >> --- a/arch/arm/mach-ep93xx/gpio.c
> >> +++ b/arch/arm/mach-ep93xx/gpio.c
> >> @@ -348,29 +348,6 @@ static int ep93xx_gpio_set_debounce(struct gpio_chip *chip,
> >>  	return 0;
> >>  }
> >>  
> >> -static void ep93xx_gpio_dbg_show(struct seq_file *s, struct gpio_chip *chip)
> >> -{
> >> -	struct ep93xx_gpio_chip *ep93xx_chip = to_ep93xx_gpio_chip(chip);
> >> -	u8 data_reg, data_dir_reg;
> >> -	int gpio, i;
> >> -
> >> -	data_reg = __raw_readb(ep93xx_chip->data_reg);
> >> -	data_dir_reg = __raw_readb(ep93xx_chip->data_dir_reg);
> >> -
> >> -	gpio = ep93xx_chip->chip.base;
> >> -	for (i = 0; i < chip->ngpio; i++, gpio++) {
> >> -		int is_out = data_dir_reg & (1 << i);
> >> -		int irq = gpio_to_irq(gpio);
> >> -
> >> -		seq_printf(s, " %s%d gpio-%-3d (%-12s) %s %s %s\n",
> >> -				chip->label, i, gpio,
> >> -				gpiochip_is_requested(chip, i) ? : "",
> >> -				is_out ? "out" : "in ",
> >> -				(data_reg & (1<<  i)) ? "hi" : "lo",
> >> -				(!is_out && irq>= 0) ? "(interrupt)" : "");
> >> -	}
> >> -}
> >> -
> >>  #define EP93XX_GPIO_BANK(name, dr, ddr, base_gpio)			\
> >>  	{								\
> >>  		.chip = {						\
> >> @@ -379,7 +356,6 @@ static void ep93xx_gpio_dbg_show(struct seq_file *s, struct gpio_chip *chip)
> >>  			.direction_output = ep93xx_gpio_direction_output, \
> >>  			.get		  = ep93xx_gpio_get,		\
> >>  			.set		  = ep93xx_gpio_set,		\
> >> -			.dbg_show	  = ep93xx_gpio_dbg_show,	\
> >>  			.base		  = base_gpio,			\
> >>  			.ngpio		  = 8,				\
> >>  		},							\
> >>
> 
> 
> -- 
> Bluewater Systems Ltd - ARM Technology Solution Centre
> 
> Ryan Mallon         		5 Amuri Park, 404 Barbadoes St
> ryan@xxxxxxxxxxxxxxxx         	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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/