[PATCH 2/3] staging: bcm2835-audio: select BCM2835_VCHIQ rather then depending on it.

Dan Carpenter dan.carpenter at oracle.com
Wed Mar 1 05:51:23 PST 2017


On Tue, Feb 28, 2017 at 09:18:56PM +0100, Stefan Wahren wrote:
> 
> > Michael Zoran <mzoran at crowfest.net> hat am 28. Februar 2017 um 19:49 geschrieben:
> > 
> > 
> > Change the audio's dependency on BCM2835_VCHIQ to a select.
> > 
> > Signed-off-by: Michael Zoran <mzoran at crowfest.net>
> > ---
> >  drivers/staging/vc04_services/bcm2835-audio/Kconfig | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/staging/vc04_services/bcm2835-audio/Kconfig b/drivers/staging/vc04_services/bcm2835-audio/Kconfig
> > index b2e6d90ef1cb..479c9e3ace11 100644
> > --- a/drivers/staging/vc04_services/bcm2835-audio/Kconfig
> > +++ b/drivers/staging/vc04_services/bcm2835-audio/Kconfig
> > @@ -1,7 +1,8 @@
> >  config SND_BCM2835
> >          tristate "BCM2835 Audio"
> > -        depends on ARCH_BCM2835 && BCM2835_VCHIQ && SND
> > +        depends on ARCH_BCM2835 && SND
> >          select SND_PCM
> > +	select BCM2835_VCHIQ
> >          help
> >            Say Y or M if you want to support BCM2835 built in audio
> >  
> 
> AFAIK "depends on" is perferred instead of "select", because it's causes less issues. Please explain in the commit messages instead of the cover letter why this patch and patch 3 is necessary.

There is a place for both depends and select.

The thing is that users basically don't care about BCM2835_VCHIQ, they
just want audio.  With a depend it shows what they don't care about and
hides what they want which is the opposite of how it should be.

Michael's reference to Linus's email is too vague (not a URL) so I have
no idea what he's talking about.  Selecting BCM2835_VCHIQ doesn't pull
in very much code, and it's all essential anyway.

regards,
dan carpenter




More information about the linux-rpi-kernel mailing list