On Sat, Jul 22, 2023 at 9:38 AM Yuanjun Gong <ruc_gongyuan...@163.com> wrote: > > check the return value of clk_prepare_enable(), and if > clk_prepare_enable() gets an unexpected return value, > imx_audmux_suspend() and imx_audmux_resume() should return > the error value. > > Signed-off-by: Yuanjun Gong <ruc_gongyuan...@163.com> > --- > sound/soc/fsl/imx-audmux.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx-audmux.c > index be003a117b39..9791e56158ef 100644 > --- a/sound/soc/fsl/imx-audmux.c > +++ b/sound/soc/fsl/imx-audmux.c > @@ -325,8 +325,11 @@ static void imx_audmux_remove(struct platform_device > *pdev) > static int imx_audmux_suspend(struct device *dev) > { > int i; > + ssize_t ret;
Why not simply "int ret" instead?