diff mbox

arm: mach-davinci: use kzalloc()

Message ID 4E3D5B88.1070606@ru.mvista.com (mailing list archive)
State Accepted
Headers show

Commit Message

Sergei Shtylyov Aug. 6, 2011, 3:19 p.m. UTC
Hello.

    Thomas, for the future, please send the DaVinci patches to the DaVinci 
mailing list and maintainers.

WBR, Sergei

-------- Original Message --------
Subject: [PATCH] arm: mach-davinci: use kzalloc()
Date: Sat, 06 Aug 2011 13:26:13 +0200
From: Thomas Meyer <thomas@m3y3r.de>
To: Russell King <linux@arm.linux.org.uk>, 
linux-arm-kernel@lists.infradead.org, Linux Kernel Mailing List 
<linux-kernel@vger.kernel.org>

From: Thomas Meyer <thomas@m3y3r.de>

  Use kzalloc rather than kmalloc followed by memset with 0

  This considers some simple cases that are common and easy to validate
  Note in particular that there are no ...s in the rule, so all of the
  matched code has to be contiguous

  The semantic patch that makes this output is available
  in scripts/coccinelle/api/alloc/kzalloc-simple.cocci.

  More information about semantic patching is available at
  http://coccinelle.lip6.fr/

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---

Comments

Thomas Meyer Aug. 8, 2011, 6:55 p.m. UTC | #1
Am Samstag, den 06.08.2011, 19:19 +0400 schrieb Sergei Shtylyov:
> Hello.

Hello, Sergei
> 
>     Thomas, for the future, please send the DaVinci patches to the DaVinci 
> mailing list and maintainers.

The get_maintainers.pl script doesn't list this mailing list as contact
address for the file arch/arm/mach-davinci/dma.c

How to add this email address to the maintainers file?

> 
> WBR, Sergei
> 
> -------- Original Message --------
> Subject: [PATCH] arm: mach-davinci: use kzalloc()
> Date: Sat, 06 Aug 2011 13:26:13 +0200
> From: Thomas Meyer <thomas@m3y3r.de>
> To: Russell King <linux@arm.linux.org.uk>, 
> linux-arm-kernel@lists.infradead.org, Linux Kernel Mailing List 
> <linux-kernel@vger.kernel.org>
> 
> From: Thomas Meyer <thomas@m3y3r.de>
> 
>   Use kzalloc rather than kmalloc followed by memset with 0
> 
>   This considers some simple cases that are common and easy to validate
>   Note in particular that there are no ...s in the rule, so all of the
>   matched code has to be contiguous
> 
>   The semantic patch that makes this output is available
>   in scripts/coccinelle/api/alloc/kzalloc-simple.cocci.
> 
>   More information about semantic patching is available at
>   http://coccinelle.lip6.fr/
> 
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
> ---
> 
> diff -u -p a/arch/arm/mach-davinci/dma.c b/arch/arm/mach-davinci/dma.c
> --- a/arch/arm/mach-davinci/dma.c 2010-10-26 18:50:12.713889353 +0200
> +++ b/arch/arm/mach-davinci/dma.c 2011-08-01 20:40:38.326583216 +0200
> @@ -1435,12 +1435,11 @@ static int __init edma_probe(struct plat
>   			goto fail1;
>   		}
> 
> -		edma_cc[j] = kmalloc(sizeof(struct edma), GFP_KERNEL);
> +		edma_cc[j] = kzalloc(sizeof(struct edma), GFP_KERNEL);
>   		if (!edma_cc[j]) {
>   			status = -ENOMEM;
>   			goto fail1;
>   		}
> -		memset(edma_cc[j], 0, sizeof(struct edma));
> 
>   		edma_cc[j]->num_channels = min_t(unsigned, info[j]->n_channel,
>   							EDMA_MAX_DMACH);
> 
> 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Sergei Shtylyov Aug. 8, 2011, 7:34 p.m. UTC | #2
Hello.

Thomas Meyer wrote:

>>     Thomas, for the future, please send the DaVinci patches to the DaVinci 
>> mailing list and maintainers.

> The get_maintainers.pl script doesn't list this mailing list as contact
> address for the file arch/arm/mach-davinci/dma.c

    Very strange, as I'm seeing the DaVinci entry with this list and both 
current maintainers (which you didn't CC either).

> How to add this email address to the maintainers file?

    It's already there. Ah, it must be that "(subscribers-only)" thing...

WBR, Sergei
diff mbox

Patch

diff -u -p a/arch/arm/mach-davinci/dma.c b/arch/arm/mach-davinci/dma.c
--- a/arch/arm/mach-davinci/dma.c 2010-10-26 18:50:12.713889353 +0200
+++ b/arch/arm/mach-davinci/dma.c 2011-08-01 20:40:38.326583216 +0200
@@ -1435,12 +1435,11 @@  static int __init edma_probe(struct plat
  			goto fail1;
  		}

-		edma_cc[j] = kmalloc(sizeof(struct edma), GFP_KERNEL);
+		edma_cc[j] = kzalloc(sizeof(struct edma), GFP_KERNEL);
  		if (!edma_cc[j]) {
  			status = -ENOMEM;
  			goto fail1;
  		}
-		memset(edma_cc[j], 0, sizeof(struct edma));

  		edma_cc[j]->num_channels = min_t(unsigned, info[j]->n_channel,
  							EDMA_MAX_DMACH);