linux-kernel - Re: [PATCH linux-next v2] ASoC: adau1373: adau1373_hw

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <513F7F46.3050200@canonical.com>
Date:	Tue, 12 Mar 2013 13:17:26 -0600
From:	Tim Gardner <tim.gardner@...onical.com>
To:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
CC:	linux-kernel@...r.kernel.org, Lars-Peter Clausen <lars@...afoo.de>,
	Liam Girdwood <lgirdwood@...il.com>,
	Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.de>,
	device-drivers-devel@...ckfin.uclinux.org,
	alsa-devel@...a-project.org
Subject: Re: [PATCH linux-next v2] ASoC: adau1373: adau1373_hw_params: Silence
 overflow warning

-----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@...afoo.de> Cc: Liam Girdwood
>> <lgirdwood@...il.com> Cc: Mark Brown
>> <broonie@...nsource.wolfsonmicro.com> Cc: Jaroslav Kysela
>> <perex@...ex.cz> Cc: Takashi Iwai <tiwai@...e.de> Cc:
>> device-drivers-devel@...ckfin.uclinux.org Cc:
>> alsa-devel@...a-project.org Signed-off-by: Tim Gardner
>> <tim.gardner@...onical.com>
> 
> 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@...afoo.de> (supporter:ANALOG DEVICES IN...)
Liam Girdwood <lgirdwood@...il.com> (supporter:SOUND - SOC LAYER...)
Mark Brown <broonie@...nsource.wolfsonmicro.com> (supporter:SOUND -
SOC LAYER...)
Jaroslav Kysela <perex@...ex.cz> (maintainer:SOUND)
Takashi Iwai <tiwai@...e.de> (maintainer:SOUND)
device-drivers-devel@...ckfin.uclinux.org (open list:ANALOG DEVICES IN...)
alsa-devel@...a-project.org (moderated list:ANALOG DEVICES IN...)
linux-kernel@...r.kernel.org (open list)

rtg
- -- 
Tim Gardner tim.gardner@...onical.com
-----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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/