Re: [PATCH v2 0/5] GPIO driver for Maxim MAX3191x

From: Linus Walleij
Date: Fri Oct 13 2017 - 08:48:55 EST


On Thu, Oct 12, 2017 at 12:40 PM, Lukas Wunner <lukas@xxxxxxxxx> wrote:

> - Patch [1/5]: Change the argument order of assign_bit() to reflect
> traditional "dst = src" in C. (Peter Zijlstra)
>
> - Patch [2/5]: Update documentation. (Linus Walleij)
> Drop const qualifier from struct gpio_desc ** in all function
> signatures to avoid a cast when passing a non-const array created
> with gpiod_get_array(), which is likely the most common use case.

Applied these two so we get the infrastructure in place so others
can use it too.

> - Patch [3/5]: Newly inserted patch to introduce common property for
> number of daisy-chained devices.
>
> - Patch [4/5]: Add vendor prefix to GPIO identifiers, use boolean
> instead of integer to select mode, rename boolean to ignore
> undervoltage alarms, separate compatible strings with newlines.
> (Rob Herring)
>
> - Patch [5/5]: Optimize algorithm in max3191x_get_multiple() to
> iterate over the bits in the mask, instead of iterating over every
> chip, thus implicitly skipping chips which are not selected at all
> by the mask.

These are pending device tree maintainer review.

Yours,
Linus Walleij