Messages in this thread | | | From | Phillip Potter <> | Date | Sat, 14 Aug 2021 18:12:52 +0100 | Subject | Re: [PATCH 3/4] staging: r8188eu: rename parameter of Hal_GetChnlGroup88E() |
| |
On Fri, 13 Aug 2021 at 08:32, Michael Straube <straube.linux@gmail.com> wrote: > > Rename parameter of Hal_GetChnlGroup88E() to avoid camel case. > pGroup -> group > > Signed-off-by: Michael Straube <straube.linux@gmail.com> > --- > drivers/staging/r8188eu/hal/rtl8188e_hal_init.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/drivers/staging/r8188eu/hal/rtl8188e_hal_init.c b/drivers/staging/r8188eu/hal/rtl8188e_hal_init.c > index d3345fa0eb60..07ba5f0eecc2 100644 > --- a/drivers/staging/r8188eu/hal/rtl8188e_hal_init.c > +++ b/drivers/staging/r8188eu/hal/rtl8188e_hal_init.c > @@ -2060,20 +2060,20 @@ static void Hal_ReadPowerValueFromPROM_8188E(struct txpowerinfo24g *pwrInfo24G, > } > } > > -static void Hal_GetChnlGroup88E(u8 chnl, u8 *pGroup) > +static void Hal_GetChnlGroup88E(u8 chnl, u8 *group) > { > if (chnl < 3) /* Channel 1-2 */ > - *pGroup = 0; > + *group = 0; > else if (chnl < 6) /* Channel 3-5 */ > - *pGroup = 1; > + *group = 1; > else if (chnl < 9) /* Channel 6-8 */ > - *pGroup = 2; > + *group = 2; > else if (chnl < 12) /* Channel 9-11 */ > - *pGroup = 3; > + *group = 3; > else if (chnl < 14) /* Channel 12-13 */ > - *pGroup = 4; > + *group = 4; > else if (chnl == 14) /* Channel 14 */ > - *pGroup = 5; > + *group = 5; > } > > void Hal_ReadPowerSavingMode88E(struct adapter *padapter, u8 *hwinfo, bool AutoLoadFail) > -- > 2.32.0 >
Dear Michael,
Looks good to me, thanks.
Acked-by: Phillip Potter <phil@philpotter.co.uk>
Regards, Phil
| |