auxdisplay: hd44780: Fix an API misuse in hd44780.c

commit 9b98a7d2e5f4e2beeff88f6571da0cdc5883c7fb upstream.

Variable allocated by charlcd_alloc() should be released
by charlcd_free(). The following patch changed kfree() to
charlcd_free() to fix an API misuse.

Fixes: 718e05ed92ec ("auxdisplay: Introduce hd44780_common.[ch]")
Cc: stable@vger.kernel.org
Signed-off-by: Haoxiang Li <haoxiang_li2024@163.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/auxdisplay/hd44780.c b/drivers/auxdisplay/hd44780.c
index 0526f0d90a793e4524a698e3148e4941c3703c1a..9d0ae9c02e9ba2c5560c9423a2314b246c460f6b 100644 (file)
--- a/drivers/auxdisplay/hd44780.c
+++ b/drivers/auxdisplay/hd44780.c
@@ -313,7 +313,7 @@ static int hd44780_probe(struct platform_device *pdev)
 fail3:
        kfree(hd);
 fail2:
-       kfree(lcd);
+       charlcd_free(lcd);
 fail1:
        kfree(hdc);
        return ret;
@@ -328,7 +328,7 @@ static void hd44780_remove(struct platform_device *pdev)
        kfree(hdc->hd44780);
        kfree(lcd->drvdata);
 
-       kfree(lcd);
+       charlcd_free(lcd);
 }
 
 static const struct of_device_id hd44780_of_match[] = {