Hi Dmitry,
+ if (pdata->device_enable) {+ error = pdata->device_enable(dev); + if (error < 0) { + dev_dbg(dev, "device enable function failed\n"); + return error; + } + } +Hi Miguel, Does this need to live in the driver? Why can't platform code do this for us? Thanks.
The reason to invoke the device_enable function in the driver is because in the testing process of the key scan driver a issue was found when the key scan is built as a module.
There is a specific PINMUX configuration that only should be set if the key scan driver is loaded in the kernel to avoid pin conflicts. So when the key scan is built as a module the board file (or platform code) doesn't know if the key scan is loaded or not, so that's why the driver is the one who must invoke the device_enable function in the probe function.
Thanks, Miguel Aguilar _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
