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-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
