On Tue, Apr 06, 2021 at 06:35:17PM -0500, Aditya Pakki wrote:

The resource release in sm501fb_remove() is not in the inverse order of sm501fb_probe(), for the buffers. Release the info object after deallocating the buffers.

Signed-off-by: Aditya Pakki pakki001@umn.edu

drivers/video/fbdev/sm501fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/sm501fb.c b/drivers/video/fbdev/sm501fb.c index 6a52eba64559..4c32c9e88850 100644 --- a/drivers/video/fbdev/sm501fb.c +++ b/drivers/video/fbdev/sm501fb.c @@ -2060,11 +2060,11 @@ static int sm501fb_remove(struct platform_device *pdev) unregister_framebuffer(fbinfo_pnl);

sm501fb_stop(info);

• kfree(info);

  framebuffer_release(fbinfo_pnl); framebuffer_release(fbinfo_crt);

• kfree(info); return 0;

}

-- 2.25.1

There is no function change here at all, please stop it with pointless patches.

greg k-h