On Mon, Jun 10, 2024 at 07:15:51PM -0700, <Tree Davies wrote: > On Mon, Jun 10, 2024 at 09:08:03AM +0300, Dan Carpenter wrote: > > On Sun, Jun 09, 2024 at 10:44:34PM -0700, Tree Davies wrote: > > > Rename variable bforced_tx20Mhz to forced_tx_20mhz > > > to fix checkpatch warning Avoid CamelCase. > > > > The correct capitalization for megahertz is MHz. So the original is > > wrong but the new name is not correct either. > > > > regards, > > dan carpenter > > > > Thanks Dan, > So... I guess just spell it out as 'forced_tx_20_megahertz' ? No, just write it as "forced_tx_20MHz". Don't bow down to checkpatch. regards, dan carpenter
