On Mon, Sep 09, 2013 at 03:01:24PM -0400, Lidza Louina wrote:
> This patch removes casting around kzalloc calls.
> The casts aren't needed because kzalloc returns
> a void pointer.
> 
> Signed-off-by: Lidza Louina <lidza.lou...@gmail.com>
> ---
>  drivers/staging/dgnc/dgnc_driver.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/dgnc/dgnc_driver.c 
> b/drivers/staging/dgnc/dgnc_driver.c
> index 89c332f..3717d86 100644
> --- a/drivers/staging/dgnc/dgnc_driver.c
> +++ b/drivers/staging/dgnc/dgnc_driver.c
> @@ -499,7 +499,7 @@ static int dgnc_found_board(struct pci_dev *pdev, int id)
>  
>       /* get the board structure and prep it */
>       brd = dgnc_Board[dgnc_NumBoards] =
> -     (struct dgnc_board *) kzalloc(sizeof(struct dgnc_board), GFP_KERNEL);
> +     kzalloc(sizeof(struct dgnc_board), GFP_KERNEL);

Very odd indentation, this should be:
        brd = dgnc_Board[dgnc_NumBoards] = kzalloc(sizeof(struct dgnc_board),
                                                   GFP_KERNEL);

right?

I've gotten patches from 2 others for this same type of fix, it seems
people are going through the whole tree for this issue at the moment.

thanks,

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to