All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mehmet Fide <mehmet.fide@gmail.com>
To: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>,
	Jaehoon Chung <jh80.chung@samsung.com>,
	Peng Fan <peng.fan@nxp.com>, Vincent Jardin <vjardin@free.fr>,
	Ye Li <ye.li@nxp.com>, Michal Simek <michal.simek@amd.com>,
	Aristo Chen <aristo.chen@canonical.com>,
	u-boot@lists.u-boot-project.org,
	Mehmet Fide <mehmet.fide@screeningeagle.com>
Subject: Re: [PATCH 0/4] dm: gpio: read GPIOs in of_to_plat(), request them in probe()
Date: Thu, 10 Sep 2026 16:58:26 +0200	[thread overview]
Message-ID: <20260910145826.1275945-1-mehmet.fide@gmail.com> (raw)
In-Reply-To: <CAFLszTi0TpqNDeh5FuB9966FRRLqChr3ffhvS7cCzKfaSufLPA@mail.gmail.com>

Hi Simon,

On 2026-09-10 Simon Glass wrote:
> I wonder how many other drivers probe in of_to_plat() ? It would be
> interested to get the scale of the problem, if you can.

I grepped next for gpio_request_by_name(), gpio_request_list_by_name()
and dm_gpio_request() called from an of_to_plat() method: 50 driver
files do it. By subsystem: video 22 (panels and backlights), net 6,
mmc 5, spi 4, usb 3 (ehci-vf among them, which is where I started),
pci 2, i2c 2, power 2 (the regulators in this series), w1, tpm, sound
and misc one each.

Most of them claim reset or enable lines that they then drive from
probe(), so they would convert the same way the regulators do here:
parse in of_to_plat(), request in probe(). I can only test two of
them: fec_mxc and fsl_esdhc_imx run on the Colibri VF50 I have, and I
will convert those in a follow-up series once the helper is settled.
I would rather not touch the other drivers,
the panels in particular, without hardware to check that moving the
claim out of of_to_plat() does not change what they power up and
when. I can post the list so their owners can pick them up.

I will answer the per-patch comments in their threads and send v2 once
those are settled.

Regards,
Mehmet

  reply	other threads:[~2026-09-10 14:58 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-28 10:52 [PATCH 0/4] dm: gpio: read GPIOs in of_to_plat(), request them in probe() Mehmet Fide
2026-08-28 10:52 ` [PATCH 1/4] gpio: add a way to parse a GPIO now and request it later Mehmet Fide
2026-09-10 14:06   ` Simon Glass
2026-09-10 14:58     ` Mehmet Fide
2026-08-28 10:52 ` [PATCH 2/4] regulator: claim the enable GPIO at probe time, not in of_to_plat() Mehmet Fide
2026-09-10 14:08   ` Simon Glass
2026-09-10 14:58     ` Mehmet Fide
2026-08-28 10:52 ` [PATCH 3/4] doc: driver-model: state that of_to_plat() must not probe or claim Mehmet Fide
2026-09-10 14:09   ` Simon Glass
2026-09-10 14:58     ` Mehmet Fide
2026-08-28 10:52 ` [PATCH 4/4] test: dm: check the fixed regulator claims its GPIO at probe time Mehmet Fide
2026-09-10 14:09   ` Simon Glass
2026-09-10 14:58     ` Mehmet Fide
2026-09-10 14:10 ` [0/4] dm: gpio: read GPIOs in of_to_plat(), request them in probe() Simon Glass
2026-09-10 14:58   ` Mehmet Fide [this message]
2026-09-10 15:08     ` [PATCH 0/4] " Simon Glass

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:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

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

  git send-email \
    --in-reply-to=20260910145826.1275945-1-mehmet.fide@gmail.com \
    --to=mehmet.fide@gmail.com \
    --cc=aristo.chen@canonical.com \
    --cc=jh80.chung@samsung.com \
    --cc=mehmet.fide@screeningeagle.com \
    --cc=michal.simek@amd.com \
    --cc=peng.fan@nxp.com \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.u-boot-project.org \
    --cc=vjardin@free.fr \
    --cc=ye.li@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.