[PATCH 3.2 135/147] mct_u232: Fix use of uninitialized pointer in mct_u323_startup()

2017-11-06 Thread Ben Hutchings
3.2.95-rc1 review patch.  If anyone has any objections, please let me know.

--

From: Ben Hutchings 

My backport of commit 4e9a0b05257f "USB: mct_u232: add sanity checking in
probe" incorrectly added a dev_err() call using port->dev before 'port' was
initialised.  Use the 'serial' parameter to look up the device instead.

Signed-off-by: Ben Hutchings 
---
--- a/drivers/usb/serial/mct_u232.c
+++ b/drivers/usb/serial/mct_u232.c
@@ -449,7 +449,7 @@ static int mct_u232_startup(struct usb_s
 
/* check first to simplify error handling */
if (!serial->port[1] || !serial->port[1]->interrupt_in_urb) {
-   dev_err(>dev, "expected endpoint missing\n");
+   dev_err(>dev->dev, "expected endpoint missing\n");
return -ENODEV;
}
 



[PATCH 3.2 135/147] mct_u232: Fix use of uninitialized pointer in mct_u323_startup()

2017-11-06 Thread Ben Hutchings
3.2.95-rc1 review patch.  If anyone has any objections, please let me know.

--

From: Ben Hutchings 

My backport of commit 4e9a0b05257f "USB: mct_u232: add sanity checking in
probe" incorrectly added a dev_err() call using port->dev before 'port' was
initialised.  Use the 'serial' parameter to look up the device instead.

Signed-off-by: Ben Hutchings 
---
--- a/drivers/usb/serial/mct_u232.c
+++ b/drivers/usb/serial/mct_u232.c
@@ -449,7 +449,7 @@ static int mct_u232_startup(struct usb_s
 
/* check first to simplify error handling */
if (!serial->port[1] || !serial->port[1]->interrupt_in_urb) {
-   dev_err(>dev, "expected endpoint missing\n");
+   dev_err(>dev->dev, "expected endpoint missing\n");
return -ENODEV;
}