]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
drop usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch from everywhere
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 21 Dec 2022 16:11:24 +0000 (17:11 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 21 Dec 2022 16:11:24 +0000 (17:11 +0100)
14 files changed:
queue-4.14/series
queue-4.14/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-4.19/series
queue-4.19/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-5.10/series
queue-5.10/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-5.15/series
queue-5.15/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-5.4/series
queue-5.4/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-6.0/series
queue-6.0/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]
queue-6.1/series
queue-6.1/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch [deleted file]

index 680c18cb7c0f764231449e0cb8615cb4dedd91c6..78dc3aceed07989b8595c4c082cf5a1f096f50bd 100644 (file)
@@ -18,6 +18,5 @@ usb-gadget-uvc-prevent-buffer-overflow-in-setup-handler.patch
 usb-serial-option-add-quectel-em05-g-modem.patch
 usb-serial-cp210x-add-kamstrup-rf-sniffer-pids.patch
 igb-initialize-mailbox-message-for-vf-reset.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 bluetooth-l2cap-fix-u8-overflow.patch
 net-loopback-use-net_name_predictable-for-name_assig.patch
diff --git a/queue-4.14/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-4.14/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 80790a0..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -210,7 +210,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index a4b0cb9f9a0edeebb29bf4fa5a885e27fa4e64fe..e4c2334327b03e3de46d76c0e723c3da59a0dc4a 100644 (file)
@@ -17,6 +17,5 @@ usb-serial-option-add-quectel-em05-g-modem.patch
 usb-serial-cp210x-add-kamstrup-rf-sniffer-pids.patch
 usb-serial-f81534-fix-division-by-zero-on-line-speed-change.patch
 igb-initialize-mailbox-message-for-vf-reset.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 bluetooth-l2cap-fix-u8-overflow.patch
 net-loopback-use-net_name_predictable-for-name_assig.patch
diff --git a/queue-4.19/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-4.19/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 8e85b8e..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -207,7 +207,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index 4235029dd0fbceed9314baf7ecfa3ccaee363c2a..d2a97ef3ac4b67a009d24f4e907ce52dab06d8c8 100644 (file)
@@ -9,7 +9,6 @@ usb-serial-f81232-fix-division-by-zero-on-line-speed-change.patch
 usb-serial-f81534-fix-division-by-zero-on-line-speed-change.patch
 xhci-apply-xhci_reset_to_default-quirk-to-adl-n.patch
 igb-initialize-mailbox-message-for-vf-reset.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 hid-ite-add-support-for-acer-s1002-keyboard-dock.patch
 hid-ite-enable-quirk_touchpad_on_off_report-on-acer-.patch
 hid-ite-enable-quirk_touchpad_on_off_report-on-acer-.patch-1053
diff --git a/queue-5.10/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-5.10/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 72401eb..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -208,7 +208,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index e3d7536ba97042ba36c8c57508e58e996b7f76a5..a14f7e4d4790fb35a4f71800b887292280007e97 100644 (file)
@@ -10,7 +10,6 @@ usb-serial-f81534-fix-division-by-zero-on-line-speed-change.patch
 xhci-apply-xhci_reset_to_default-quirk-to-adl-n.patch
 igb-initialize-mailbox-message-for-vf-reset.patch
 usb-dwc3-pci-update-pcie-device-id-for-usb3-controller-on-cpu-sub-system-for-raptor-lake.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 hid-uclogic-add-hid_quirk_hidinput_force-quirk.patch
 bluetooth-l2cap-fix-u8-overflow.patch
 selftests-net-use-grep-e-instead-of-egrep.patch
diff --git a/queue-5.15/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-5.15/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index d090f36..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -206,7 +206,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index 0e1221b57732cfb7e42cd4b31500367991f5a840..649b5440057f6e92802d265f2515261507158e2e 100644 (file)
@@ -9,7 +9,6 @@ usb-serial-cp210x-add-kamstrup-rf-sniffer-pids.patch
 usb-serial-f81232-fix-division-by-zero-on-line-speed-change.patch
 usb-serial-f81534-fix-division-by-zero-on-line-speed-change.patch
 igb-initialize-mailbox-message-for-vf-reset.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 xen-netback-move-removal-of-hotplug-status-to-the-right-place.patch
 hid-ite-add-support-for-acer-s1002-keyboard-dock.patch
 hid-ite-enable-quirk_touchpad_on_off_report-on-acer-.patch
