usb: Convert pr_warning to pr_warn

Use the more common logging mechanism.

Miscellanea:

o Realign multiline statements
o Coalesce format

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/usb/gadget/function/rndis.c     |  9 ++++-----
 drivers/usb/gadget/function/u_serial.c  |  4 ++--
 drivers/usb/gadget/udc/at91_udc.h       |  2 +-
 drivers/usb/gadget/udc/atmel_usba_udc.c |  4 ++--
 drivers/usb/gadget/udc/fsl_usb2_udc.h   |  2 +-
 drivers/usb/gadget/udc/m66592-udc.c     |  4 ++--
 drivers/usb/gadget/udc/omap_udc.h       |  2 +-
 drivers/usb/gadget/udc/pxa25x_udc.h     |  2 +-
 drivers/usb/host/isp1362-hcd.c          | 27 ++++++++++++++-------------
 drivers/usb/isp1760/isp1760-if.c        |  2 +-
 10 files changed, 29 insertions(+), 29 deletions(-)

Joe Perches <joe@perches.com> writes:

> Use the more common logging mechanism.
>
> Miscellanea:
>
> o Realign multiline statements
> o Coalesce format
>
> Signed-off-by: Joe Perches <joe@perches.com>
For pxa25x_udc.h:
Acked-by: Robert Jarzmik <robert.jarzmik@free.fr>

Cheers.

--
Robert

Le 27/09/2016 à 18:16, Joe Perches a écrit :
> Use the more common logging mechanism.
> 
> Miscellanea:
> 
> o Realign multiline statements
> o Coalesce format
> 
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
>  drivers/usb/gadget/function/rndis.c     |  9 ++++-----
>  drivers/usb/gadget/function/u_serial.c  |  4 ++--
>  drivers/usb/gadget/udc/at91_udc.h       |  2 +-
>  drivers/usb/gadget/udc/atmel_usba_udc.c |  4 ++--

For AT91 and Atmel parts:
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Thanks

