Re: [alsa-devel] [PATCH] ASoC: Intel: Add defaults for SND_SST_ options and machine drivers

2017-11-16 Thread Takashi Iwai
On Fri, 17 Nov 2017 00:01:06 +0100, Pierre-Louis Bossart wrote: > > > > On 11/16/2017 04:24 PM, Linus Torvalds wrote: > > On Thu, Nov 16, 2017 at 2:16 PM, Pierre-Louis Bossart > > wrote: > >> Add 'default m' when sensible > > No. > > > > This is not

Re: [alsa-devel] [PATCH] ASoC: Intel: Add defaults for SND_SST_ options and machine drivers

2017-11-16 Thread Takashi Iwai
On Fri, 17 Nov 2017 00:01:06 +0100, Pierre-Louis Bossart wrote: > > > > On 11/16/2017 04:24 PM, Linus Torvalds wrote: > > On Thu, Nov 16, 2017 at 2:16 PM, Pierre-Louis Bossart > > wrote: > >> Add 'default m' when sensible > > No. > > > > This is not sensible, and is not at all what I

Re: [alsa-devel] [PATCH] ASoC: Intel: Add defaults for SND_SST_ options and machine drivers

2017-11-16 Thread Pierre-Louis Bossart
On 11/16/2017 04:24 PM, Linus Torvalds wrote: On Thu, Nov 16, 2017 at 2:16 PM, Pierre-Louis Bossart wrote: Add 'default m' when sensible No. This is not sensible, and is not at all what I suggested. Actual new code should *never* be default 'm' or

Re: [alsa-devel] [PATCH] ASoC: Intel: Add defaults for SND_SST_ options and machine drivers

2017-11-16 Thread Pierre-Louis Bossart
On 11/16/2017 04:24 PM, Linus Torvalds wrote: On Thu, Nov 16, 2017 at 2:16 PM, Pierre-Louis Bossart wrote: Add 'default m' when sensible No. This is not sensible, and is not at all what I suggested. Actual new code should *never* be default 'm' or 'y'. You may think it's supremely