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.