20 Oct
2020
20 Oct
'20
2:09 p.m.
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?