Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 13 Dec 2019 10:04:48 +0100 (CET)
From:   =?utf-8?Q?Cl=C3=A9ment?= Leger <cleger@kalray.eu>
To:     Linus Walleij <linus.walleij@linaro.org>
Cc:     Bartosz Golaszewski <bgolaszewski@baylibre.com>,
        Hoan Tran <hoan@os.amperecomputing.com>,
        "open list, GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
        devicetree <devicetree@vger.kernel.org>,
        linux-kernel <linux-kernel@vger.kernel.org>
Message-ID: <695291639.96501294.1576227887986.JavaMail.zimbra@kalray.eu>
In-Reply-To: <CACRpkdbHGitYzwVEVYLUmaE+Qn0ix1O1bOSjoTJomf+h9b4+rg@mail.gmail.com>
References: <20191204101042.4275-1-cleger@kalray.eu> <CACRpkdbHGitYzwVEVYLUmaE+Qn0ix1O1bOSjoTJomf+h9b4+rg@mail.gmail.com>
Subject: Re: [PATCH 0/5] Add pinctrl support for dwapb gpio driver
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Thread-Topic: Add pinctrl support for dwapb gpio driver
Thread-Index: SRXaXJiVFyFDIZi2qjVcUnwBi2wsHQ==
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Ok thanks for your review, I will do that.

Regards,

Cl=C3=A9ment L=C3=A9ger

----- On 13 Dec, 2019, at 09:59, Linus Walleij linus.walleij@linaro.org wro=
te:

> On Wed, Dec 4, 2019 at 11:10 AM Clement Leger <cleger@kalray.eu> wrote:
>=20
>> Synopsys DWAPB IP includes support for pin control. This control is basi=
c
>> and allows to switch between a hardware and a software function.
>> Software function is when driving GPIOs from IP and hardware is controll=
ed
>> 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.
>=20
> 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.
>=20
> Yours,
> Linus Walleij