Re: [PATCH v1 0/6] mmc: sdhci-pci: Add some CD GPIO related quirks

From: Andy Shevchenko
Date: Tue Oct 12 2021 - 13:42:29 EST


On Fri, Oct 8, 2021 at 1:29 AM Raul Rangel <rrangel@xxxxxxxxxxxx> wrote:
>
> It looks like you were previously using the `cd-gpios` DT property to
> determine this.

No, it's not the case. The case is that we switched from hard coded
stuff (board files) to ACPI.

> It also sounds like you switched from DT to ACPI so
> you lost the ability to use this field?
>
> Can you not use something like the following:
> https://source.chromium.org/chromiumos/chromiumos/codesearch/+/main:src/third_party/kernel/v5.10/drivers/mmc/host/sdhci-acpi.c;l=945

No. It won't work. Anything like this won't work.

The problem here is on PCB level due to different uSD connectors ()
being use: one is Active High, the other is Active Low for CD signal.

...

> > > Any comments, hints, advice are welcome!

--
With Best Regards,
Andy Shevchenko