Re: [PATCH v5 2/3] staging: vt6655: fix camelcase in ldBmThreshold

On Tue, Sep 28, 2021 at 06:42:13AM +0200, Greg Kroah-Hartman wrote:

On Mon, Sep 27, 2021 at 11:46:24PM +0200, Tommaso Merciai wrote:

quoted

On Mon, Sep 27, 2021 at 05:26:36PM +0200, Greg Kroah-Hartman wrote:

quoted

On Sun, Sep 26, 2021 at 06:25:19PM +0200, Tommaso Merciai wrote:

quoted

Replace camel case variable ldBmThreshold with snake case
variable ld_bm_threshold.

Same here, what exactly does this name mean and why did you pick it?

  You are right the same here. What do you think about "bm_threshold"?

What does "bm" stand for?

  Got it. In the current variable name "l_" is for long (hungarian
  notation), "dBm" is decibel milliwatts, signal strenght.
  A good solution could be "dbm_threshold"?

  thanks.
  tommaso

thanks,

greg k-h