Re: [PATCH] usb: cdns3: imx: simplify system pm using _force_suspend/_resume

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

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@nxp.com>
> Signed-off-by: Shenwei Wang <shenwei.wang@nxp.com>
> ---
>  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?