On 11/9/23 09:27, Guenter Roeck wrote:
On 11/8/23 17:49, Delphine CC Chiu wrote:
+ info->m[PSC_POWER] = temp_setting; + } else { + info->m[PSC_POWER] = rsense;I told you before, the default range needs to be set. The range may have been changed by the BIOS/ROMMON, or someone could have changed it manually with i2cset or some other application, or some other operating system was loaded earlier which did its own setting. I do understand by now that you don't want to do that, but I won't accept the driver without it, sorry.
As a follow-up on this, it turns out that LTC4287 supports an external EEPROM to store its configuration as well as "STORE_USER_ALL" and "RESTORE_USER_ALL" PMBus commands. It is therefore very well possible and even likely that the voltage range (together with other configurable parameters) was set during system production. With that in mind, it seems to me that not only the default (property not present) case has to be handled, but that a boolean property to reflect the voltage range is insufficient. I think what is needed instead is a property which specifies the requested range and which, if not present, explicitly means "use whatever is currently programmed into the chip". Thanks, Guenter
