Re: [PATCH v2 9/9] usb: dwc3: qcom: clean up suspend callbacks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Johan Hovold <johan+linaro@xxxxxxxxxx>
Subject: Re: [PATCH v2 9/9] usb: dwc3: qcom: clean up suspend callbacks
From: Matthias Kaehlcke <mka@xxxxxxxxxxxx>
Date: Thu, 4 Aug 2022 10:08:03 -0700
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Felipe Balbi <balbi@xxxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>, Andy Gross <agross@xxxxxxxxxx>, Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>, Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>, Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx>, Krishna Kurapati <quic_kriskura@xxxxxxxxxxx>, Stephen Boyd <swboyd@xxxxxxxxxxxx>, Doug Anderson <dianders@xxxxxxxxxxxx>, Pavankumar Kondeti <quic_pkondeti@xxxxxxxxxxx>, quic_ppratap@xxxxxxxxxxx, quic_vpulyala@xxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, linux-usb@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20220804151001.23612-10-johan+linaro@kernel.org>
References: <20220804151001.23612-1-johan+linaro@kernel.org> <20220804151001.23612-10-johan+linaro@kernel.org>

On Thu, Aug 04, 2022 at 05:10:01PM +0200, Johan Hovold wrote:
> Clean up the suspend callbacks by separating the error and success paths
> to improve readability.
> 
> Also drop a related redundant initialisation.
> 
> Signed-off-by: Johan Hovold <johan+linaro@xxxxxxxxxx>

Reviewed-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>

References:
- [PATCH v2 0/9] usb: dwc3: qcom: fix wakeup implementation
  - From: Johan Hovold
- [PATCH v2 9/9] usb: dwc3: qcom: clean up suspend callbacks
  - From: Johan Hovold

Prev by Date: Re: [PATCH v2 8/9] usb: dwc3: qcom: fix wakeup implementation
Next by Date: Re: [PATCH 1/2] drm/msm: Move hangcheck timer restart
Previous by thread: [PATCH v2 9/9] usb: dwc3: qcom: clean up suspend callbacks
Next by thread: [PATCH v2 4/9] usb: dwc3: qcom: fix use-after-free on runtime-PM wakeup
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]