# ILI9320: Ensure resource cleanup on exit.

- From: Ben Dooks \<ben-linux@fluff.org\>
- Date: 2008-06-24 18:23:48 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2008/06/13458412
- Thread: https://ratatoskr.run/linux-fbdev/2008/06/13458412/t.md

```text
Ensure we free our 'struct ili9320' on exit or error.

Signed-off-by: Ben Dooks <ben-linux@fluff.org>

Index: linux-2.6.26-rc7-quilt1/drivers/video/backlight/ili9320.c
===================================================================
--- linux-2.6.26-rc7-quilt1.orig/drivers/video/backlight/ili9320.c	2008-06-24 12:58:38.000000000 +0100
+++ linux-2.6.26-rc7-quilt1/drivers/video/backlight/ili9320.c	2008-06-24 14:33:30.000000000 +0100
@@ -259,18 +259,19 @@ int __devinit ili9320_probe_spi(struct s
 	lcd_device_unregister(lcd);
 
  err_free:
-	kfree(lcd);
+	kfree(ili);
+
 	return ret;
 }
 
 EXPORT_SYMBOL_GPL(ili9320_probe_spi);
 
-int __devexit ili9320_remove(struct ili9320 *lcd)
+int __devexit ili9320_remove(struct ili9320 *ili)
 {
-	ili9320_power(lcd, FB_BLANK_POWERDOWN);
+	ili9320_power(ili, FB_BLANK_POWERDOWN);
 
-	lcd_device_unregister(lcd->lcd);
-	kfree(lcd);
+	lcd_device_unregister(ili->lcd);
+	kfree(ili);
 
 	return 0;
 }

-- 
Ben (ben@fluff.org, http://www.fluff.org/)

  'a smiley only costs 4 bytes'

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
```