On Wed, Jan 30, 2013 at 06:15:12PM -0700, Toshi Kani wrote:

quoted

Please make it a "real" pointer, and not a void *, those shouldn't be
used at all if possible.

How about changing the "void *handle" to acpi_dev_node below?   

   struct acpi_dev_node    acpi_node;

Basically, it has the same challenge as struct device, which uses
acpi_dev_node as well.  We can add other FW node when needed (just like
device also has *of_node).

That sounds good to me.