From: Vladimir Oltean <vladimir.olt...@nxp.com>
Date: Mon, 21 Sep 2020 19:27:41 +0300

> @@ -110,10 +222,15 @@ int sja1105_devlink_setup(struct dsa_switch *ds)
>       if (rc)
>               return rc;
>  
> +     rc = sja1105_setup_devlink_regions(ds);
> +     if (rc < 0)
> +             return rc;
> +
>       return 0;
>  }

I think you need to release the devlink params on failure here.  Otherwise
I don't see anything that will clean that up.

Reply via email to