| Message ID | 20201210144808.64384-1-christophe.jaillet@wanadoo.fr (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | [1/2] video: fbdev: Use framebuffer_release instead of kfree to free a frame buffer | expand |
diff --git a/drivers/video/fbdev/ep93xx-fb.c b/drivers/video/fbdev/ep93xx-fb.c index ba33b4dce0df..80a70e5796b8 100644 --- a/drivers/video/fbdev/ep93xx-fb.c +++ b/drivers/video/fbdev/ep93xx-fb.c @@ -566,7 +566,7 @@ static int ep93xxfb_probe(struct platform_device *pdev) failed_videomem: fb_dealloc_cmap(&info->cmap); failed_cmap: - kfree(info); + framebuffer_release(info); return err; } @@ -584,7 +584,7 @@ static int ep93xxfb_remove(struct platform_device *pdev) if (fbi->mach_info->teardown) fbi->mach_info->teardown(pdev); - kfree(info); + framebuffer_release(info); return 0; }
Use 'framebuffer_release()' instead of 'kfree()' to undo a 'framebuffer_alloc()' call, both in the error handling path of the probe function and in remove function. Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- drivers/video/fbdev/ep93xx-fb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)