On Mon, Jul 7, 2014 at 6:12 PM, Rojhalat Ibrahim [off-list ref] wrote:

On Monday 07 July 2014 17:23:34 Linus Walleij wrote:

quoted

On Mon, Jun 16, 2014 at 3:51 PM, Rojhalat Ibrahim [off-list ref] wrote:

quoted

Introduce new functions gpiod_set_array & gpiod_set_raw_array to the consumer
interface which allow setting multiple outputs with just one function call.
Also add an optional set_multiple function to the driver interface. Without an
implementation of that function in the chip driver outputs are set
sequentially.

Yes this looks good.

Except for one thing I mentioned quite early  I think:

no users!

I'm pretty sure you did _not_ mention this before.

I'm not referring to your patch set specifically. The idea of array handling
has been proposed something like three times, I cannot tell these discussion
threads apart in my head.

But it's a reasonable request don't you think?

I'll take a look at those drivers and hopefully come back with an extended
patch set.

Thanks.

Yours,
Linus Walleij