On 22-10-2021 15:07, Karolina Drobnik wrote:
> Drop Hungarian notation in `byRFType` variable in
> `RFvWriteWakeProgSyn` function. Change it to use snake case.
>
> Fix issue detected by checkpatch.pl:
> CHECK: Avoid CamelCase: <byRFType>
>
> Signed-off-by: Karolina Drobnik <karolinadrobnik@xxxxxxxxx>
> ---
> drivers/staging/vt6655/rf.c | 4 ++--
> drivers/staging/vt6655/rf.h | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/vt6655/rf.c b/drivers/staging/vt6655/rf.c
> index 0dd91b0433f6..3f57834901a6 100644
> --- a/drivers/staging/vt6655/rf.c
> +++ b/drivers/staging/vt6655/rf.c
> @@ -676,7 +676,7 @@ bool RFbSelectChannel(struct vnt_private *priv, unsigned char byRFType,
> * Return Value: None.
> *
> */
> -bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char byRFType,
> +bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char rf_type,
> u16 channel)
> {
> void __iomem *iobase = priv->port_offset;
> @@ -685,7 +685,7 @@ bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char byRFType,
> unsigned char sleep_count = 0;
>
> VNSvOutPortW(iobase + MAC_REG_MISCFFNDEX, 0);
> - switch (byRFType) {
> + switch (rf_type) {
> case RF_AIROHA:
> case RF_AL2230S:
>
> diff --git a/drivers/staging/vt6655/rf.h b/drivers/staging/vt6655/rf.h
> index d3f2b8dd6afc..0939937d47a8 100644
> --- a/drivers/staging/vt6655/rf.h
> +++ b/drivers/staging/vt6655/rf.h
> @@ -60,7 +60,7 @@
> bool IFRFbWriteEmbedded(struct vnt_private *priv, unsigned long dwData);
> bool RFbSelectChannel(struct vnt_private *priv, unsigned char byRFType, u16 byChannel);
IMO, probably we can sync RFbSelectChannel as well with similar notation in another patch.
> bool RFbInit(struct vnt_private *priv);
> -bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char byRFType, u16 channel);
> +bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char rf_type, u16 channel);
> bool RFbSetPower(struct vnt_private *priv, unsigned int rate, u16 uCH);
> bool RFbRawSetPower(struct vnt_private *priv, unsigned char byPwr,
> unsigned int rate);
>
Regards,
~Praveen.
