Linux-RISC-V Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
@ 2025-07-08 15:36 Geert Uytterhoeven
  2025-07-08 16:48 ` Conor Dooley
  0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2025-07-08 15:36 UTC (permalink / raw)
  To: Mark Brown, Kuninori Morimoto, Liam Girdwood, Jaroslav Kysela,
	Takashi Iwai, Hal Feng, Xingyu Wu, Walker Chen, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Alexandre Ghiti, Conor Dooley
  Cc: linux-riscv, linux-sound, linux-kernel, Geert Uytterhoeven

The SND_SOC_STARFIVE Kconfig symbol was removed, but it is still enabled
in the RISC-V defconfig.  Just remove it, as it is no longer needed.

Fixes: acc317e5254a739e ("ASoC: starfive: Standardize ASoC menu")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/riscv/configs/defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig
index 37c98c0f98ffc0ee..82be92dd70dc29a8 100644
--- a/arch/riscv/configs/defconfig
+++ b/arch/riscv/configs/defconfig
@@ -191,7 +191,6 @@ CONFIG_SOUND=y
 CONFIG_SND=y
 CONFIG_SND_SOC=y
 CONFIG_SND_DESIGNWARE_I2S=m
-CONFIG_SND_SOC_STARFIVE=m
 CONFIG_SND_SOC_JH7110_PWMDAC=m
 CONFIG_SND_SOC_JH7110_TDM=m
 CONFIG_SND_SOC_WM8978=m
-- 
2.43.0


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
  2025-07-08 15:36 [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m Geert Uytterhoeven
@ 2025-07-08 16:48 ` Conor Dooley
  2025-07-08 17:56   ` Palmer Dabbelt
  2025-07-09  9:29   ` Geert Uytterhoeven
  0 siblings, 2 replies; 5+ messages in thread
From: Conor Dooley @ 2025-07-08 16:48 UTC (permalink / raw)
  To: Mark Brown, Kuninori Morimoto, Liam Girdwood, Jaroslav Kysela,
	Takashi Iwai, Hal Feng, Xingyu Wu, Walker Chen, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Alexandre Ghiti, Geert Uytterhoeven
  Cc: conor, Conor Dooley, linux-riscv, linux-sound, linux-kernel

From: Conor Dooley <conor.dooley@microchip.com>

On Tue, 08 Jul 2025 17:36:48 +0200, Geert Uytterhoeven wrote:
> The SND_SOC_STARFIVE Kconfig symbol was removed, but it is still enabled
> in the RISC-V defconfig.  Just remove it, as it is no longer needed.
> 
> 

As I mentioned to Palmer a few mins ago, I'll grab this along with a
couple other soc-related defconfig patches I have. (Apologies if this
goes out twice, I got a warning last time about a timeout, so ending again)

[1/1] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
      https://git.kernel.org/conor/c/493e9b085299

