On Mon, Feb 24, 2020 at 05:35:54PM +0200, Vladimir Zapolskiy wrote: > On 2/24/20 5:15 PM, Andy Shevchenko wrote: > > Since we have generic definitions for bus frequencies, let's use them. > > - bus_clk_rate = 100000; /* 100 kHz default clock rate */ > > + bus_clk_rate = I2C_STANDARD_MODE_FREQ; /* 100 kHz default clock rate */ > > The line above becomes longer than 80 symbols, please fix it by simply > removing the comment, note that it might be an issue through the series. Will do. > Could I2C_STD_MODE_FREQ be a shorter and still acceptable name? I/m fine with it, but I will wait for Wolfram and others to comment. > After the requested fix: > > Acked-by: Vladimir Zapolskiy <vz@xxxxxxxxx> Thanks! -- With Best Regards, Andy Shevchenko
