mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Lee Jones <lee@kernel.org>, Krzysztof Kozlowski <krzk@kernel.org>,
	Peter Griffin <peter.griffin@linaro.org>,
	Alim Akhtar <alim.akhtar@samsung.com>,
	Russell King <linux@armlinux.org.uk>,
	Mark Brown <broonie@kernel.org>,
	Linus Walleij <linusw@kernel.org>,
	Bartosz Golaszewski <brgl@bgdev.pl>,
	mfd@lists.linux.dev, linux-samsung-soc@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	patches@opensource.cirrus.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 1/4] mfd: wm831x: Support software nodes for DCDC regulators
Date: Fri, 6 Nov 2026 09:55:18 +0000	[thread overview]
Message-ID: <au2kBlSsypdsdQve@opensource.cirrus.com> (raw)
In-Reply-To: <9fa9add9-6bba-4cd9-9f15-d6ca222a1a23@app.fastmail.com>

On Thu, Nov 05, 2026 at 06:24:48PM +0100, Arnd Bergmann wrote:
> On Thu, Nov 5, 2026, at 18:12, Dmitry Torokhov wrote:
> > On Thu, Nov 05, 2026 at 02:43:17PM +0100, Arnd Bergmann wrote:
> >> 
> >> As far as I understand the situation, wm831x was used on the
> >> reference board (Samsung smdk6410, removed in 2019) in addition
> >> to crag6410 but never anywhere else upstream. The question
> >> then is whether we'll ever see a DT conversion of crag6410.
> >> This would require finding someone willing to actually do
> >> the work, and coming up with a way to handle the autodetection
> >> in mach-crag6410-module.c so the correct spi/i2c codec
> >> child devices are probed.
> >
> > So I guess the question then: do we need to keep this board at all? Or
> > it is time for it to go the way of the dodo? And then we could drop some
> > more drivers as well.
> >
> > My goal for the series was to move along legacy gpio API conversions, as
> > well as move away from the GPIO lookup tables (ideally I want them gone
> > now that we have software nodes supporting describing GPIOs).
> 
> Mark Brown is using the board for testing eight Cirrus/Wolfson
> audio codecs:
> 
>  - arizona and wm8994 have actual users with DT, but this is
>    the board he has for testing with platform_data

Yeah both of these series of parts sold into a lot of systems.

>  - wm0010, wm1250-ev1, wm5100, wm8996 and wm9081 have no other
>    upstream users besides this board and no DT support, so arguably
>    these are probably obsolete and could also be removed.

Personally, I am not super fussed about testing for any of these
parts most of them didn't sell loads and they are quite old.

> I assume Charles can comment on how important the testing still
> is for the codec above, or if there is other hardware that could
> be used for testing the arizona and wm8994 drivers in case the
> other ones are in fact as obsolete as they appear.

Without wanting to put words into Mark's mouth (please do shout
at me if I am off course here), I assume his use of Cragganmore
is less about testing specific parts and more about it being a
very flexible complex audio system with good debug, allowing
testing parts of the core frameworks that would be hard to hit
with off the shelf hardware.

Thanks,
Charles

  parent reply	other threads:[~2026-11-06  9:57 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-25  5:03 [PATCH v4 0/4] ARM: s3c: crag6410: Convert to software nodes and properties Dmitry Torokhov
2026-10-25  5:03 ` [PATCH v4 1/4] mfd: wm831x: Support software nodes for DCDC regulators Dmitry Torokhov
2026-10-26 10:15   ` Charles Keepax
2026-10-28  9:38   ` Bartosz Golaszewski
2026-11-04 18:14   ` Lee Jones
2026-11-05  1:15     ` Dmitry Torokhov
2026-11-05  8:53       ` Lee Jones
2026-11-05 13:43         ` Arnd Bergmann
2026-11-05 13:55           ` Charles Keepax
2026-11-05 14:13             ` Lee Jones
2026-11-05 17:12           ` Dmitry Torokhov
2026-11-05 17:19             ` Mark Brown
2026-11-05 18:07               ` Dmitry Torokhov
2026-11-06 14:07                 ` Mark Brown
2026-11-05 17:24             ` Arnd Bergmann
2026-11-05 17:33               ` Mark Brown
2026-11-06  9:55               ` Charles Keepax [this message]
2026-11-06 11:53                 ` Mark Brown
2026-10-25  5:03 ` [PATCH v4 2/4] ARM: s3c: crag6410: convert PMIC DVS GPIO to software properties Dmitry Torokhov
2026-10-28  9:38   ` Bartosz Golaszewski
2026-10-25  5:03 ` [PATCH v4 3/4] ARM: s3c: crag6410: convert basic-mmio-gpio and LEDs " Dmitry Torokhov
2026-10-25  5:03 ` [PATCH v4 4/4] ARM: s3c: crag6410: convert WM1250 and SPI0 GPIO lookup tables to property entries Dmitry Torokhov
2026-10-26 13:51 ` [PATCH v4 0/4] ARM: s3c: crag6410: Convert to software nodes and properties Arnd Bergmann
2026-10-27  1:07   ` Dmitry Torokhov

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=au2kBlSsypdsdQve@opensource.cirrus.com \
    --to=ckeepax@opensource.cirrus.com \
    --cc=alim.akhtar@samsung.com \
    --cc=arnd@arndb.de \
    --cc=brgl@bgdev.pl \
    --cc=broonie@kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=krzk@kernel.org \
    --cc=lee@kernel.org \
    --cc=linusw@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mfd@lists.linux.dev \
    --cc=patches@opensource.cirrus.com \
    --cc=peter.griffin@linaro.org \
    /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

all inboxes | Powered by JetHome®