archive mirror
 help / color / mirror / Atom feed
From: "Arnd Bergmann" <>
To: "Dmitry Torokhov" <>,
	"Alexandre Belloni" <>
	"Claudiu Beznea" <>
Subject: Re: [PATCH] pcmcia: at91_cf: switch to using gpiod API
Date: Sat, 24 Sep 2022 10:33:29 +0200	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <Yy6d7TjqzUwGQnQa@penguin>

On Sat, Sep 24, 2022, at 8:04 AM, Dmitry Torokhov wrote:
> This patch switches the driver to use newer gpiod API instead of legacy
> gpio API. This moves us closer to the goal of stopping exporting
> OF-specific APIs of gpiolib.
> While at it, stop using module-global for regmap.
> Signed-off-by: Dmitry Torokhov <>

This looks good to me overall. Three comments:

> @@ -63,7 +62,7 @@ struct at91_cf_socket {
>  static inline int at91_cf_present(struct at91_cf_socket *cf)
>  {
> -	return !gpio_get_value(cf->board->det_pin);
> +	return gpiod_get_value(cf->board->det_pin);
>  }

a) The change in polarity looks wrong here, I can't really tell
from the patch. If this is intentional, maybe explain it in
the changelog. With that addressed (either way)

Reviewed-by: Arnd Bergmann <>

b) In case you are doing more patches like this one at the moment,
note that I'm in the process of removing all unused board files
for arch/arm/, which will in turn make a lot of drivers unused.
I should be able to provide a branch soon, which can be used to
identify drivers that don't have DT support any more and don't
have any board files. Rather than converting them to gpio
descriptors, we can probably just remove those drivers.

c) I'm not sure about the state of the at91_cf driver. Apparently
we used to have three drivers for the same hardware (pcmcia,
pata and ide), and only the pcmcia driver remained in the tree
after drivers/ide/ was removed and pata_at91 did not get converted
to DT. I think in the long run we will remove the pcmcia layer,
so if you are actually trying to use this hardware, we may want to
revive the pata variant and drop this one instead.
There is no dts file in tree that actually declares either of them,
so chances are that nobody is actually using the CF slot on at91
any more.


  reply	other threads:[~2022-09-24  8:34 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-24  6:04 [PATCH] pcmcia: at91_cf: switch to using gpiod API Dmitry Torokhov
2022-09-24  8:33 ` Arnd Bergmann [this message]
2022-09-24 11:42   ` Alexandre Belloni
2022-09-24 14:17     ` Dominik Brodowski
2022-09-25  4:54       ` Dmitry Torokhov
2022-09-25 12:24       ` Linus Walleij
2022-09-26 15:26         ` Nicolas Ferre
2022-09-25 18:09       ` Alexandre Belloni
2022-09-25 19:06       ` Arnd Bergmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).