>  drivers/usb/gadget/udc/fsl_usb2_udc.h   |  2 +-
>  drivers/usb/gadget/udc/m66592-udc.c     |  4 ++--
>  drivers/usb/gadget/udc/omap_udc.h       |  2 +-
>  drivers/usb/gadget/udc/pxa25x_udc.h     |  2 +-
>  drivers/usb/host/isp1362-hcd.c          | 27 ++++++++++++++-------------
>  drivers/usb/isp1760/isp1760-if.c        |  2 +-
>  10 files changed, 29 insertions(+), 29 deletions(-)
> 
> diff --git a/drivers/usb/gadget/function/rndis.c b/drivers/usb/gadget/function/rndis.c
> index ab6ac1b74ac0..766c328c15c0 100644
> --- a/drivers/usb/gadget/function/rndis.c
> +++ b/drivers/usb/gadget/function/rndis.c
> @@ -474,8 +474,7 @@ static int gen_ndis_query_resp(struct rndis_params *params, u32 OID, u8 *buf,
>  		break;
>  
>  	default:
> -		pr_warning("%s: query unknown OID 0x%08X\n",
> -			 __func__, OID);
> +		pr_warn("%s: query unknown OID 0x%08X\n", __func__, OID);
>  	}
>  	if (retval < 0)
>  		length = 0;
> @@ -546,8 +545,8 @@ static int gen_ndis_set_resp(struct rndis_params *params, u32 OID,
>  		break;
>  
>  	default:
> -		pr_warning("%s: set unknown OID 0x%08X, size %d\n",
> -			 __func__, OID, buf_len);
> +		pr_warn("%s: set unknown OID 0x%08X, size %d\n",
> +			__func__, OID, buf_len);
>  	}
>  
>  	return retval;
> @@ -854,7 +853,7 @@ int rndis_msg_parser(struct rndis_params *params, u8 *buf)
>  		 * In one case those messages seemed to relate to the host
>  		 * suspending itself.
>  		 */
> -		pr_warning("%s: unknown RNDIS message 0x%08X len %d\n",
> +		pr_warn("%s: unknown RNDIS message 0x%08X len %d\n",
>  			__func__, MsgType, MsgLength);
>  		print_hex_dump_bytes(__func__, DUMP_PREFIX_OFFSET,
>  				     buf, MsgLength);
> diff --git a/drivers/usb/gadget/function/u_serial.c b/drivers/usb/gadget/function/u_serial.c
> index e0cd1e4c8892..62ec842874aa 100644
> --- a/drivers/usb/gadget/function/u_serial.c
> +++ b/drivers/usb/gadget/function/u_serial.c
> @@ -622,8 +622,8 @@ static void gs_write_complete(struct usb_ep *ep, struct usb_request *req)
>  	switch (req->status) {
>  	default:
>  		/* presumably a transient fault */
> -		pr_warning("%s: unexpected %s status %d\n",
> -				__func__, ep->name, req->status);
> +		pr_warn("%s: unexpected %s status %d\n",
> +			__func__, ep->name, req->status);
>  		/* FALL THROUGH */
>  	case 0:
>  		/* normal completion */
> diff --git a/drivers/usb/gadget/udc/at91_udc.h b/drivers/usb/gadget/udc/at91_udc.h
> index 0a433e6b346b..9bbe72764f31 100644
> --- a/drivers/usb/gadget/udc/at91_udc.h
> +++ b/drivers/usb/gadget/udc/at91_udc.h
> @@ -175,7 +175,7 @@ struct at91_request {
>  #endif
>  
>  #define ERR(stuff...)		pr_err("udc: " stuff)
> -#define WARNING(stuff...)	pr_warning("udc: " stuff)
> +#define WARNING(stuff...)	pr_warn("udc: " stuff)
>  #define INFO(stuff...)		pr_info("udc: " stuff)
>  #define DBG(stuff...)		pr_debug("udc: " stuff)
>  
> diff --git a/drivers/usb/gadget/udc/atmel_usba_udc.c b/drivers/usb/gadget/udc/atmel_usba_udc.c
> index 45bc997d0711..1ef7a9a9d7f5 100644
> --- a/drivers/usb/gadget/udc/atmel_usba_udc.c
> +++ b/drivers/usb/gadget/udc/atmel_usba_udc.c
> @@ -1464,8 +1464,8 @@ static void usba_control_irq(struct usba_udc *udc, struct usba_ep *ep)
>  		pkt_len = USBA_BFEXT(BYTE_COUNT, usba_ep_readl(ep, STA));
>  		DBG(DBG_HW, "Packet length: %u\n", pkt_len);
>  		if (pkt_len != sizeof(crq)) {
> -			pr_warning("udc: Invalid packet length %u "
> -				"(expected %zu)\n", pkt_len, sizeof(crq));
> +			pr_warn("udc: Invalid packet length %u (expected %zu)\n",
> +				pkt_len, sizeof(crq));
>  			set_protocol_stall(udc, ep);
>  			return;
>  		}
> diff --git a/drivers/usb/gadget/udc/fsl_usb2_udc.h b/drivers/usb/gadget/udc/fsl_usb2_udc.h
> index 84715625b2b3..e92b8408b6f6 100644
> --- a/drivers/usb/gadget/udc/fsl_usb2_udc.h
> +++ b/drivers/usb/gadget/udc/fsl_usb2_udc.h
> @@ -554,7 +554,7 @@ static void dump_msg(const char *label, const u8 * buf, unsigned int length)
>  #endif
>  
>  #define ERR(stuff...)		pr_err("udc: " stuff)
> -#define WARNING(stuff...)		pr_warning("udc: " stuff)
> +#define WARNING(stuff...)	pr_warn("udc: " stuff)
>  #define INFO(stuff...)		pr_info("udc: " stuff)
>  
>  /*-------------------------------------------------------------------------*/
> diff --git a/drivers/usb/gadget/udc/m66592-udc.c b/drivers/usb/gadget/udc/m66592-udc.c
> index 6e977dc22570..de3e03483659 100644
> --- a/drivers/usb/gadget/udc/m66592-udc.c
> +++ b/drivers/usb/gadget/udc/m66592-udc.c
> @@ -637,7 +637,7 @@ static void init_controller(struct m66592 *m66592)
>  			clock = M66592_XTAL48;
>  			break;
>  		default:
> -			pr_warning("m66592-udc: xtal configuration error\n");
> +			pr_warn("m66592-udc: xtal configuration error\n");
>  			clock = 0;
>  		}
>  
> @@ -649,7 +649,7 @@ static void init_controller(struct m66592 *m66592)
>  			irq_sense = 0;
>  			break;
>  		default:
> -			pr_warning("m66592-udc: irq trigger config error\n");
> +			pr_warn("m66592-udc: irq trigger config error\n");
>  			irq_sense = 0;
>  		}
>  
> diff --git a/drivers/usb/gadget/udc/omap_udc.h b/drivers/usb/gadget/udc/omap_udc.h
> index cfadeb5fc5de..26974196cf44 100644
> --- a/drivers/usb/gadget/udc/omap_udc.h
> +++ b/drivers/usb/gadget/udc/omap_udc.h
> @@ -187,7 +187,7 @@ struct omap_udc {
>  #endif
>  
>  #define ERR(stuff...)		pr_err("udc: " stuff)
> -#define WARNING(stuff...)	pr_warning("udc: " stuff)
> +#define WARNING(stuff...)	pr_warn("udc: " stuff)
>  #define INFO(stuff...)		pr_info("udc: " stuff)
>  #define DBG(stuff...)		pr_debug("udc: " stuff)
>  
> diff --git a/drivers/usb/gadget/udc/pxa25x_udc.h b/drivers/usb/gadget/udc/pxa25x_udc.h
> index 4b8b72d7ab37..a458bec2536d 100644
> --- a/drivers/usb/gadget/udc/pxa25x_udc.h
> +++ b/drivers/usb/gadget/udc/pxa25x_udc.h
> @@ -248,7 +248,7 @@ dump_state(struct pxa25x_udc *dev)
>  #define DBG(lvl, stuff...) do{if ((lvl) <= UDC_DEBUG) DMSG(stuff);}while(0)
>  
>  #define ERR(stuff...)		pr_err("udc: " stuff)
> -#define WARNING(stuff...)	pr_warning("udc: " stuff)
> +#define WARNING(stuff...)	pr_warn("udc: " stuff)
>  #define INFO(stuff...)		pr_info("udc: " stuff)
>  
>  
> diff --git a/drivers/usb/host/isp1362-hcd.c b/drivers/usb/host/isp1362-hcd.c
> index 6cf82ee460a6..0f2b4b358e1a 100644
> --- a/drivers/usb/host/isp1362-hcd.c
> +++ b/drivers/usb/host/isp1362-hcd.c
> @@ -147,7 +147,7 @@ static inline struct isp1362_ep_queue *get_ptd_queue(struct isp1362_hcd *isp1362
>  	if (epq)
>  		DBG(1, "%s: PTD $%04x is on %s queue\n", __func__, offset, epq->name);
>  	else
> -		pr_warning("%s: invalid PTD $%04x\n", __func__, offset);
> +		pr_warn("%s: invalid PTD $%04x\n", __func__, offset);
>  
>  	return epq;
>  }
> @@ -157,8 +157,9 @@ static inline int get_ptd_offset(struct isp1362_ep_queue *epq, u8 index)
>  	int offset;
>  
>  	if (index * epq->blk_size > epq->buf_size) {
> -		pr_warning("%s: Bad %s index %d(%d)\n", __func__, epq->name, index,
> -		     epq->buf_size / epq->blk_size);
> +		pr_warn("%s: Bad %s index %d(%d)\n",
> +			__func__, epq->name, index,
> +			epq->buf_size / epq->blk_size);
>  		return -EINVAL;
>  	}
>  	offset = epq->buf_start + index * epq->blk_size;
> @@ -902,8 +903,8 @@ static void start_iso_transfers(struct isp1362_hcd *isp1362_hcd)
>  
>  			ptd_offset = next_ptd(epq, ep);
>  			if (ptd_offset < 0) {
> -				pr_warning("%s: req %d No more %s PTD buffers available\n", __func__,
> -				     ep->num_req, epq->name);
> +				pr_warn("%s: req %d No more %s PTD buffers available\n",
> +					__func__, ep->num_req, epq->name);
>  				break;
>  			}
>  		}
> @@ -973,8 +974,8 @@ static void finish_transfers(struct isp1362_hcd *isp1362_hcd, unsigned long done
>  			break;
>  	}
>  	if (done_map)
> -		pr_warning("%s: done_map not clear: %08lx:%08lx\n", __func__, done_map,
> -		     epq->skip_map);
> +		pr_warn("%s: done_map not clear: %08lx:%08lx\n",
> +			__func__, done_map, epq->skip_map);
>  	atomic_dec(&epq->finishing);
>  }
>  
> @@ -1433,7 +1434,7 @@ static int isp1362_urb_dequeue(struct usb_hcd *hcd, struct urb *urb, int status)
>  		} else
>  			DBG(1, "%s: urb %p active; wait4irq\n", __func__, urb);
>  	} else {
> -		pr_warning("%s: No EP in URB %p\n", __func__, urb);
> +		pr_warn("%s: No EP in URB %p\n", __func__, urb);
>  		retval = -EINVAL;
>  	}
>  done:
> @@ -1748,10 +1749,10 @@ static int isp1362_bus_suspend(struct usb_hcd *hcd)
>  		/* FALL THROUGH */
>  	case OHCI_USB_RESET:
>  		status = -EBUSY;
> -		pr_warning("%s: needs reinit!\n", __func__);
> +		pr_warn("%s: needs reinit!\n", __func__);
>  		goto done;
>  	case OHCI_USB_SUSPEND:
> -		pr_warning("%s: already suspended?\n", __func__);
> +		pr_warn("%s: already suspended?\n", __func__);
>  		goto done;
>  	}
>  	DBG(0, "%s: suspend root hub\n", __func__);
> @@ -1839,7 +1840,7 @@ static int isp1362_bus_resume(struct usb_hcd *hcd)
>  	isp1362_hcd->hc_control = isp1362_read_reg32(isp1362_hcd, HCCONTROL);
>  	pr_info("%s: HCCONTROL: %08x\n", __func__, isp1362_hcd->hc_control);
>  	if (hcd->state == HC_STATE_RESUMING) {
> -		pr_warning("%s: duplicate resume\n", __func__);
> +		pr_warn("%s: duplicate resume\n", __func__);
>  		status = 0;
>  	} else
>  		switch (isp1362_hcd->hc_control & OHCI_CTRL_HCFS) {
> @@ -2474,8 +2475,8 @@ static int isp1362_chip_test(struct isp1362_hcd *isp1362_hcd)
>  					    __func__, offset);
>  					break;
>  				}
> -				pr_warning("%s: memory check with offset %02x ok after second read\n",
> -				     __func__, offset);
> +				pr_warn("%s: memory check with offset %02x ok after second read\n",
> +					__func__, offset);
>  			}
>  		}
>  		kfree(ref);
> diff --git a/drivers/usb/isp1760/isp1760-if.c b/drivers/usb/isp1760/isp1760-if.c
> index 9535b2872183..79205b31e4a9 100644
> --- a/drivers/usb/isp1760/isp1760-if.c
> +++ b/drivers/usb/isp1760/isp1760-if.c
> @@ -197,7 +197,7 @@ static int isp1760_plat_probe(struct platform_device *pdev)
>  
>  	irq_res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
>  	if (!irq_res) {
> -		pr_warning("isp1760: IRQ resource not available\n");
> +		pr_warn("isp1760: IRQ resource not available\n");
>  		return -ENODEV;
>  	}
>  	irqflags = irq_res->flags & IRQF_TRIGGER_MASK;
>