Commit 632691ad authored Jun 02, 2024 by Erick Archer Committed by Andy Shevchenko Jun 04, 2024

auxdisplay: Use sizeof(*pointer) instead of sizeof(type)



It is preferred to use sizeof(*pointer) instead of sizeof(type)
due to the type of the variable can change and one needs not
change the former (unlike the latter). This patch has no effect
on runtime behavior.

Signed-off-by: Erick Archer <erick.archer@outlook.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

parent c56a4506

drivers/auxdisplay/arm-charlcd.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -270,7 +270,7 @@ static int __init charlcd_probe(struct platform_device *pdev)
		struct charlcd *lcd;
		struct resource *res;

		lcd = kzalloc(sizeof(struct charlcd), GFP_KERNEL);
		lcd = kzalloc(sizeof(*lcd), GFP_KERNEL);
		if (!lcd)
		return -ENOMEM;

drivers/auxdisplay/hd44780.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -230,7 +230,7 @@ static int hd44780_probe(struct platform_device *pdev)
		if (!lcd)
		goto fail1;

		hd = kzalloc(sizeof(struct hd44780), GFP_KERNEL);
		hd = kzalloc(sizeof(*hd), GFP_KERNEL);
		if (!hd)
		goto fail2;