[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ba45d980-c6b1-482a-8bfd-457a92d3e24b@stanley.mountain>
Date: Fri, 28 Feb 2025 20:40:00 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: Gabriel Lima Luz <lima.gabriel.luz@...il.com>
Cc: linux-fbdev@...r.kernel.org, linux-staging@...ts.linux.dev,
linux-kernel@...r.kernel.org,
Sudip Mukherjee <sudipm.mukherjee@...il.com>,
Teddy Wang <teddy.wang@...iconmotion.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
~lkcamp/patches@...ts.sr.ht
Subject: Re: [PATCH] staging: sm750fb: Fix CamelCase variable naming
On Fri, Feb 28, 2025 at 01:23:54PM -0300, Gabriel Lima Luz wrote:
> Adhere to Linux kernel coding style.
>
> Reported by checkpatch:
>
> CHECK: Avoid CamelCase
>
> Signed-off-by: Gabriel Lima Luz <lima.gabriel.luz@...il.com>
> ---
> drivers/staging/sm750fb/ddk750_power.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/sm750fb/ddk750_power.h b/drivers/staging/sm750fb/ddk750_power.h
> index 63c9e8b6ffb3..33e852fe6949 100644
> --- a/drivers/staging/sm750fb/ddk750_power.h
> +++ b/drivers/staging/sm750fb/ddk750_power.h
> @@ -3,10 +3,10 @@
> #define DDK750_POWER_H__
>
> enum dpms {
> - crtDPMS_ON = 0x0,
> - crtDPMS_STANDBY = 0x1,
> - crtDPMS_SUSPEND = 0x2,
> - crtDPMS_OFF = 0x3,
> + crt_DPMS_ON = 0x0,
> + crt_DPMS_STANDBY = 0x1,
> + crt_DPMS_SUSPEND = 0x2,
> + crt_DPMS_OFF = 0x3,
> };
>
It seems these are not used. Just delete them.
regards,
dan carpenter
Powered by blists - more mailing lists