On Fri, Feb 17, 2023 at 12:49:04PM -0600, Danny Kaehn wrote: > Bind i2c and gpio interfaces to subnodes with names I2C GPIO > "i2c" and "gpio" if they exist, respectively. This > allows the gpio and i2c controllers to be described GPIO I2C > in firmware as usual. Additionally, support configuring the > i2c bus speed from the clock-frequency device property. I2C ... > +err_free_i2c_of: > + fwnode_handle_put(dev->adap.dev.fwnode); Please, use dev_fwnode(&dev->adap.dev) ... > + fwnode_handle_put(dev->adap.dev.fwnode); Ditto. -- With Best Regards, Andy Shevchenko