[PATCH 1/2] spi/s3c24xx: remove unnecessary memset of s3c24xx_spi
Daniel Kurtz
djkurtz at chromium.org
Tue Jun 30 21:25:44 PDT 2015
On Wed, Jul 1, 2015 at 2:03 AM, Alexey Klimov <klimov.linux at gmail.com> wrote:
> Memory for this struct is allocated by spi_alloc_master() using
> kzalloc() so it doesn't need to be set to 0 one more time.
>
> Signed-off-by: Alexey Klimov <klimov.linux at gmail.com>
Yup, I just noticed this too when reviewing the mediatek SPI driver.
For both patches:
Reviewed-by: Daniel Kurtz <djkurtz at chromium.org>
> ---
> drivers/spi/spi-s3c24xx.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/spi/spi-s3c24xx.c b/drivers/spi/spi-s3c24xx.c
> index f747ca2..f36bc32 100644
> --- a/drivers/spi/spi-s3c24xx.c
> +++ b/drivers/spi/spi-s3c24xx.c
> @@ -501,7 +501,6 @@ static int s3c24xx_spi_probe(struct platform_device *pdev)
> }
>
> hw = spi_master_get_devdata(master);
> - memset(hw, 0, sizeof(struct s3c24xx_spi));
>
> hw->master = master;
> hw->pdata = pdata = dev_get_platdata(&pdev->dev);
> --
> 1.9.1
>
>
>
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
More information about the Linux-rockchip
mailing list