Thanks,
Conor.

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
  2025-07-08 16:48 ` Conor Dooley
@ 2025-07-08 17:56   ` Palmer Dabbelt
  2025-07-09  9:29   ` Geert Uytterhoeven
  1 sibling, 0 replies; 5+ messages in thread
From: Palmer Dabbelt @ 2025-07-08 17:56 UTC (permalink / raw)
  To: Conor Dooley
  Cc: broonie, kuninori.morimoto.gx, lgirdwood, perex, tiwai, hal.feng,
	xingyu.wu, walker.chen, Paul Walmsley, aou, Alexandre Ghiti,
	geert+renesas, Conor Dooley, Conor Dooley, linux-riscv,
	linux-sound, linux-kernel

On Tue, 08 Jul 2025 09:48:27 PDT (-0700), Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
>
> On Tue, 08 Jul 2025 17:36:48 +0200, Geert Uytterhoeven wrote:
>> The SND_SOC_STARFIVE Kconfig symbol was removed, but it is still enabled
>> in the RISC-V defconfig.  Just remove it, as it is no longer needed.
>>
>>
>
> As I mentioned to Palmer a few mins ago, I'll grab this along with a
> couple other soc-related defconfig patches I have. (Apologies if this
> goes out twice, I got a warning last time about a timeout, so ending again)
>
> [1/1] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
>       https://git.kernel.org/conor/c/493e9b085299
>
> Thanks,
> Conor.

Ya, I think it's easiest to just keep these together.  So

Acked-by: Palmer Dabbelt <palmer@dabbelt.com>

in case anyone cares (I don't really care if it makes it to the commit).

Thanks!

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
  2025-07-08 16:48 ` Conor Dooley
  2025-07-08 17:56   ` Palmer Dabbelt
@ 2025-07-09  9:29   ` Geert Uytterhoeven
  2025-07-09 16:12     ` Conor Dooley
  1 sibling, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2025-07-09  9:29 UTC (permalink / raw)
  To: Conor Dooley
  Cc: Mark Brown, Kuninori Morimoto, Liam Girdwood, Jaroslav Kysela,
	Takashi Iwai, Hal Feng, Xingyu Wu, Walker Chen, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Alexandre Ghiti, Geert Uytterhoeven,
	Conor Dooley, linux-riscv, linux-sound, linux-kernel

Hi Conor,

On Tue, 8 Jul 2025 at 18:49, Conor Dooley <conor@kernel.org> wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
>
> On Tue, 08 Jul 2025 17:36:48 +0200, Geert Uytterhoeven wrote:
> > The SND_SOC_STARFIVE Kconfig symbol was removed, but it is still enabled
> > in the RISC-V defconfig.  Just remove it, as it is no longer needed.
>
> As I mentioned to Palmer a few mins ago, I'll grab this along with a
> couple other soc-related defconfig patches I have. (Apologies if this
> goes out twice, I got a warning last time about a timeout, so ending again)
>
> [1/1] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
>       https://git.kernel.org/conor/c/493e9b085299

Sorry, I forgot to mention that the commit my patch fixes is only in
the sound tree.  One other patch in the original series did include a
defconfig update for a  MIPS platform, so it may be more appropriate
for Mark to include it. Or just wait for v6.17 ;-)

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
  2025-07-09  9:29   ` Geert Uytterhoeven
@ 2025-07-09 16:12     ` Conor Dooley
  0 siblings, 0 replies; 5+ messages in thread
From: Conor Dooley @ 2025-07-09 16:12 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Mark Brown, Kuninori Morimoto, Liam Girdwood, Jaroslav Kysela,
	Takashi Iwai, Hal Feng, Xingyu Wu, Walker Chen, Paul Walmsley,
	Palmer Dabbelt, Albert Ou, Alexandre Ghiti, Geert Uytterhoeven,
	Conor Dooley, linux-riscv, linux-sound, linux-kernel


[-- Attachment #1.1: Type: text/plain, Size: 1205 bytes --]

On Wed, Jul 09, 2025 at 11:29:24AM +0200, Geert Uytterhoeven wrote:
> Hi Conor,
> 
> On Tue, 8 Jul 2025 at 18:49, Conor Dooley <conor@kernel.org> wrote:
> > From: Conor Dooley <conor.dooley@microchip.com>
> >
> > On Tue, 08 Jul 2025 17:36:48 +0200, Geert Uytterhoeven wrote:
> > > The SND_SOC_STARFIVE Kconfig symbol was removed, but it is still enabled
> > > in the RISC-V defconfig.  Just remove it, as it is no longer needed.
> >
> > As I mentioned to Palmer a few mins ago, I'll grab this along with a
> > couple other soc-related defconfig patches I have. (Apologies if this
> > goes out twice, I got a warning last time about a timeout, so ending again)
> >
> > [1/1] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m
> >       https://git.kernel.org/conor/c/493e9b085299
> 
> Sorry, I forgot to mention that the commit my patch fixes is only in
> the sound tree.  One other patch in the original series did include a
> defconfig update for a  MIPS platform, so it may be more appropriate
> for Mark to include it. Or just wait for v6.17 ;-)

I was going to include it in 6.17 material, with the other soc-related
defconfig bits. I'll drop it if it's problematic of course.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-07-09 19:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-08 15:36 [PATCH] riscv: defconfig: Remove CONFIG_SND_SOC_STARFIVE=m Geert Uytterhoeven
2025-07-08 16:48 ` Conor Dooley
2025-07-08 17:56   ` Palmer Dabbelt
2025-07-09  9:29   ` Geert Uytterhoeven
2025-07-09 16:12     ` Conor Dooley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox