On Mon, Oct 19, 2020 at 05:44:27PM -0500, David Rhodes wrote: > - /* Only certain ratios are supported in I2S Slave Mode */ > - if (cs35l35->slave_mode) { > + /* Only certain ratios are supported in I2S Device Mode */ > + if (cs35l35->clock_mode) { > - /* Only certain ratios supported in I2S MASTER Mode */ > + /* Only certain ratios supported in I2S Controller Mode */ Your renaming moves this to a boolean variable called clock mode which maps onto device and controller. I am not sure people are going to be able to follow that. Perhaps clock_source or something?
Attachment:
signature.asc
Description: PGP signature