On 5/28/2020 2:05 PM, Andrei Botila (OSS) wrote: > @@ -4698,6 +4698,9 @@ static void dpaa2_dpseci_free(struct dpaa2_caam_priv > *priv) > struct device *dev = priv->dev; > struct fsl_mc_device *ls_dev = to_fsl_mc_device(dev); > > + if (DPSECI_VER(priv->major_ver, priv->minor_ver) > DPSECI_VER(5, 3)) > + dpseci_reset(priv->mc_io, 0, ls_dev->mc_handle); Even though dpseci_close() should be called in all cases, a warning when reset fails would be useful.
> + > dpaa2_dpseci_congestion_free(priv); > dpseci_close(priv->mc_io, 0, ls_dev->mc_handle); > }