Messages in this thread | | | Date | Wed, 2 Aug 2023 16:33:24 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] staging: vt6655: replace camel case by small case |
| |
On Wed, Aug 02, 2023 at 06:56:35PM +0530, Pavan Bobba wrote: > Replace array name of camel case by small case. Issue found > by checkpatch > > Signed-off-by: Pavan Bobba <opensource206@gmail.com> > --- > drivers/staging/vt6655/baseband.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/vt6655/baseband.c b/drivers/staging/vt6655/baseband.c > index 0e135af8316b..8b913c64845c 100644 > --- a/drivers/staging/vt6655/baseband.c > +++ b/drivers/staging/vt6655/baseband.c > @@ -499,7 +499,7 @@ static const unsigned char by_vt3253_init_tab_rfmd[CB_VT3253_INIT_FOR_RFMD][2] = > }; > > #define CB_VT3253B0_INIT_FOR_RFMD 256 > -static const unsigned char byVT3253B0_RFMD[CB_VT3253B0_INIT_FOR_RFMD][2] = { > +static const unsigned char byvt3253b0_rfmd[CB_VT3253B0_INIT_FOR_RFMD][2] = {
The one before it is called something liek by_vt3253b0_rfmd.
I don't know what by_ or bb_ meant to the driver authors so I can't say if that's a good name, but lets at least keep it consistent and put an underscore between by and vt, by_vt.
regards, dan carpenter
| |