Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 14 Jul 2022 11:26:07 +0200
Message-ID: <87sfn4vyds.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     Vitaly Rodionov <vitalyr@opensource.cirrus.com>
Cc:     Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
        Mark Brown <broonie@kernel.org>, <alsa-devel@alsa-project.org>,
        <patches@opensource.cirrus.com>, <linux-kernel@vger.kernel.org>,
        Stefan Binding <sbinding@opensource.cirrus.com>
Subject: Re: [PATCH v8 01/14] ALSA: hda: hda_cs_dsp_ctl: Add Library to support CS_DSP ALSA controls
In-Reply-To: <87tu7kvzav.wl-tiwai@suse.de>
References: <20220630002335.366545-1-vitalyr@opensource.cirrus.com>
        <20220630002335.366545-2-vitalyr@opensource.cirrus.com>
        <87v8s0vzfv.wl-tiwai@suse.de>
        <87tu7kvzav.wl-tiwai@suse.de>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=US-ASCII
Precedence: bulk

On Thu, 14 Jul 2022 11:06:16 +0200,
Takashi Iwai wrote:
> 
> On Thu, 14 Jul 2022 11:03:16 +0200,
> Takashi Iwai wrote:
> > 
> > On Thu, 30 Jun 2022 02:23:22 +0200,
> > Vitaly Rodionov wrote:
> > > --- a/sound/pci/hda/Kconfig
> > > +++ b/sound/pci/hda/Kconfig
> > > @@ -94,6 +94,10 @@ config SND_HDA_PATCH_LOADER
> > >  config SND_HDA_SCODEC_CS35L41
> > >  	tristate
> > >  
> > > +config SND_HDA_CS_DSP_CONTROLS
> > > +	tristate
> > > +	depends on CS_DSP
> > > +
> > >  config SND_HDA_SCODEC_CS35L41_I2C
> > >  	tristate "Build CS35L41 HD-audio side codec support for I2C Bus"
> > >  	depends on I2C
> > 
> > This change alone doesn't give anything useful, unfortunately.
> > 
> > The above form (without prompt) is basically only to be "selected" by
> > others.  And when selected, the "depends" there is just ignored, so
> > it's useless.
> > 
> > That is, a proper way would be something like:
> > 
> > config SND_HDA_CS_DSP_CONTROLS
> > 	tristate
> > 
> > config SND_HDA_SCODEC_CS35L41
> > 	....
> > 	select SND_HDA_CS_DSP_CONTROLS if CS_DSP
> > 
> > ... if you want / need to enable CONFIG_SND_HDA_CS_DSP_CONTROLS
> > conditionally.
> 
> And now I see that the patch 4 has those selects (but select both
> SND_HDA_CS_DSP_CONTROLS and CS_DSP).  So what we need here is to drop
> the superfluous "depends on CS_DSP" and mention in the description
> that the Kconfig will be actually enabled in the later patch.

But no need for resubmission; I fixed this locally.

So I applied up to the patch 8.  At patch 9, it can't build because of
the lack of ASoC side change.

Still waiting for *ANY* reaction from Mark...  Sigh.


thanks,

Takashi