On Thu, Mar 26, 2020 at 12:00:42PM +0100, Wolfram Sang wrote: > > > However, looking into the code, I would go a bit further (perhaps as a separate > > change) and export parsing of clock-frequency, because tons of drivers only > > need one property, i.e. clock-frequency out of firmware. > > Cool idea. Could be easily something like this (typed from the top of my > head): > > static inline u32 i2c_parse_fw_bus_speed(struct device *dev) > { > u32 speed; > > i2c_parse_timing(dev, "clock-frequency", &speed, I2C_MAX_STANDARD_MODE_FREQ, true); > > return speed; > } Yes, looks like this, thanks! -- With Best Regards, Andy Shevchenko