* [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() @ 2018-05-29 23:30 Pierre-Louis Bossart 2018-05-30 3:16 ` Vinod 2018-05-30 11:02 ` Applied "ASoC: fix 0-day warnings with snd_soc_new_compress()" to the asoc tree Mark Brown 0 siblings, 2 replies; 7+ messages in thread From: Pierre-Louis Bossart @ 2018-05-29 23:30 UTC (permalink / raw) To: alsa-devel; +Cc: tiwai, liam.r.girdwood, vkoul, broonie, Pierre-Louis Bossart All conditionally-defined routines in include/sound/soc.h expose a static inline fallback to avoid 0-day warnings and compilation issues, except snd_soc_new_compress(). Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> --- include/sound/soc.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/include/sound/soc.h b/include/sound/soc.h index 600a7ebd10c0..1378dcd2128a 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); #ifdef CONFIG_SND_SOC_COMPRESS int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); +#else +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) +{ + return 0; +} #endif void snd_soc_disconnect_sync(struct device *dev); -- 2.14.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() 2018-05-29 23:30 [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() Pierre-Louis Bossart @ 2018-05-30 3:16 ` Vinod 2018-05-30 12:12 ` Pierre-Louis Bossart 2018-05-30 11:02 ` Applied "ASoC: fix 0-day warnings with snd_soc_new_compress()" to the asoc tree Mark Brown 1 sibling, 1 reply; 7+ messages in thread From: Vinod @ 2018-05-30 3:16 UTC (permalink / raw) To: Pierre-Louis Bossart; +Cc: tiwai, liam.r.girdwood, alsa-devel, broonie On 29-05-18, 18:30, Pierre-Louis Bossart wrote: > All conditionally-defined routines in include/sound/soc.h expose a > static inline fallback to avoid 0-day warnings and compilation issues, > except snd_soc_new_compress(). Hey Pierre, Thanks for the fix, but typically the title should not mention things like reporter. It should describe the change, so something like "Add snd_soc_new_compress() wrapper when CONFIG_SND_SOC_COMPRESS is not defined" Right now patch title is not describing the change > Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > --- > include/sound/soc.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/sound/soc.h b/include/sound/soc.h > index 600a7ebd10c0..1378dcd2128a 100644 > --- a/include/sound/soc.h > +++ b/include/sound/soc.h > @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, > int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); > #ifdef CONFIG_SND_SOC_COMPRESS > int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); > +#else > +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) > +{ > + return 0; > +} > #endif > > void snd_soc_disconnect_sync(struct device *dev); > -- > 2.14.1 -- ~Vinod ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() 2018-05-30 3:16 ` Vinod @ 2018-05-30 12:12 ` Pierre-Louis Bossart 2018-05-30 12:22 ` Takashi Iwai 0 siblings, 1 reply; 7+ messages in thread From: Pierre-Louis Bossart @ 2018-05-30 12:12 UTC (permalink / raw) To: Vinod; +Cc: tiwai, liam.r.girdwood, alsa-devel, broonie On 5/29/18 10:16 PM, Vinod wrote: > On 29-05-18, 18:30, Pierre-Louis Bossart wrote: >> All conditionally-defined routines in include/sound/soc.h expose a >> static inline fallback to avoid 0-day warnings and compilation issues, >> except snd_soc_new_compress(). > > Hey Pierre, > > Thanks for the fix, but typically the title should not mention things like > reporter. It should describe the change, so something like "Add > snd_soc_new_compress() wrapper when CONFIG_SND_SOC_COMPRESS is not defined" > > Right now patch title is not describing the change I follow the guidelines in [1] of describing the problem statement, provide details and then how the problem is addressed in the message. It's not clear to me that you need to describe the solution in the title. [1] https://www.kernel.org/doc/html/v4.16/process/submitting-patches.html > >> Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') >> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> >> --- >> include/sound/soc.h | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/include/sound/soc.h b/include/sound/soc.h >> index 600a7ebd10c0..1378dcd2128a 100644 >> --- a/include/sound/soc.h >> +++ b/include/sound/soc.h >> @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, >> int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); >> #ifdef CONFIG_SND_SOC_COMPRESS >> int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); >> +#else >> +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) >> +{ >> + return 0; >> +} >> #endif >> >> void snd_soc_disconnect_sync(struct device *dev); >> -- >> 2.14.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() 2018-05-30 12:12 ` Pierre-Louis Bossart @ 2018-05-30 12:22 ` Takashi Iwai 2018-05-30 12:48 ` Pierre-Louis Bossart 2018-05-30 13:56 ` Mark Brown 0 siblings, 2 replies; 7+ messages in thread From: Takashi Iwai @ 2018-05-30 12:22 UTC (permalink / raw) To: Pierre-Louis Bossart; +Cc: liam.r.girdwood, Vinod, broonie, alsa-devel On Wed, 30 May 2018 14:12:00 +0200, Pierre-Louis Bossart wrote: > > On 5/29/18 10:16 PM, Vinod wrote: > > On 29-05-18, 18:30, Pierre-Louis Bossart wrote: > >> All conditionally-defined routines in include/sound/soc.h expose a > >> static inline fallback to avoid 0-day warnings and compilation issues, > >> except snd_soc_new_compress(). > > > > Hey Pierre, > > > > Thanks for the fix, but typically the title should not mention things like > > reporter. It should describe the change, so something like "Add > > snd_soc_new_compress() wrapper when CONFIG_SND_SOC_COMPRESS is not defined" > > > > Right now patch title is not describing the change > > I follow the guidelines in [1] of describing the problem statement, > provide details and then how the problem is addressed in the message. > > It's not clear to me that you need to describe the solution in the title. > > [1] https://www.kernel.org/doc/html/v4.16/process/submitting-patches.html Well, a good patch describes itself concisely but clearly in the subject line in general. And, in this regard, whether it's reported by 0-day bot or not is pretty much useless. We'd merge a fix not because 0-day bot complains; instead, we merge it because it addresses some real bug. In this particular case, a build error with CONFIG_SND_SOC_COMPRESS=n. So this should have been mentioned in the subject line. Takashi > > > >> Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') > >> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > >> --- > >> include/sound/soc.h | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/include/sound/soc.h b/include/sound/soc.h > >> index 600a7ebd10c0..1378dcd2128a 100644 > >> --- a/include/sound/soc.h > >> +++ b/include/sound/soc.h > >> @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, > >> int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); > >> #ifdef CONFIG_SND_SOC_COMPRESS > >> int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); > >> +#else > >> +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) > >> +{ > >> + return 0; > >> +} > >> #endif > >> void snd_soc_disconnect_sync(struct device *dev); > >> -- > >> 2.14.1 > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() 2018-05-30 12:22 ` Takashi Iwai @ 2018-05-30 12:48 ` Pierre-Louis Bossart 2018-05-30 13:56 ` Mark Brown 1 sibling, 0 replies; 7+ messages in thread From: Pierre-Louis Bossart @ 2018-05-30 12:48 UTC (permalink / raw) To: Takashi Iwai; +Cc: liam.r.girdwood, Vinod, broonie, alsa-devel On 5/30/18 7:22 AM, Takashi Iwai wrote: > On Wed, 30 May 2018 14:12:00 +0200, > Pierre-Louis Bossart wrote: >> >> On 5/29/18 10:16 PM, Vinod wrote: >>> On 29-05-18, 18:30, Pierre-Louis Bossart wrote: >>>> All conditionally-defined routines in include/sound/soc.h expose a >>>> static inline fallback to avoid 0-day warnings and compilation issues, >>>> except snd_soc_new_compress(). >>> >>> Hey Pierre, >>> >>> Thanks for the fix, but typically the title should not mention things like >>> reporter. It should describe the change, so something like "Add >>> snd_soc_new_compress() wrapper when CONFIG_SND_SOC_COMPRESS is not defined" >>> >>> Right now patch title is not describing the change >> >> I follow the guidelines in [1] of describing the problem statement, >> provide details and then how the problem is addressed in the message. >> >> It's not clear to me that you need to describe the solution in the title. >> >> [1] https://www.kernel.org/doc/html/v4.16/process/submitting-patches.html > > Well, a good patch describes itself concisely but clearly in the > subject line in general. And, in this regard, whether it's reported > by 0-day bot or not is pretty much useless. We'd merge a fix not > because 0-day bot complains; instead, we merge it because it addresses > some real bug. In this particular case, a build error with > CONFIG_SND_SOC_COMPRESS=n. So this should have been mentioned in the > subject line. Got it, thanks! > > > Takashi > >>> >>>> Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') >>>> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> >>>> --- >>>> include/sound/soc.h | 5 +++++ >>>> 1 file changed, 5 insertions(+) >>>> >>>> diff --git a/include/sound/soc.h b/include/sound/soc.h >>>> index 600a7ebd10c0..1378dcd2128a 100644 >>>> --- a/include/sound/soc.h >>>> +++ b/include/sound/soc.h >>>> @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, >>>> int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); >>>> #ifdef CONFIG_SND_SOC_COMPRESS >>>> int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); >>>> +#else >>>> +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) >>>> +{ >>>> + return 0; >>>> +} >>>> #endif >>>> void snd_soc_disconnect_sync(struct device *dev); >>>> -- >>>> 2.14.1 >>> >> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() 2018-05-30 12:22 ` Takashi Iwai 2018-05-30 12:48 ` Pierre-Louis Bossart @ 2018-05-30 13:56 ` Mark Brown 1 sibling, 0 replies; 7+ messages in thread From: Mark Brown @ 2018-05-30 13:56 UTC (permalink / raw) To: Takashi Iwai; +Cc: liam.r.girdwood, Vinod, Pierre-Louis Bossart, alsa-devel [-- Attachment #1.1: Type: text/plain, Size: 342 bytes --] On Wed, May 30, 2018 at 02:22:46PM +0200, Takashi Iwai wrote: > Well, a good patch describes itself concisely but clearly in the > subject line in general. And, in this regard, whether it's reported > by 0-day bot or not is pretty much useless. We'd merge a fix not Describing the source of the problem report is what Reported-by is for. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 0 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Applied "ASoC: fix 0-day warnings with snd_soc_new_compress()" to the asoc tree 2018-05-29 23:30 [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() Pierre-Louis Bossart 2018-05-30 3:16 ` Vinod @ 2018-05-30 11:02 ` Mark Brown 1 sibling, 0 replies; 7+ messages in thread From: Mark Brown @ 2018-05-30 11:02 UTC (permalink / raw) To: Pierre-Louis Bossart; +Cc: tiwai, liam.r.girdwood, alsa-devel, broonie, vkoul The patch ASoC: fix 0-day warnings with snd_soc_new_compress() has been applied to the asoc tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark >From 0b014d72ebae14c0c6ab3fb36a442fda91e1a1b3 Mon Sep 17 00:00:00 2001 From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Date: Tue, 29 May 2018 18:30:02 -0500 Subject: [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() All conditionally-defined routines in include/sound/soc.h expose a static inline fallback to avoid 0-day warnings and compilation issues, except snd_soc_new_compress(). Fixes: 5db6aab6f36f ('ASoC: topology: Add support for compressed PCMs') Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Mark Brown <broonie@kernel.org> --- include/sound/soc.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/include/sound/soc.h b/include/sound/soc.h index 600a7ebd10c0..1378dcd2128a 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -462,6 +462,11 @@ struct snd_soc_component *snd_soc_lookup_component(struct device *dev, int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num); #ifdef CONFIG_SND_SOC_COMPRESS int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num); +#else +static inline int snd_soc_new_compress(struct snd_soc_pcm_runtime *rtd, int num) +{ + return 0; +} #endif void snd_soc_disconnect_sync(struct device *dev); -- 2.17.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-05-30 13:56 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-05-29 23:30 [PATCH] ASoC: fix 0-day warnings with snd_soc_new_compress() Pierre-Louis Bossart 2018-05-30 3:16 ` Vinod 2018-05-30 12:12 ` Pierre-Louis Bossart 2018-05-30 12:22 ` Takashi Iwai 2018-05-30 12:48 ` Pierre-Louis Bossart 2018-05-30 13:56 ` Mark Brown 2018-05-30 11:02 ` Applied "ASoC: fix 0-day warnings with snd_soc_new_compress()" to the asoc tree Mark Brown
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.