diff options
| author | Johan Hovold <johan@kernel.org> | 2017-03-02 14:51:31 +0300 |
|---|---|---|
| committer | Johan Hovold <johan@kernel.org> | 2017-03-16 12:33:46 +0300 |
| commit | 590298b2232503ee5088c3abfe6cf69d51ade427 (patch) | |
| tree | 86344462537302a280d9c3b76ed338c9d8c28be3 | |
| parent | 32814c87f446b90efe4350bb97924eca53152d26 (diff) | |
| download | linux-590298b2232503ee5088c3abfe6cf69d51ade427.tar.xz | |
USB: serial: pl2303: simplify endpoint check
Simplify the endpoint sanity check by letting core verify that the
required endpoints are present.
Signed-off-by: Johan Hovold <johan@kernel.org>
| -rw-r--r-- | drivers/usb/serial/pl2303.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c index ca69eb42071b..60840004568a 100644 --- a/drivers/usb/serial/pl2303.c +++ b/drivers/usb/serial/pl2303.c @@ -221,17 +221,9 @@ static int pl2303_probe(struct usb_serial *serial, static int pl2303_startup(struct usb_serial *serial) { struct pl2303_serial_private *spriv; - unsigned char num_ports = serial->num_ports; enum pl2303_type type = TYPE_01; unsigned char *buf; - if (serial->num_bulk_in < num_ports || - serial->num_bulk_out < num_ports || - serial->num_interrupt_in < num_ports) { - dev_err(&serial->interface->dev, "missing endpoints\n"); - return -ENODEV; - } - spriv = kzalloc(sizeof(*spriv), GFP_KERNEL); if (!spriv) return -ENOMEM; @@ -939,6 +931,9 @@ static struct usb_serial_driver pl2303_device = { }, .id_table = id_table, .num_ports = 1, + .num_bulk_in = 1, + .num_bulk_out = 1, + .num_interrupt_in = 1, .bulk_in_size = 256, .bulk_out_size = 256, .open = pl2303_open, |