Re: [Outreachy kernel] [PATCH v5 4/6] Staging: emxx_udc: Replace custom printk macro ERR with dev_err

[Greg KH <gregkh@linuxfoundation.org>]

On Tue, Feb 24, 2015 at 05:36:18PM +0300, Haneen Mohammed wrote:
> This patch removes the other half of ERR macros that uses "printk(KERN_ERR"
> and replace them with dev_err, to fix issue addressed by checkpatch.pl.
> 
> Signed-off-by: Haneen Mohammed <hamohammed.sa@gmail.com>
> ---
> v5: Fix spelling mistakes
> 
>  drivers/staging/emxx_udc/emxx_udc.c | 34 +++++++++++++++++-----------------
>  1 file changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c
> index f33980c..ed1d044 100644
> --- a/drivers/staging/emxx_udc/emxx_udc.c
> +++ b/drivers/staging/emxx_udc/emxx_udc.c
> @@ -808,7 +808,7 @@ static int _nbu2ss_ep0_out_transfer(
>  		return 0;		/* Short Packet Transfer End */
>  
>  	if (req->req.actual > req->req.length) {
> -		ERR(" *** Overrun Error\n");
> +		dev_err(udc->dev, " *** Overrun Error\n");
>  		return -EOVERFLOW;
>  	}

As I didn't take your previous patches, this one doesn't apply.  Please
redo the whole series and resend.

thanks,

greg k-h