Hi Rafael,
2013/02/08 9:24, Rafael J. Wysocki wrote:
On Monday, February 04, 2013 12:47:31 AM Rafael J. Wysocki wrote:
From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
The only useful thing that the ACPI container driver does is to
install system notify handlers for all container and module device
objects it finds in the namespace. The driver structure,
acpi_container_driver, and the data structures created by its
.add() callback are in fact not used by the driver, so remove
them entirely.
It also makes a little sense to build that driver as a module,
so make it non-modular and add its initialization to the
namespace scanning code.
In addition to that, make the namespace walk callback used for
installing the notify handlers more straightforward.
As pointed out by Toshi Kani, the above changes would make acpi_eject_store()
fail for containers and it is the only way to eject them currently, so patch
[2/2] is an improved version of this (with Toshi's changes folded in).
Patch [1/2] is just a cleanup removing a useless #ifndef from acpi_eject_store().
I confimed the patch series works well.
Acked-by: Yasuaki Ishimatsu <isimatu.yasuaki@xxxxxxxxxxxxxx>
Tested-by: Yasuaki Ishimatsu <isimatu.yasuaki@xxxxxxxxxxxxxx>
Thanks,
Yasuaki Ishimatsu
Thanks,
Rafael
--
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
