Re: [Outreachy kernel] [PATCH] Staging: emxx_udc: Remove argument test from function

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, Mar 02, 2015 at 12:18:13AM +0300, Haneen Mohammed wrote:
> This patch removes the test statement for an argument to  _nbu2ss_pullup function,
> for it can't be null due to previous derefrences when its called.
> 
> Signed-off-by: Haneen Mohammed <hamohammed.sa@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>  drivers/staging/emxx_udc/emxx_udc.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c
> index 0bfd268..b2a89a6 100644
> --- a/drivers/staging/emxx_udc/emxx_udc.c
> +++ b/drivers/staging/emxx_udc/emxx_udc.c
> @@ -2257,11 +2257,6 @@ static int _nbu2ss_pullup(struct nbu2ss_udc *udc, int is_on)
>  {
>  	u32	reg_dt;
>  
> -	if (!udc) {
> -		pr_err("udc: %s, bad param\n", __func__);
> -		return -EINVAL;
> -	}
> -
>  	if (udc->vbus_active == 0)
>  		return -ESHUTDOWN;
>  

Does not apply :(