Message-ID: <20120928075145.GW17667@game.jcrosoft.org>
Date:	Fri, 28 Sep 2012 09:51:45 +0200
From:	Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>
To:	Roland Stigge <stigge@...com.de>
Cc:	grant.likely@...retlab.ca, linus.walleij@...aro.org,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	w.sang@...gutronix.de, jbe@...gutronix.de
Subject: Re: [PATCH RFC 1/2] gpio: Add a block GPIO API to gpiolib

On 09:14 Fri 28 Sep     , Roland Stigge wrote:
> 
> Hi!
> 
> On 09/28/2012 04:47 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> >> +Block GPIO (optional)
> >> +---------------------
> >> +
> >> +The above described interface concentrates on handling single GPIOs.  However,
> >> +in applications where it is critical to set several GPIOs at once, this
> >> +interface doesn't work well, e.g. bit-banging protocols via GPIO lines.
> >> +Consider a GPIO controller that is connected via a slow I2C line. When
> >> +switching two or more GPIOs one after another, there can be considerable time
> >> +between those events. This is solved by an interface called Block GPIO:
> >> +
> >> +void gpio_get_block(unsigned int gpio, u8* values, size_t size);
> >> +void gpio_set_block(unsigned int gpio, u8* set, u8* clr, size_t size);
> >> +
> >> +The function gpio_get_block() detects the current state of several GPIOs at
> >> +once, practically by doing only one query at the hardware level (e.g. memory
> >> +mapped or via bus transfers like I2C). There are some limits to this interface:
> >> +A certain gpio_chip (see below) must be specified via the gpio parameter as the
> >> +first GPIO in the gpio_chip group. The Block GPIO interface only supports
> >> +simultaneous handling of GPIOs in the same gpio_chip group since different
> >> +gpio_chips typically map to different GPIO hardware blocks.
> > so basicaly you use a gpio numberthat you do not request, that is maybe
> > requested. This is broken if you want to get or set block you need to pass the
> > list of GPIO you want to control not some fancy magic
> 
> Right - will add checking for the request state of the respective GPIOs.
> 
> The list of GPIOs to handle is defined by the offset (specified GPIO)
> and bitmapped list.
> 
> If it looks more natural, I can change this to a list of ints specifying
> GPIOs directly.
you pass the correctly information to the gpiolib

as if you do not request the gpio the gpiolib will auto request the gpios
so you api will be 
int gpio_get_block(unsigned int *gpios, u8* values, size_t size);

return an error

int gpio_set_block(unsigned int *gpios, u8* set, size_t size);

so you do not care about the banks you work on the gpiolib framework will call
each gpio_chip seperatly. If the set_block get_block is not availlable the
gpiolib could fallback to get/set

inside of the gpiolib that you call each bank with a bitmapped list is correct
but not in the public gpiolib API

> 
> > And how you can hope to describe this via DT
> 
> Haven't had planned that yet. Finally, this interface should just be
> another view on the GPIOs already requested / assigned. Or which
> additional info do you mean?
how do you plan to give the gpio base vai DT to the driver as via DT we just
pass the list of GPIO to work on
> 
> Thanks for your notes!
> 
> Roland
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists