| From | Linus Walleij <> | Date | Sun, 1 May 2022 23:45:06 +0200 | Subject | Re: [PATCH 21/48] ARM: pxa: eseries: use gpio lookup for audio |
| |
On Tue, Apr 19, 2022 at 6:41 PM Arnd Bergmann <arnd@kernel.org> wrote:
> From: Arnd Bergmann <arnd@arndb.de> > > The three eseries machines have very similar drivers for audio, all > using the mach/eseries-gpio.h header for finding the gpio numbers. > > Change these to use gpio descriptors to avoid the header file > dependency. > > I convert the _OFF gpio numbers into GPIO_ACTIVE_LOW ones for > consistency here. > > Acked-by: Mark Brown <broonie@kernel.org> > Acked-by: Robert Jarzmik <robert.jarzmik@free.fr> > Reviewed-by: Linus Walleij <linus.walleij@linaro.org> > Cc: alsa-devel@alsa-project.org > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Nice use of active low. Maybe I would simply have dropped the _OFF suffix on these GPIO lines as it can be confusing now that their active level is encoded but no big deal. Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours, Linus Walleij
|