On Friday 11 September 2009 10:55:45 pm Herton Ronaldo Krzesinski wrote:
> +static int acpi_topstar_remove(struct acpi_device *device, int type)
> +{
> + struct topstar_hkey *tps_hkey = acpi_driver_data(device);
> +
> + acpi_topstar_fncx_switch(device, false);
> +
> + input_unregister_device(tps_hkey->inputdev);
> + kfree(tps_hkey);
> + device->driver_data = NULL;
I think you should remove this "driver_data = NULL" assignment.
Clearing it in the driver helps cover up bugs like the one Alan
just fixed:
http://marc.info/?l=linux-acpi&m=125276975701005&w=2
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
