Re: [PATCH v5] gpio: winbond: add driver

From: Maciej S. Szmigiero
Date: Tue Jan 09 2018 - 08:57:55 EST


On 09.01.2018 14:53, Linus Walleij wrote:
> On Sat, Jan 6, 2018 at 12:46 AM, Maciej S. Szmigiero
> <mail@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
>> This commit adds GPIO driver for Winbond Super I/Os.
>>
>> Currently, only W83627UHG model (also known as Nuvoton NCT6627UD) is
>> supported but in the future a support for other Winbond models, too, can
>> be added to the driver.
>>
>> A module parameter "gpios" sets a bitmask of GPIO ports to enable (bit 0 is
>> GPIO1, bit 1 is GPIO2, etc.).
>> One should be careful which ports one tinkers with since some might be
>> managed by the firmware (for functions like powering on and off, sleeping,
>> BIOS recovery, etc.) and some of GPIO port pins are physically shared with
>> other devices included in the Super I/O chip.
>>
>> Signed-off-by: Maciej S. Szmigiero <mail@xxxxxxxxxxxxxxxxxxxxx>
>
>> Changes from v4:
>
> This v5 version applied, I also added Andy's review tag.
>
> Good work!

Thanks!

> Yours,
> Linus Walleij

Best regards,
Maciej Szmigiero