On Tue, Feb 28, 2017 at 09:18:56PM +0100, Stefan Wahren wrote: > > > Michael Zoran <mzoran@xxxxxxxxxxxx> 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@xxxxxxxxxxxx> > > --- > > 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 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
