On 04.05.2018 16:37, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > If an error happens during display controller initialization, the host1x > syncpoint previously requested would be leaked. Properly clean up the > syncpoint along with the other resources. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > drivers/gpu/drm/tegra/dc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/tegra/dc.c b/drivers/gpu/drm/tegra/dc.c > index f20648f58e49..c843f11043db 100644 > --- a/drivers/gpu/drm/tegra/dc.c > +++ b/drivers/gpu/drm/tegra/dc.c > @@ -1925,6 +1925,8 @@ static int tegra_dc_init(struct host1x_client *client) > iommu_group_put(dc->group); > } > > + host1x_syncpt_free(dc->syncpt); > + > return err; > } Reviewed-by: Dmitry Osipenko <digetx@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html