On Sat, Sep 29, 2012 at 01:31:52PM +0000, Zhang, Rui wrote: > > > > +{ > > > + acpi_handle handle = DEVICE_ACPI_HANDLE(dev); > > > > If the device is not bound to an ACPI handle this will return NULL. And > > I don't see you doing that in this series meaning that.. > > > > > You're right, I should set pdev->archdata.acpi_handle to the I2C > controller in i2c_root.c. There already is an API for that - check drivers/acpi/glue.c.