[alsa-devel] [PATCH v2 18/21] ASoC: samsung: i2s: Comments clean up
Sylwester Nawrocki
s.nawrocki at samsung.com
Wed Feb 13 20:07:29 CET 2019
On 2/13/19 09:06, Krzysztof Kozlowski wrote:
>> @@ -742,7 +742,7 @@ static int i2s_hw_params(struct snd_pcm_substream *substream,
>> switch (params_channels(params)) {
>> case 6:
>> val |= MOD_DC2_EN;
>> - /* fall through */
>> + /* Fall through */
> Just to be sure - did you check whether this passes GCC
> -Wimplicit-fallthrough=3 (you need GCC v7)? It looks correct - like
> one of accepted patterns
> (https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html).
I just checked it now and with comment in upper case the warning
is also being suppressed.
--
Regards,
Sylwester
More information about the Alsa-devel
mailing list