* [PATCH] ASoC: soc-cache: A few minor stylistic changes
@ 2010-12-14 15:15 Dimitris Papastamos
2010-12-14 16:02 ` Liam Girdwood
2010-12-14 18:16 ` Mark Brown
0 siblings, 2 replies; 3+ messages in thread
From: Dimitris Papastamos @ 2010-12-14 15:15 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: alsa-devel, patches
Remove redundant parentheses/spaces in the use of the sizeof
operator.
Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
---
sound/soc/soc-cache.c | 14 +++++++-------
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/sound/soc/soc-cache.c b/sound/soc/soc-cache.c
index 678fd75..0e17b40 100644
--- a/sound/soc/soc-cache.c
+++ b/sound/soc/soc-cache.c
@@ -85,7 +85,7 @@ static int snd_soc_4_12_spi_write(void *control_data, const char *data,
msg[1] = data[0];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -166,7 +166,7 @@ static int snd_soc_7_9_spi_write(void *control_data, const char *data,
msg[1] = data[1];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -246,7 +246,7 @@ static int snd_soc_8_8_spi_write(void *control_data, const char *data,
msg[1] = data[1];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -326,7 +326,7 @@ static int snd_soc_8_16_spi_write(void *control_data, const char *data,
msg[2] = data[2];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -513,7 +513,7 @@ static int snd_soc_16_8_spi_write(void *control_data, const char *data,
msg[2] = data[2];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -633,7 +633,7 @@ static int snd_soc_16_16_spi_write(void *control_data, const char *data,
msg[3] = data[3];
spi_message_init(&m);
- memset(&t, 0, (sizeof t));
+ memset(&t, 0, sizeof t);
t.tx_buf = &msg[0];
t.len = len;
@@ -1342,7 +1342,7 @@ static int snd_soc_lzo_cache_init(struct snd_soc_codec *codec)
* that register.
*/
bmp_size = codec_drv->reg_cache_size;
- sync_bmp = kmalloc(BITS_TO_LONGS(bmp_size) * sizeof (long),
+ sync_bmp = kmalloc(BITS_TO_LONGS(bmp_size) * sizeof(long),
GFP_KERNEL);
if (!sync_bmp) {
ret = -ENOMEM;
--
1.7.3.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: soc-cache: A few minor stylistic changes
2010-12-14 15:15 [PATCH] ASoC: soc-cache: A few minor stylistic changes Dimitris Papastamos
@ 2010-12-14 16:02 ` Liam Girdwood
2010-12-14 18:16 ` Mark Brown
1 sibling, 0 replies; 3+ messages in thread
From: Liam Girdwood @ 2010-12-14 16:02 UTC (permalink / raw)
To: Dimitris Papastamos; +Cc: alsa-devel, Mark Brown, patches
On Tue, 2010-12-14 at 15:15 +0000, Dimitris Papastamos wrote:
> Remove redundant parentheses/spaces in the use of the sizeof
> operator.
>
> Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
> ---
> sound/soc/soc-cache.c | 14 +++++++-------
> 1 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/sound/soc/soc-cache.c b/sound/soc/soc-cache.c
> index 678fd75..0e17b40 100644
> --- a/sound/soc/soc-cache.c
> +++ b/sound/soc/soc-cache.c
> @@ -85,7 +85,7 @@ static int snd_soc_4_12_spi_write(void *control_data, const char *data,
> msg[1] = data[0];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -166,7 +166,7 @@ static int snd_soc_7_9_spi_write(void *control_data, const char *data,
> msg[1] = data[1];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -246,7 +246,7 @@ static int snd_soc_8_8_spi_write(void *control_data, const char *data,
> msg[1] = data[1];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -326,7 +326,7 @@ static int snd_soc_8_16_spi_write(void *control_data, const char *data,
> msg[2] = data[2];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -513,7 +513,7 @@ static int snd_soc_16_8_spi_write(void *control_data, const char *data,
> msg[2] = data[2];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -633,7 +633,7 @@ static int snd_soc_16_16_spi_write(void *control_data, const char *data,
> msg[3] = data[3];
>
> spi_message_init(&m);
> - memset(&t, 0, (sizeof t));
> + memset(&t, 0, sizeof t);
>
> t.tx_buf = &msg[0];
> t.len = len;
> @@ -1342,7 +1342,7 @@ static int snd_soc_lzo_cache_init(struct snd_soc_codec *codec)
> * that register.
> */
> bmp_size = codec_drv->reg_cache_size;
> - sync_bmp = kmalloc(BITS_TO_LONGS(bmp_size) * sizeof (long),
> + sync_bmp = kmalloc(BITS_TO_LONGS(bmp_size) * sizeof(long),
> GFP_KERNEL);
> if (!sync_bmp) {
> ret = -ENOMEM;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: soc-cache: A few minor stylistic changes
2010-12-14 15:15 [PATCH] ASoC: soc-cache: A few minor stylistic changes Dimitris Papastamos
2010-12-14 16:02 ` Liam Girdwood
@ 2010-12-14 18:16 ` Mark Brown
1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2010-12-14 18:16 UTC (permalink / raw)
To: Dimitris Papastamos; +Cc: alsa-devel, patches, Liam Girdwood
On Tue, Dec 14, 2010 at 03:15:36PM +0000, Dimitris Papastamos wrote:
> Remove redundant parentheses/spaces in the use of the sizeof
> operator.
>
> Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-12-14 18:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-14 15:15 [PATCH] ASoC: soc-cache: A few minor stylistic changes Dimitris Papastamos
2010-12-14 16:02 ` Liam Girdwood
2010-12-14 18:16 ` 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.