On Tuesday 27 October 2009 12:02:01 pm Alexey Starikovskiy wrote: > Currently EC driver work by doing either ECDT parse or acpi_walk_namespace, > Do you think it will be appropriate to change it to use PNP layer? I don't see this. In acpi_ec_init(), there's an acpi_bus_register_driver(). There's also the ECDT parsing in acpi_ec_ecdt_probe(). There's a comment near the acpi_ec_ecdt_probe() call that says it must happen before the EC is found in the namespace. There is an acpi_walk_namespace() inside acpi_ec_add(), but that's only to find _Qx methods under an EC device we've already found. It's conceivable that the EC driver could use PNP registration rather than ACPI registration -- that would let you get rid of the acpi_walk_resources() in ec_parse_device(). But I don't know that it's worth it right now. For IPMI, we're adding a bunch of new code and there are structural problems that EC doesn't have. So IPMI feels more urgent to me. 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
