Messages in this thread | | | Date | Fri, 26 Aug 2022 09:16:38 +0200 | From | Johan Hovold <> | Subject | Re: [PATCH v2 2/2] clk: qcom: gcc-sc7280: Keep the USB GDSCs always on |
| |
On Thu, Aug 25, 2022 at 06:21:59PM -0700, Matthias Kaehlcke wrote: > When the GDSC is disabled during system suspend USB is broken on > sc7280 when the system resumes. Mark the GDSC as always on to > make sure USB still works after system suspend. > > Signed-off-by: Matthias Kaehlcke <mka@chromium.org> > --- > > Changes in v2: > - set the flags of the GDSC not of the GDSC power domain > - updated commit message > > drivers/clk/qcom/gcc-sc7280.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/qcom/gcc-sc7280.c b/drivers/clk/qcom/gcc-sc7280.c > index 7ff64d4d5920..adef68d2cb0b 100644 > --- a/drivers/clk/qcom/gcc-sc7280.c > +++ b/drivers/clk/qcom/gcc-sc7280.c
Perhaps you can add a comment here about why this is needed similar to what I did for sc8280xp:
https://lore.kernel.org/all/20220805121250.10347-3-johan+linaro@kernel.org/
> @@ -3127,7 +3127,7 @@ static struct gdsc gcc_usb30_prim_gdsc = { > .name = "gcc_usb30_prim_gdsc", > }, > .pwrsts = PWRSTS_OFF_ON, > - .flags = VOTABLE, > + .flags = VOTABLE | ALWAYS_ON, > }; > > static struct gdsc gcc_usb30_sec_gdsc = {
Look good otherwise. For both patches:
Reviewed-by: Johan Hovold <johan+linaro@kernel.org>
Johan
| |