On 1/25/2018 11:57 AM, Rafael J. Wysocki wrote: >> That was the original patch. Folks are saying that it breaks some systems. >> >> Conversation is here: >> >> https://patchwork.kernel.org/patch/10142425/ > Couldn't that be addressed by preventing the I2C module from being > unloaded in the relevant cases? > Maybe, I'll let Andy And Hans speak for themselves. It is not my code. I listed the spec behavior and expectation from the OS in the link above. Apparently, Linux isn't playing nice. Hans is also raising a concern about driver load order that is not being handled well between the I2C driver, its sub modules like PMIC and the ACPI support. "Lets pretend that all DSTDs are perfect and that some device described in ACPI has a _PS0 method which uses an opregion to turn on some regulator powering the device through i2c, but only if the _REG method for that opregion has been called. So now lets say that the driver for this device loads and tries to bind before the i2c-module has loaded. Before the driver's probe method gets called the driver-core will call _PS0 to power-up the device, which is a nop (*). Then the drivers probe function tries to talk to the device, but fails as the device is not powered, so it returns with -ENODEV." I'd think that driver would return -EPROBE_DEFER in order to wait until its dependencies come in place. Maybe, driver was not designed for this. -- Sinan Kaya Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
