+ Cc: Jarkko, he spent a lot of nice hours to debug i2c HID touchscreen devices on x86 ACPI enabled platforms, so, he might have a better idea or some comments. On Mon, 2017-10-02 at 14:32 -0700, Rajat Jain wrote: > The property "post-power-on-delay-ms"" allows a platform to specify > the delay needed after power-on, but only via device trees. Thus > allow ACPI systems to also provide the same information. This one is even less acceptable (in given form), see below why. > + if (!device_property_read_u32(&client->dev, "post-power-on- > delay-ms", > + &val)) The main idea behind unified device properties API is to provide a way which will be resource provider agnostic, i.e. callers will get data in some kind of generic way independently on the source of it. Since I2C HID protocol is well defined by Microsoft and it doesn't involve _DSD, you make here even more gnostic solution. Besides the fact, each property must be registered in Device Tree bindings (yes, even if it's going to be used for ACPI enabled platforms initially). Thus, _if_ (and only if) we have no other solution, you need to clean up your first version and send it as v3. Don't forget to add a version to the patch (git-format-patch has a command line option to make this simpler). -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