diff --git a/queue-5.4/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-5.4/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 8e85b8e..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -207,7 +207,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index eb6f41522fd90ac61794de9e51fc8807ff94087e..782e6c9bfa867bdf854d130ecefcb4bc560060cb 100644 (file)
@@ -22,7 +22,6 @@ igb-initialize-mailbox-message-for-vf-reset.patch
 usb-typec-ucsi-resume-in-separate-work.patch
 usb-dwc3-pci-update-pcie-device-id-for-usb3-controller-on-cpu-sub-system-for-raptor-lake.patch
 keys-encrypted-fix-key-instantiation-with-user-provided-data.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
 bluetooth-l2cap-fix-u8-overflow.patch
 selftests-net-use-grep-e-instead-of-egrep.patch
 net-loopback-use-net_name_predictable-for-name_assig.patch
diff --git a/queue-6.0/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-6.0/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 8e85b8e..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -207,7 +207,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)
index 7e50e23ebeac377642bd08436e186e7082449973..236531d9f299352345bb2de1326194943785a5eb 100644 (file)
@@ -22,4 +22,3 @@ usb-typec-ucsi-resume-in-separate-work.patch
 usb-dwc3-pci-update-pcie-device-id-for-usb3-controller-on-cpu-sub-system-for-raptor-lake.patch
 cifs-fix-oops-during-encryption.patch
 keys-encrypted-fix-key-instantiation-with-user-provided-data.patch
-usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
diff --git a/queue-6.1/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch b/queue-6.1/usb-ulpi-defer-ulpi_register-on-ulpi_read_id-timeout.patch
deleted file mode 100644 (file)
index 8e85b8e..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-From 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac Mon Sep 17 00:00:00 2001
-From: Ferry Toth <ftoth@exalondelft.nl>
-Date: Mon, 5 Dec 2022 21:15:26 +0100
-Subject: usb: ulpi: defer ulpi_register on ulpi_read_id timeout
-
-From: Ferry Toth <ftoth@exalondelft.nl>
-
-commit 8a7b31d545d3a15f0e6f5984ae16f0ca4fd76aac upstream.
-
-Since commit 0f0101719138 ("usb: dwc3: Don't switch OTG -> peripheral
-if extcon is present") Dual Role support on Intel Merrifield platform
-broke due to rearranging the call to dwc3_get_extcon().
-
-It appears to be caused by ulpi_read_id() on the first test write failing
-with -ETIMEDOUT. Currently ulpi_read_id() expects to discover the phy via
-DT when the test write fails and returns 0 in that case, even if DT does not
-provide the phy. As a result usb probe completes without phy.
-
-Make ulpi_read_id() return -ETIMEDOUT to its user if the first test write
-fails. The user should then handle it appropriately. A follow up patch
-will make dwc3_core_init() set -EPROBE_DEFER in this case and bail out.
-
-Fixes: ef6a7bcfb01c ("usb: ulpi: Support device discovery via DT")
-Cc: stable@vger.kernel.org
-Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
-Signed-off-by: Ferry Toth <ftoth@exalondelft.nl>
-Link: https://lore.kernel.org/r/20221205201527.13525-2-ftoth@exalondelft.nl
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/usb/common/ulpi.c |    2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
---- a/drivers/usb/common/ulpi.c
-+++ b/drivers/usb/common/ulpi.c
-@@ -207,7 +207,7 @@ static int ulpi_read_id(struct ulpi *ulp
-       /* Test the interface */
-       ret = ulpi_write(ulpi, ULPI_SCRATCH, 0xaa);
-       if (ret < 0)
--              goto err;
-+              return ret;
-       ret = ulpi_read(ulpi, ULPI_SCRATCH);
-       if (ret < 0)