Re: [PATCH] aic7xxx: fix coccicheck warnings

[Silviu Popescu]

On Sun, Mar 10, 2013 at 10:04 PM, Silviu-Mihai Popescu
<silviupopescu1990@xxxxxxxxx> wrote:
> This replaces a call to kmalloc() followed by memset() with just one
> call to kzalloc().
>
> Signed-off-by: Silviu-Mihai Popescu <silviupopescu1990@xxxxxxxxx>
> ---
>  drivers/scsi/aic7xxx/aic79xx_osm.c |    3 +--
>  drivers/scsi/aic7xxx/aic7xxx_osm.c |    3 +--
>  2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/aic7xxx/aic79xx_osm.c b/drivers/scsi/aic7xxx/aic79xx_osm.c
> index 9328121..f6fdb65 100644
> --- a/drivers/scsi/aic7xxx/aic79xx_osm.c
> +++ b/drivers/scsi/aic7xxx/aic79xx_osm.c
> @@ -1324,10 +1324,9 @@ int
>  ahd_platform_alloc(struct ahd_softc *ahd, void *platform_arg)
>  {
>         ahd->platform_data =
> -           kmalloc(sizeof(struct ahd_platform_data), GFP_ATOMIC);
> +           kzalloc(sizeof(struct ahd_platform_data), GFP_ATOMIC);
>         if (ahd->platform_data == NULL)
>                 return (ENOMEM);
> -       memset(ahd->platform_data, 0, sizeof(struct ahd_platform_data));
>         ahd->platform_data->irq = AHD_LINUX_NOIRQ;
>         ahd_lockinit(ahd);
>         ahd->seltime = (aic79xx_seltime & 0x3) << 4;
> diff --git a/drivers/scsi/aic7xxx/aic7xxx_osm.c b/drivers/scsi/aic7xxx/aic7xxx_osm.c
> index 5a477cd..54bb1de 100644
> --- a/drivers/scsi/aic7xxx/aic7xxx_osm.c
> +++ b/drivers/scsi/aic7xxx/aic7xxx_osm.c
> @@ -1222,10 +1222,9 @@ ahc_platform_alloc(struct ahc_softc *ahc, void *platform_arg)
>  {
>
>         ahc->platform_data =
> -           kmalloc(sizeof(struct ahc_platform_data), GFP_ATOMIC);
> +           kzalloc(sizeof(struct ahc_platform_data), GFP_ATOMIC);
>         if (ahc->platform_data == NULL)
>                 return (ENOMEM);
> -       memset(ahc->platform_data, 0, sizeof(struct ahc_platform_data));
>         ahc->platform_data->irq = AHC_LINUX_NOIRQ;
>         ahc_lockinit(ahc);
>         ahc->seltime = (aic7xxx_seltime & 0x3) << 4;
> --
> 1.7.9.5
>

Hello,

Would anyone be so kind as to review this?

Thanks,
Silviu Popescu
--
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/