Re: [PATCH 4/4] usb/gadget: Add driver for Aspeed SoC virtual hub

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2017-06-26 at 13:12 +0800, kbuild test robot wrote:
> Hi Benjamin,
> 
> [auto build test WARNING on robh/for-next]
> [also build test WARNING on v4.12-rc6]
> [cannot apply to balbi-usb/next next-20170623]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

Hrm... I wouldn't worry about these as this IP only exists on 32-bit
SoC's, but if you're adamant then I'll look into putting the gory
horrible format strings in a respin...

> url:    https://github.com/0day-ci/linux/commits/Benjamin-Herrenschmidt/usb-Add-Aspeed-SoC-USB-controllers-to-device-tree/20170626-124044
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
> config: tile-allyesconfig (attached as .config)
> compiler: tilegx-linux-gcc (GCC) 4.6.2
> reproduce:
>         wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=tile 
> 
> All warnings (new ones prefixed by >>):
> 
>    drivers/usb/gadget/udc/aspeed-vhub/ep0.c: In function 'ast_vhub_ep0_do_send':
> > > drivers/usb/gadget/udc/aspeed-vhub/ep0.c:196:2: warning: format '%d' expects argument of type 'int', but argument 6 has type 'size_t' [-Wformat]
> 
>    drivers/usb/gadget/udc/aspeed-vhub/ep0.c: In function 'ast_vhub_ep0_do_receive':
>    drivers/usb/gadget/udc/aspeed-vhub/ep0.c:230:2: warning: format '%d' expects argument of type 'int', but argument 6 has type 'size_t' [-Wformat]
>    drivers/usb/gadget/udc/aspeed-vhub/ep0.c:230:2: warning: format '%d' expects argument of type 'int', but argument 7 has type 'size_t' [-Wformat]
>    drivers/usb/gadget/udc/aspeed-vhub/ep0.c:234:3: warning: format '%d' expects argument of type 'int', but argument 6 has type 'size_t' [-Wformat]
> --
>    drivers/usb/gadget/udc/aspeed-vhub/epn.c: In function 'ast_vhub_epn_kick_desc':
> > > drivers/usb/gadget/udc/aspeed-vhub/epn.c:209:14: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
> 
>    drivers/usb/gadget/udc/aspeed-vhub/epn.c: In function 'ast_vhub_epn_queue':
> > > drivers/usb/gadget/udc/aspeed-vhub/epn.c:378:2: warning: format '%x' expects argument of type 'unsigned int', but argument 7 has type 'dma_addr_t' [-Wformat]
> 
> vim +196 drivers/usb/gadget/udc/aspeed-vhub/ep0.c
> 
>    180			ep->ep0.state = ep0_state_status;
>    181			writel(VHUB_EP0_RX_BUFF_RDY, ep->ep0.ctlstat);
>    182			ast_vhub_done(ep, req, 0);
>    183			return;
>    184		}
>    185	
>    186		/*
>    187		 * Next chunk cropped to max packet size. Also check if this
>    188		 * is the last packet
>    189		 */
>    190		chunk = req->req.length - req->req.actual;
>    191		if (chunk > ep->ep.maxpacket)
>    192			chunk = ep->ep.maxpacket;
>    193		else if ((chunk < ep->ep.maxpacket) || !req->req.zero)
>    194			req->last_desc = 1;
>    195	
>  > 196		EPVDBG(ep, "send chunk=%d last=%d, req->act=%d mp=%d\n",
>    197		       chunk, req->last_desc, req->req.actual, ep->ep.maxpacket);
>    198	
>    199		/*
>    200		 * Copy data if any (internal requests already have data
>    201		 * in the EP buffer)
>    202		 */
>    203		if (chunk && req->req.buf)
>    204			memcpy(ep->buf, req->req.buf + req->req.actual, chunk);
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux