On Thu, Jul 25, 2024 at 04:28:11PM -0500, Shenwei Wang wrote: > By utilizing _force_suspend and _force_resume, ensures a more consistent > and simple approach to handling system sleep states. It also aligns the > driver's PM behavior with the other drivers' common practices. > > Fixes: db3c4e366287 ("usb: cdns3: imx: Rework system PM to avoid duplicated operations") > Reviewed-by: Frank Li <frank.li@xxxxxxx> > Signed-off-by: Shenwei Wang <shenwei.wang@xxxxxxx> > --- > drivers/usb/cdns3/cdns3-imx.c | 18 +++++++----------- > 1 file changed, 7 insertions(+), 11 deletions(-) > > diff --git a/drivers/usb/cdns3/cdns3-imx.c b/drivers/usb/cdns3/cdns3-imx.c > index 281de47e2a3b..9051cbe9d68b 100644 > --- a/drivers/usb/cdns3/cdns3-imx.c > +++ b/drivers/usb/cdns3/cdns3-imx.c > @@ -360,7 +360,6 @@ static int cdns_imx_suspend(struct device *dev) > return 0; > } > > - > /* Indicate if the controller was power lost before */ > static inline bool cdns_imx_is_power_lost(struct cdns_imx *data) > { Unneeded change here?