diff options
author | Peter Seiderer <ps.report@gmx.net> | 2013-12-08 16:03:58 -0500 |
---|---|---|
committer | Philipp Zabel <p.zabel@pengutronix.de> | 2015-01-06 11:36:13 -0500 |
commit | dcbc9eb1919359719be0ef652afe1d90b0fa98ed (patch) | |
tree | d0196378a74ff582088ccfd00cf6e39be0b14fd1 /drivers/gpu/drm/imx/parallel-display.c | |
parent | 97bf6af1f928216fd6c5a66e8a57bfa95a659672 (diff) |
drm/imx: parallel-display: fix imxpd-->edid memleak
If edid was allocated during bind, it must be freed again during unbind.
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
Acked-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Diffstat (limited to 'drivers/gpu/drm/imx/parallel-display.c')
-rw-r--r-- | drivers/gpu/drm/imx/parallel-display.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c index 796c3c1c170a..88cf7ca1e886 100644 --- a/drivers/gpu/drm/imx/parallel-display.c +++ b/drivers/gpu/drm/imx/parallel-display.c | |||
@@ -257,6 +257,8 @@ static void imx_pd_unbind(struct device *dev, struct device *master, | |||
257 | 257 | ||
258 | imxpd->encoder.funcs->destroy(&imxpd->encoder); | 258 | imxpd->encoder.funcs->destroy(&imxpd->encoder); |
259 | imxpd->connector.funcs->destroy(&imxpd->connector); | 259 | imxpd->connector.funcs->destroy(&imxpd->connector); |
260 | |||
261 | kfree(imxpd->edid); | ||
260 | } | 262 | } |
261 | 263 | ||
262 | static const struct component_ops imx_pd_ops = { | 264 | static const struct component_ops imx_pd_ops = { |
@@ -272,6 +274,7 @@ static int imx_pd_probe(struct platform_device *pdev) | |||
272 | static int imx_pd_remove(struct platform_device *pdev) | 274 | static int imx_pd_remove(struct platform_device *pdev) |
273 | { | 275 | { |
274 | component_del(&pdev->dev, &imx_pd_ops); | 276 | component_del(&pdev->dev, &imx_pd_ops); |
277 | |||
275 | return 0; | 278 | return 0; |
276 | } | 279 | } |
277 | 280 | ||