Thread (5 messages) 5 messages, 4 authors, 2020-09-04

Re: [PATCH] Revert "usb: dwc3: meson-g12a: fix shared reset control use"

From: Philipp Zabel <p.zabel@pengutronix.de>
Date: 2020-08-27 15:05:00
Also in: linux-amlogic, linux-usb, lkml

On Thu, 2020-08-27 at 16:48 +0200, Amjad Ouled-Ameur wrote:
This reverts commit 7a410953d1fb4dbe91ffcfdee9cbbf889d19b0d7.

This commit breaks USB on meson-gxl-s905x-libretech-cc. Reverting
the change solves the issue.

In fact, according to the reset framework code, consumers must not use
reset_control_(de)assert() on shared reset lines when reset_control_reset
has been used, and vice-versa.

Moreover, with this commit, usb is not guaranted to be reset since the
reset is likely to be initially deasserted.

Reverting the commit will bring back the suspend warning mentioned in the
commit description. Nevertheless, a warning is much less critical than
breaking dwc3-meson-g12a USB completely. We will address the warning
issue in another way as a 2nd step.

Signed-off-by: Amjad Ouled-Ameur <redacted>
Reported-by: Jerome Brunet <jbrunet@baylibre.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>

regards
Philipp

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help