Re: [PATCH v2 2/2] gpio: use "active" and "inactive" instead of "high" and "low" for output hogs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On Wed, May 31, 2023 at 12:51:15PM +0300, Andy Shevchenko wrote:
> On Wed, May 31, 2023 at 9:58 AM Uwe Kleine-König
> <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
> > On Wed, May 31, 2023 at 12:51:34AM +0300, Andy Shevchenko wrote:
> > > On Tue, May 30, 2023 at 6:19 PM Uwe Kleine-König
> > > <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
> 
> ...
> 
> > > As I said before, this does not cover the ACPI case. Consider
> >
> > I don't understand that concern. Currently there is nothing for ACPI
> > that parses "output-high" et al.
> 
> This is not true.
> 
> > So you want me to introduce support for
> > hogs defined by ACPI to fix the strange semantic for dt-defined hogs?
> > What am I missing?
> 
> https://elixir.bootlin.com/linux/v6.4-rc4/source/drivers/gpio/gpiolib-acpi.c#L1262

Ah, that was the necessary hint. Adding the aliases there would be a
third patch in this series, right?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux