Re: [PATCH 09/24] Staging: rtl8192e: Rename variable bforced_tx20Mhz

From: Dan Carpenter
Date: Mon Jun 10 2024 - 02:08:20 EST

Next message: Mike Rapoport: "Re: [PATCH] mm: sparse: clarify a variable name and its value"
Previous message: Martin Schiller: "Re: [PATCH net-next 12/13] net: dsa: lantiq_gswip: Add and use a GSWIP_TABLE_MAC_BRIDGE_FID macro"
In reply to: Tree Davies: "[PATCH 09/24] Staging: rtl8192e: Rename variable bforced_tx20Mhz"
Next in thread: <Tree Davies: "Re: [PATCH 09/24] Staging: rtl8192e: Rename variable bforced_tx20Mhz"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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

Next message: Mike Rapoport: "Re: [PATCH] mm: sparse: clarify a variable name and its value"
Previous message: Martin Schiller: "Re: [PATCH net-next 12/13] net: dsa: lantiq_gswip: Add and use a GSWIP_TABLE_MAC_BRIDGE_FID macro"
In reply to: Tree Davies: "[PATCH 09/24] Staging: rtl8192e: Rename variable bforced_tx20Mhz"
Next in thread: <Tree Davies: "Re: [PATCH 09/24] Staging: rtl8192e: Rename variable bforced_tx20Mhz"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]