Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-18 10:54:37 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

usb: Add nonblock argument to submit_int_msg

Browse source 
This will be used to implement non-blocking keyboard polling in case of
errors.

Signed-off-by: Michal Suchanek <msuchanek@suse.de>
This commit is contained in:
Michal Suchanek 2019-08-18 10:55:27 +02:00 • committed by marex
parent 50dce8fbf0
commit 3437121c03
14 changed files with 60 additions and 40 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
common/usb.c
View file

@ -192,12 +192,15 @@ int usb_disable_asynch(int disable)
*/ */
/* /*
* submits an Interrupt Message * submits an Interrupt Message. Some drivers may implement non-blocking
* polling: when non-block is true and the device is not responding return
* -EAGAIN instead of waiting for device to respond.
*/ */
int usb_int_msg(struct usb_device *dev, unsigned long pipe, int usb_int_msg(struct usb_device *dev, unsigned long pipe,
void *buffer, int transfer_len, int interval) void *buffer, int transfer_len, int interval, bool nonblock)
{ {
return submit_int_msg(dev, pipe, buffer, transfer_len, interval); return submit_int_msg(dev, pipe, buffer, transfer_len, interval,
nonblock);
} }
/* /*

4
common/usb_kbd.c
View file

@ -341,7 +341,7 @@ static inline void usb_kbd_poll_for_event(struct usb_device *dev)
/* Submit a interrupt transfer request */ /* Submit a interrupt transfer request */
if (usb_int_msg(dev, data->intpipe, &data->new[0], if (usb_int_msg(dev, data->intpipe, &data->new[0],
data->intpktsize, data->intinterval) >= 0) data->intpktsize, data->intinterval, true) >= 0)
usb_kbd_irq_worker(dev); usb_kbd_irq_worker(dev);
#elif defined(CONFIG_SYS_USB_EVENT_POLL_VIA_CONTROL_EP) || \ #elif defined(CONFIG_SYS_USB_EVENT_POLL_VIA_CONTROL_EP) || \
defined(CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE) defined(CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE)
@ -505,7 +505,7 @@ static int usb_kbd_probe_dev(struct usb_device *dev, unsigned int ifnum)
1, 0, data->new, USB_KBD_BOOT_REPORT_SIZE) < 0) { 1, 0, data->new, USB_KBD_BOOT_REPORT_SIZE) < 0) {
#else #else
if (usb_int_msg(dev, data->intpipe, data->new, data->intpktsize, if (usb_int_msg(dev, data->intpipe, data->new, data->intpktsize,
data->intinterval) < 0) { data->intinterval, false) < 0) {
#endif #endif
printf("Failed to get keyboard state from device %04x:%04x\n", printf("Failed to get keyboard state from device %04x:%04x\n",
dev->descriptor.idVendor, dev->descriptor.idProduct); dev->descriptor.idVendor, dev->descriptor.idProduct);

2
common/usb_storage.c
View file

@ -651,7 +651,7 @@ static int usb_stor_CBI_get_status(struct scsi_cmd *srb, struct us_data *us)
us->ip_wanted = 1; us->ip_wanted = 1;
usb_int_msg(us->pusb_dev, us->irqpipe, usb_int_msg(us->pusb_dev, us->irqpipe,
(void *)&us->ip_data, us->irqmaxp, us->irqinterval); (void *)&us->ip_data, us->irqmaxp, us->irqinterval, false);
timeout = 1000; timeout = 1000;
while (timeout--) { while (timeout--) {
if (us->ip_wanted == 0) if (us->ip_wanted == 0)

3
drivers/usb/emul/sandbox_keyb.c
View file

@ -178,7 +178,8 @@ static int sandbox_keyb_control(struct udevice *dev, struct usb_device *udev,
} }
static int sandbox_keyb_interrupt(struct udevice *dev, struct usb_device *udev, static int sandbox_keyb_interrupt(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, int interval) unsigned long pipe, void *buffer, int length, int interval,
bool nonblock)
{ {
struct sandbox_keyb_priv *priv = dev_get_priv(dev); struct sandbox_keyb_priv *priv = dev_get_priv(dev);
uint8_t *data = buffer; uint8_t *data = buffer;

6
drivers/usb/emul/usb-emul-uclass.c
View file

@ -245,7 +245,8 @@ int usb_emul_bulk(struct udevice *emul, struct usb_device *udev,
} }
int usb_emul_int(struct udevice *emul, struct usb_device *udev, int usb_emul_int(struct udevice *emul, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, int interval) unsigned long pipe, void *buffer, int length, int interval,
bool nonblock)
{ {
struct dm_usb_ops *ops = usb_get_emul_ops(emul); struct dm_usb_ops *ops = usb_get_emul_ops(emul);
@ -253,7 +254,8 @@ int usb_emul_int(struct udevice *emul, struct usb_device *udev,
return -ENOSYS; return -ENOSYS;
debug("%s: dev=%s\n", __func__, emul->name); debug("%s: dev=%s\n", __func__, emul->name);
return ops->interrupt(emul, udev, pipe, buffer, length, interval); return ops->interrupt(emul, udev, pipe, buffer, length, interval,
nonblock);
} }
int usb_emul_setup_device(struct udevice *dev, struct usb_string *strings, int usb_emul_setup_device(struct udevice *dev, struct usb_string *strings,

13
drivers/usb/host/dwc2.c
View file

@ -1108,7 +1108,8 @@ static int _submit_control_msg(struct dwc2_priv *priv, struct usb_device *dev,
} }
int _submit_int_msg(struct dwc2_priv *priv, struct usb_device *dev, int _submit_int_msg(struct dwc2_priv *priv, struct usb_device *dev,
unsigned long pipe, void *buffer, int len, int interval) unsigned long pipe, void *buffer, int len, int interval,
bool nonblock)
{ {
unsigned long timeout; unsigned long timeout;
int ret; int ret;
@ -1236,9 +1237,10 @@ int submit_bulk_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
} }
int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
int len, int interval) int len, int interval, bool nonblock)
{ {
return _submit_int_msg(&local, dev, pipe, buffer, len, interval); return _submit_int_msg(&local, dev, pipe, buffer, len, interval,
nonblock);
} }
/* U-Boot USB control interface */ /* U-Boot USB control interface */
@ -1292,13 +1294,14 @@ static int dwc2_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
static int dwc2_submit_int_msg(struct udevice *dev, struct usb_device *udev, static int dwc2_submit_int_msg(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
struct dwc2_priv *priv = dev_get_priv(dev); struct dwc2_priv *priv = dev_get_priv(dev);
debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev); debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev);
return _submit_int_msg(priv, udev, pipe, buffer, length, interval); return _submit_int_msg(priv, udev, pipe, buffer, length, interval,
nonblock);
} }
static int dwc2_usb_ofdata_to_platdata(struct udevice *dev) static int dwc2_usb_ofdata_to_platdata(struct udevice *dev)

13
drivers/usb/host/ehci-hcd.c
View file

@ -1482,7 +1482,8 @@ out:
} }
static int _ehci_submit_int_msg(struct usb_device *dev, unsigned long pipe, static int _ehci_submit_int_msg(struct usb_device *dev, unsigned long pipe,
void *buffer, int length, int interval) void *buffer, int length, int interval,
bool nonblock)
{ {
void *backbuffer; void *backbuffer;
struct int_queue *queue; struct int_queue *queue;
@ -1532,9 +1533,10 @@ int submit_control_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
} }
int submit_int_msg(struct usb_device *dev, unsigned long pipe, int submit_int_msg(struct usb_device *dev, unsigned long pipe,
void *buffer, int length, int interval) void *buffer, int length, int interval, bool nonblock)
{ {
return _ehci_submit_int_msg(dev, pipe, buffer, length, interval); return _ehci_submit_int_msg(dev, pipe, buffer, length, interval,
nonblock);
} }
struct int_queue *create_int_queue(struct usb_device *dev, struct int_queue *create_int_queue(struct usb_device *dev,
@ -1576,10 +1578,11 @@ static int ehci_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
static int ehci_submit_int_msg(struct udevice *dev, struct usb_device *udev, static int ehci_submit_int_msg(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev); debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev);
return _ehci_submit_int_msg(udev, pipe, buffer, length, interval); return _ehci_submit_int_msg(udev, pipe, buffer, length, interval,
nonblock);
} }
static struct int_queue *ehci_create_int_queue(struct udevice *dev, static struct int_queue *ehci_create_int_queue(struct udevice *dev,

4
drivers/usb/host/ohci-hcd.c
View file

@ -1703,7 +1703,7 @@ int submit_bulk_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
} }
int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
int transfer_len, int interval) int transfer_len, int interval, bool nonblock)
{ {
info("submit_int_msg"); info("submit_int_msg");
return submit_common_msg(&gohci, dev, pipe, buffer, transfer_len, NULL, return submit_common_msg(&gohci, dev, pipe, buffer, transfer_len, NULL,
@ -2155,7 +2155,7 @@ static int ohci_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
static int ohci_submit_int_msg(struct udevice *dev, struct usb_device *udev, static int ohci_submit_int_msg(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
ohci_t *ohci = dev_get_priv(usb_get_bus(dev)); ohci_t *ohci = dev_get_priv(usb_get_bus(dev));

2
drivers/usb/host/sl811-hcd.c
View file

@ -384,7 +384,7 @@ int submit_control_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
} }
int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
int len, int interval) int len, int interval, bool nonblock)
{ {
PDEBUG(0, "dev = %p pipe = %#lx buf = %p size = %d int = %d\n", dev, pipe, PDEBUG(0, "dev = %p pipe = %#lx buf = %p size = %d int = %d\n", dev, pipe,
buffer, len, interval); buffer, len, interval);

5
drivers/usb/host/usb-sandbox.c
View file

@ -99,7 +99,7 @@ static int sandbox_submit_bulk(struct udevice *bus, struct usb_device *udev,
static int sandbox_submit_int(struct udevice *bus, struct usb_device *udev, static int sandbox_submit_int(struct udevice *bus, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
struct udevice *emul; struct udevice *emul;
int ret; int ret;
@ -110,7 +110,8 @@ static int sandbox_submit_int(struct udevice *bus, struct usb_device *udev,
usbmon_trace(bus, pipe, NULL, emul); usbmon_trace(bus, pipe, NULL, emul);
if (ret) if (ret)
return ret; return ret;
ret = usb_emul_int(emul, udev, pipe, buffer, length, interval); ret = usb_emul_int(emul, udev, pipe, buffer, length, interval,
nonblock);
return ret; return ret;
} }

5
drivers/usb/host/usb-uclass.c
View file

@ -31,7 +31,7 @@ int usb_disable_asynch(int disable)
} }
int submit_int_msg(struct usb_device *udev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *udev, unsigned long pipe, void *buffer,
int length, int interval) int length, int interval, bool nonblock)
{ {
struct udevice *bus = udev->controller_dev; struct udevice *bus = udev->controller_dev;
struct dm_usb_ops *ops = usb_get_ops(bus); struct dm_usb_ops *ops = usb_get_ops(bus);
@ -39,7 +39,8 @@ int submit_int_msg(struct usb_device *udev, unsigned long pipe, void *buffer,
if (!ops->interrupt) if (!ops->interrupt)
return -ENOSYS; return -ENOSYS;
return ops->interrupt(bus, udev, pipe, buffer, length, interval); return ops->interrupt(bus, udev, pipe, buffer, length, interval,
nonblock);
} }
int submit_control_msg(struct usb_device *udev, unsigned long pipe, int submit_control_msg(struct usb_device *udev, unsigned long pipe,

13
drivers/usb/host/xhci.c
View file

@ -1109,7 +1109,8 @@ unknown:
* @return 0 * @return 0
*/ */
static int _xhci_submit_int_msg(struct usb_device *udev, unsigned long pipe, static int _xhci_submit_int_msg(struct usb_device *udev, unsigned long pipe,
void *buffer, int length, int interval) void *buffer, int length, int interval,
bool nonblock)
{ {
if (usb_pipetype(pipe) != PIPE_INTERRUPT) { if (usb_pipetype(pipe) != PIPE_INTERRUPT) {
printf("non-interrupt pipe (type=%lu)", usb_pipetype(pipe)); printf("non-interrupt pipe (type=%lu)", usb_pipetype(pipe));
@ -1277,9 +1278,10 @@ int submit_bulk_msg(struct usb_device *udev, unsigned long pipe, void *buffer,
} }
int submit_int_msg(struct usb_device *udev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *udev, unsigned long pipe, void *buffer,
int length, int interval) int length, int interval, bool nonblock)
{ {
return _xhci_submit_int_msg(udev, pipe, buffer, length, interval); return _xhci_submit_int_msg(udev, pipe, buffer, length, interval,
nonblock);
} }
/** /**
@ -1386,10 +1388,11 @@ static int xhci_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
static int xhci_submit_int_msg(struct udevice *dev, struct usb_device *udev, static int xhci_submit_int_msg(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev); debug("%s: dev='%s', udev=%p\n", __func__, dev->name, udev);
return _xhci_submit_int_msg(udev, pipe, buffer, length, interval); return _xhci_submit_int_msg(udev, pipe, buffer, length, interval,
nonblock);
} }
static int xhci_alloc_device(struct udevice *dev, struct usb_device *udev) static int xhci_alloc_device(struct udevice *dev, struct usb_device *udev)

12
drivers/usb/musb-new/musb_uboot.c
View file

@ -110,7 +110,7 @@ static int _musb_submit_bulk_msg(struct musb_host_data *host,
static int _musb_submit_int_msg(struct musb_host_data *host, static int _musb_submit_int_msg(struct musb_host_data *host,
struct usb_device *dev, unsigned long pipe, struct usb_device *dev, unsigned long pipe,
void *buffer, int len, int interval) void *buffer, int len, int interval, bool nonblock)
{ {
construct_urb(&host->urb, &host->hep, dev, USB_ENDPOINT_XFER_INT, pipe, construct_urb(&host->urb, &host->hep, dev, USB_ENDPOINT_XFER_INT, pipe,
buffer, len, NULL, interval); buffer, len, NULL, interval);
@ -268,9 +268,10 @@ int submit_control_msg(struct usb_device *dev, unsigned long pipe,
} }
int submit_int_msg(struct usb_device *dev, unsigned long pipe, int submit_int_msg(struct usb_device *dev, unsigned long pipe,
void *buffer, int length, int interval) void *buffer, int length, int interval, bool nonblock)
{ {
return _musb_submit_int_msg(&musb_host, dev, pipe, buffer, length, interval); return _musb_submit_int_msg(&musb_host, dev, pipe, buffer, length,
interval, nonblock);
} }
struct int_queue *create_int_queue(struct usb_device *dev, struct int_queue *create_int_queue(struct usb_device *dev,
@ -320,10 +321,11 @@ static int musb_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
static int musb_submit_int_msg(struct udevice *dev, struct usb_device *udev, static int musb_submit_int_msg(struct udevice *dev, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval) int interval, bool nonblock)
{ {
struct musb_host_data *host = dev_get_priv(dev); struct musb_host_data *host = dev_get_priv(dev);
return _musb_submit_int_msg(host, udev, pipe, buffer, length, interval); return _musb_submit_int_msg(host, udev, pipe, buffer, length, interval,
nonblock);
} }
static struct int_queue *musb_create_int_queue(struct udevice *dev, static struct int_queue *musb_create_int_queue(struct udevice *dev,

9
include/usb.h
View file

@ -184,7 +184,7 @@ int submit_bulk_msg(struct usb_device *dev, unsigned long pipe,
int submit_control_msg(struct usb_device *dev, unsigned long pipe, void *buffer, int submit_control_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
int transfer_len, struct devrequest *setup); int transfer_len, struct devrequest *setup);
int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer, int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer,
int transfer_len, int interval); int transfer_len, int interval, bool nonblock);
#if defined CONFIG_USB_EHCI_HCD || defined CONFIG_USB_MUSB_HOST \ #if defined CONFIG_USB_EHCI_HCD || defined CONFIG_USB_MUSB_HOST \
|| CONFIG_IS_ENABLED(DM_USB) || CONFIG_IS_ENABLED(DM_USB)
@ -262,7 +262,7 @@ int usb_control_msg(struct usb_device *dev, unsigned int pipe,
int usb_bulk_msg(struct usb_device *dev, unsigned int pipe, int usb_bulk_msg(struct usb_device *dev, unsigned int pipe,
void *data, int len, int *actual_length, int timeout); void *data, int len, int *actual_length, int timeout);
int usb_int_msg(struct usb_device *dev, unsigned long pipe, int usb_int_msg(struct usb_device *dev, unsigned long pipe,
void *buffer, int transfer_len, int interval); void *buffer, int transfer_len, int interval, bool nonblock);
int usb_disable_asynch(int disable); int usb_disable_asynch(int disable);
int usb_maxpacket(struct usb_device *dev, unsigned long pipe); int usb_maxpacket(struct usb_device *dev, unsigned long pipe);
int usb_get_configuration_no(struct usb_device *dev, int cfgno, int usb_get_configuration_no(struct usb_device *dev, int cfgno,
@ -708,7 +708,7 @@ struct dm_usb_ops {
*/ */
int (*interrupt)(struct udevice *bus, struct usb_device *udev, int (*interrupt)(struct udevice *bus, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, unsigned long pipe, void *buffer, int length,
int interval); int interval, bool nonblock);
/** /**
* create_int_queue() - Create and queue interrupt packets * create_int_queue() - Create and queue interrupt packets
@ -1029,7 +1029,8 @@ int usb_emul_bulk(struct udevice *emul, struct usb_device *udev,
* @return 0 if OK, -ve on error * @return 0 if OK, -ve on error
*/ */
int usb_emul_int(struct udevice *emul, struct usb_device *udev, int usb_emul_int(struct udevice *emul, struct usb_device *udev,
unsigned long pipe, void *buffer, int length, int interval); unsigned long pipe, void *buffer, int length, int interval,
bool nonblock);
/** /**
* usb_emul_find() - Find an emulator for a particular device * usb_emul_find() - Find an emulator for a particular device