linux-kernel - Re: [PATCH 0/5] Add pinctrl support for dwapb gpio driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <695291639.96501294.1576227887986.JavaMail.zimbra@kalray.eu>
Date:   Fri, 13 Dec 2019 10:04:48 +0100 (CET)
From:   Clément Leger <cleger@...ray.eu>
To:     Linus Walleij <linus.walleij@...aro.org>
Cc:     Bartosz Golaszewski <bgolaszewski@...libre.com>,
        Hoan Tran <hoan@...amperecomputing.com>,
        "open list, GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>,
        devicetree <devicetree@...r.kernel.org>,
        linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/5] Add pinctrl support for dwapb gpio driver

Ok thanks for your review, I will do that.

Regards,

Clément Léger

----- On 13 Dec, 2019, at 09:59, Linus Walleij linus.walleij@...aro.org wrote:

> On Wed, Dec 4, 2019 at 11:10 AM Clement Leger <cleger@...ray.eu> wrote:
> 
>> Synopsys DWAPB IP includes support for pin control. This control is basic
>> and allows to switch between a hardware and a software function.
>> Software function is when driving GPIOs from IP and hardware is controlled
>> by external signals.
>> This serie export necessary interface to be able to move the driver to
>> pinctrl folder and then implement the pinctrl support which is based on the
>> digicolor driver. The idea is to avoid hardcoding pins in driver since
>> this IP is a generic one available on multiple SoC.
> 
> The overall approach is correct, just tidy up the patch series the
> way indicated by Andy and we can probably proceed efficiently with
> this. I will review the patch adding the pinctrl interfaces separately.
> 
> Yours,
> Linus Walleij