From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Mark Brown <broonie@kernel.org>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
"Danilo Krummrich" <dakr@kernel.org>,
"Lyude Paul" <lyude@redhat.com>,
"Alexander Lobakin" <aleksander.lobakin@intel.com>,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
"Bjorn Helgaas" <bhelgaas@google.com>,
"Jonathan Cameron" <Jonathan.Cameron@huawei.com>,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Lukas Wunner" <lukas@wunner.de>,
"Maíra Canal" <mairacanal@riseup.net>,
"Robin Murphy" <robin.murphy@arm.com>,
"Simona Vetter" <simona.vetter@ffwll.ch>,
"Zijun Hu" <quic_zijuhu@quicinc.com>,
linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org,
rust-for-linux@vger.kernel.org
Subject: Re: [PATCH 2/3] regulator: dummy: convert to use the faux bus
Date: Mon, 3 Feb 2025 16:46:02 +0100 [thread overview]
Message-ID: <2025020313-constant-ravishing-1506@gregkh> (raw)
In-Reply-To: <3238a3af-3296-435a-a326-859d0188d51c@sirena.org.uk>
On Mon, Feb 03, 2025 at 03:39:06PM +0000, Mark Brown wrote:
> On Mon, Feb 03, 2025 at 03:25:18PM +0100, Greg Kroah-Hartman wrote:
> > The dummy regulator driver does not need to create a platform device, it
> > only did so because it was simple to do. Change it over to use the
> > faux bus instead as this is NOT a real platform device, and it makes
> > the code even smaller than before.
>
> No, they did this because you explicitly asked that this be done....
I did? What was it attempting to be before this? I don't remember that
at all, sorry.
> > -static int dummy_regulator_probe(struct platform_device *pdev)
> > +static int dummy_regulator_probe(struct faux_device *vdev)
>
> Just dev or fdev - we could just pass a struct device in here, we don't
> actually care that it's a platform device at this point. Having the
> abbreviation mismatch with the device type is odd.
Ah, that's a mistake from my first pass when this was a "struct
virtual_device" and I called this "vdev". I'll go fix that up, thanks.
greg k-h
next prev parent reply other threads:[~2025-02-03 15:46 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-03 14:25 [PATCH 0/3] Driver core: faux bus Greg Kroah-Hartman
2025-02-03 14:25 ` [PATCH 1/3] driver core: add a faux bus for use when a simple device/bus is needed Greg Kroah-Hartman
2025-02-03 15:11 ` Andy Shevchenko
2025-02-03 15:13 ` Andy Shevchenko
2025-02-03 15:35 ` Greg Kroah-Hartman
2025-02-03 15:46 ` Greg Kroah-Hartman
2025-02-03 16:05 ` Andy Shevchenko
2025-02-03 16:05 ` Andy Shevchenko
2025-02-03 16:10 ` Andy Shevchenko
2025-02-03 16:13 ` Greg Kroah-Hartman
2025-02-03 16:22 ` Andy Shevchenko
2025-02-04 9:25 ` Jonathan Cameron
2025-02-04 10:14 ` Greg Kroah-Hartman
2025-02-04 10:08 ` Thomas Weißschuh
2025-02-04 10:20 ` Greg Kroah-Hartman
2025-02-04 10:44 ` Thomas Weißschuh
2025-02-04 10:49 ` Greg Kroah-Hartman
2025-02-03 14:25 ` [PATCH 2/3] regulator: dummy: convert to use the faux bus Greg Kroah-Hartman
2025-02-03 15:39 ` Mark Brown
2025-02-03 15:46 ` Greg Kroah-Hartman [this message]
2025-02-03 16:11 ` Mark Brown
2025-02-03 14:25 ` [PATCH 3/3] USB: phy: convert usb_phy_generic logic to use a faux device Greg Kroah-Hartman
2025-02-03 15:15 ` Andy Shevchenko
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=2025020313-constant-ravishing-1506@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=Jonathan.Cameron@huawei.com \
--cc=aleksander.lobakin@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=broonie@kernel.org \
--cc=dakr@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=lukas@wunner.de \
--cc=lyude@redhat.com \
--cc=mairacanal@riseup.net \
--cc=quic_zijuhu@quicinc.com \
--cc=rafael@kernel.org \
--cc=robin.murphy@arm.com \
--cc=rust-for-linux@vger.kernel.org \
--cc=simona.vetter@ffwll.ch \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox