Linux Sound subsystem development
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux@treblig.org
Cc: arnd@arndb.de, lee@kernel.org, mchehab@kernel.org,
	lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz,
	tiwai@suse.com, linux-media@vger.kernel.org,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/4] Remove the wl1273 FM Radio
Date: Fri, 8 Aug 2025 18:49:03 +0300	[thread overview]
Message-ID: <20250808154903.GB23187@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20250625133258.78133-1-linux@treblig.org>

On Wed, Jun 25, 2025 at 02:32:54PM +0100, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> I noticed that the wl1273 radio had an unused symbol, but then noticed
> it is on Arnd's unused driver list:
>   https://lore.kernel.org/lkml/a15bb180-401d-49ad-a212-0c81d613fbc8@app.fastmail.com/
> 
> So, delete it.
> The components seem pretty separable, except for Kconfig dependencies.
> 
> That lore URL is over 75 chars, which checkpatch warns about,
> suggestions welcome.
> 
> Dave
> 
> Dr. David Alan Gilbert (4):
>   media: radio-wl1273: Remove
>   ASoC: wl1273: Remove
>   mfd: wl1273-core: Remove
>   mfd: wl1273-core: Remove the header

Mark, Lee, how would you like this to be merged ? I have a large patch
series targetting v6.18 that depends on 1/4, and I would like to merge
it in the media tree as soon as possible after -rc1 gets released.
Patches 1/4, 2/4 and 3/4 are independent of each other, but patch 4/4
depends on the first three. Can we merge 1/4 in the media tree and
provide a stable branch right on top of -rc1 ?

>  .../admin-guide/media/radio-cardlist.rst      |    1 -
>  drivers/media/radio/Kconfig                   |   17 -
>  drivers/media/radio/Makefile                  |    1 -
>  drivers/media/radio/radio-wl1273.c            | 2159 -----------------
>  drivers/mfd/Kconfig                           |   10 -
>  drivers/mfd/Makefile                          |    1 -
>  drivers/mfd/wl1273-core.c                     |  262 --
>  include/linux/mfd/wl1273-core.h               |  277 ---
>  sound/soc/codecs/Kconfig                      |    4 -
>  sound/soc/codecs/Makefile                     |    4 +-
>  sound/soc/codecs/wl1273.c                     |  500 ----
>  sound/soc/codecs/wl1273.h                     |   16 -
>  12 files changed, 1 insertion(+), 3251 deletions(-)
>  delete mode 100644 drivers/media/radio/radio-wl1273.c
>  delete mode 100644 drivers/mfd/wl1273-core.c
>  delete mode 100644 include/linux/mfd/wl1273-core.h
>  delete mode 100644 sound/soc/codecs/wl1273.c
>  delete mode 100644 sound/soc/codecs/wl1273.h

-- 
Regards,

Laurent Pinchart

  parent reply	other threads:[~2025-08-08 15:49 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-25 13:32 [PATCH 0/4] Remove the wl1273 FM Radio linux
2025-06-25 13:32 ` [PATCH 1/4] media: radio-wl1273: Remove linux
2025-08-08 15:52   ` Laurent Pinchart
2025-06-25 13:32 ` [PATCH 2/4] ASoC: wl1273: Remove linux
2025-06-25 14:04   ` Mark Brown
2025-08-14 13:59     ` Dr. David Alan Gilbert
2025-06-25 13:32 ` [PATCH 3/4] mfd: wl1273-core: Remove linux
2025-06-25 13:32 ` [PATCH 4/4] mfd: wl1273-core: Remove the header linux
2025-06-25 14:00 ` [PATCH 0/4] Remove the wl1273 FM Radio Arnd Bergmann
2025-07-01 13:27 ` (subset) " Lee Jones
2025-08-08 15:51   ` Laurent Pinchart
2025-08-08 16:01     ` Dr. David Alan Gilbert
2025-08-08 15:49 ` Laurent Pinchart [this message]
2025-09-02 10:32   ` Lee Jones
2025-09-02 10:35     ` Mark Brown
2025-09-02 11:35     ` Laurent Pinchart
2025-09-02 11:47       ` Mark Brown
2025-09-02 12:10         ` Laurent Pinchart
2025-09-02 17:49           ` Dr. David Alan Gilbert
2025-09-03  7:23             ` Lee Jones
2025-10-02 20:32               ` Dr. David Alan Gilbert
2025-08-14 20:08 ` (subset) " Mark Brown
2025-08-15 11:29   ` Dr. David Alan Gilbert
2025-10-09 15:08 ` Lee Jones

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=20250808154903.GB23187@pendragon.ideasonboard.com \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=lee@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=linux@treblig.org \
    --cc=mchehab@kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox