On Sun, Sep 1, 2024 at 3:43 PM Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > > On 7/17/24 07:42, Guenter Roeck wrote: > > Patrick, > > > > On 7/17/24 00:29, Patrick Rudolph wrote: > >> Some drivers, like the max15301 or zl6100, are intentionally delaying > >> SMBus communications, to prevent transmission errors. As this is necessary > >> on additional PMBus compatible devices, implement a generic delay mechanism > >> in the pmbus core. > >> > >> Introduces two delay settings in the pmbus_driver_info struct, one applies > >> to every SMBus transaction and the other is for write transaction only. > >> Once set by the driver the SMBus traffic, using the generic pmbus access > >> helpers, is automatically delayed when necessary. > >> > >> The two settings are: > >> access_delay: > >> - Unit in microseconds > >> - Stores the accessed timestamp after every SMBus access > >> - Delays when necessary before the next SMBus access > >> > >> write_delay: > >> - Unit in microseconds > >> - Stores the written timestamp after a write SMBus access > >> - Delays when necessary before the next SMBus access > >> > >> This allows to drop the custom delay code from the drivers and easily > >> introduce this feature in additional pmbus drivers. > >> > >> Signed-off-by: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx> > >> --- > > > > Sigh (sorry, but this isn't your first patch, and you should know). > > > > Change log goes here. > > > > Maybe I was not explicit enough, or maybe I offended you with the above. > If so, I am sorry. > > Either case, I am not accepting patches or patch series without change log. > > Guenter > Hi Guenter, Thanks for reminding me and being patient. It's a good habit to make no exceptions when it comes to the quality of submitted patches. Please find my updated patch series, this time with a changelog. Patrick