Re: [PATCH linux-next v2] ASoC: adau1373: adau1373_hw_params: Silenceoverflow warning

From: Tim Gardner
Date: Tue Mar 12 2013 - 15:17:36 EST


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

On 03/12/2013 12:47 PM, Mark Brown wrote:
> On Mon, Mar 11, 2013 at 01:18:23PM -0600, Tim Gardner wrote:
>> ADAU1373_BCLKDIV_SOURCE is defined as BIT(5) which uses UL
>> constants. On amd64 the result of the ones complement operator is
>> then truncated to
>
> Applied, but please don't bury patches in the middle of e-mail
> threads (I nearly deleted this unread) and avoid including
>

Point taken. In the future I'll start a new thread for each v(n).

>> Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> Cc: Liam Girdwood
>> <lgirdwood@xxxxxxxxx> Cc: Mark Brown
>> <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> Cc: Jaroslav Kysela
>> <perex@xxxxxxxx> Cc: Takashi Iwai <tiwai@xxxxxxx> Cc:
>> device-drivers-devel@xxxxxxxxxxxxxxxxxxxx Cc:
>> alsa-devel@xxxxxxxxxxxxxxxx Signed-off-by: Tim Gardner
>> <tim.gardner@xxxxxxxxxxxxx>
>
> enormous CC lists like this in the body of the mail.
>

I'm just going by the default output of scripts/get_maintainer.pl. Are
there better options to filter the roles ? In this case I received a
review from a supporter (Lars), and the patch is being committed by
yet another supporter (yourself). Would you have noticed this patch
had I restricted the Cc list to just maintainers and lists ?

Lars-Peter Clausen <lars@xxxxxxxxxx> (supporter:ANALOG DEVICES IN...)
Liam Girdwood <lgirdwood@xxxxxxxxx> (supporter:SOUND - SOC LAYER...)
Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> (supporter:SOUND -
SOC LAYER...)
Jaroslav Kysela <perex@xxxxxxxx> (maintainer:SOUND)
Takashi Iwai <tiwai@xxxxxxx> (maintainer:SOUND)
device-drivers-devel@xxxxxxxxxxxxxxxxxxxx (open list:ANALOG DEVICES IN...)
alsa-devel@xxxxxxxxxxxxxxxx (moderated list:ANALOG DEVICES IN...)
linux-kernel@xxxxxxxxxxxxxxx (open list)

rtg
- --
Tim Gardner tim.gardner@xxxxxxxxxxxxx
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQIcBAEBCgAGBQJRP38/AAoJED12yEX6FEfKppEP/jAi6jFpqKwNF3KPiH4j7E9m
PYiydQTVHxgHlqzhJw+7+hyfz7fgbDSew0BpC84gtUYp2xcqCePkNoExT8COk08U
wzIVm6Z1OT57RZDBMoICcH/Xm9/Y2/cf4eXOgiEw5tfP25Gwae2Vep9NcHkCz3iA
4XNg+Cpq+Bu+l4f0WTTUmZlJmhbuai/knEodz413hIHXOhAXLe/ZnMSE2wh9/UnD
H17z7pijKcAMAtcYJmLGT928U2rereW4rlR3+jh9k7CQ4wADo8XrJHNVXo7fgS4Q
ws7hTWsaMuDTXKvg6fWbD9uMJc3OFodZ/ig0uEbZz7LXoxQd1M8MywJnInz6mGBQ
9UnSg4AZOqs64Hm9X98902Fzro8dAa0ltjyPCxYgFhuORG86+SFseS514Y8aYWS8
3/fdgR/kb4kNK4Zd+bDZv34/IEeIJiXlkUshIbhc58ErzVKP54shova2793UVcOk
f5JmM7Tx/ctm2jzj3NYUS6HUsXucaE1K6HSPZ5qJlZyk6s59jJJwNWTILkwNu3Nj
qxlPLWlRcyY8QuPQILx5podOhDmmAduPc8VOVF0YRed+J39BHJRVer3k/i2S/jkz
ACULLlA6edESk3/poeGdBP1S0UhuI3PlzMh9Kwkkx6O8r8COgNVa1hfBSc6DAj+i
YRSlhhPGE022ZQ6VhKUf
=VhzH
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/