staging: rtl8723bs: hal: Remove unnecessary blank lines in hal_com.c

[PATCH] staging: rtl8723bs: hal: Remove unnecessary blank lines in hal_com.c

Posted by Praveen Jayaprakash Pattar 1 week ago

Remove multiple consecutive blank lines that violate kernel coding style.
Detected by checkpatch.pl with --strict flag.

Signed-off-by: Praveen Jayaprakash Pattar <praveen.pattar2022@gmail.com>
---
 drivers/staging/rtl8723bs/hal/hal_com.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index 728a2171f..de1fe8a98 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -30,7 +30,6 @@ void rtw_hal_data_deinit(struct adapter *padapter)
 	}
 }
 
-
 void dump_chip_info(struct hal_version	chip_version)
 {
 	char buf[128];
@@ -71,7 +70,6 @@ void dump_chip_info(struct hal_version	chip_version)
 	cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "RomVer(%d)\n", chip_version.ROMVer);
 }
 
-
 #define	EEPROM_CHANNEL_PLAN_BY_HW_MASK	0x80
 
 /*
@@ -378,7 +376,6 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
 
 	} else { /* typical setting */
 
-
 		/* BK,	BE,	VI,	VO,	BCN,	CMD, MGT, HIGH, HCCA */
 		/*   1,		1,	0,	0,	0,	0,	0,	0,		0	}; */
 		/* 0:ep_0 num, 1:ep_1 num */
@@ -419,7 +416,6 @@ static void _ThreeOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
 
 	} else { /* typical setting */
 
-
 		/* 	BK,	BE,	VI,	VO,	BCN,	CMD, MGT, HIGH, HCCA */
 		/*   2,		2,	1,	0,	0,	0,	0,	0,		0	}; */
 		/* 0:H, 1:N, 2:L */
@@ -751,7 +747,6 @@ void SetHalODMVar(
 	}
 }
 
-
 bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8 *pInt)
 {
 	u16 i = 0;
-- 
2.54.0

Re: [PATCH] staging: rtl8723bs: hal: Remove unnecessary blank lines in hal_com.c

Posted by Greg KH 3 days, 19 hours ago

On Sun, May 17, 2026 at 10:45:37PM +0530, Praveen Jayaprakash Pattar wrote:
> Remove multiple consecutive blank lines that violate kernel coding style.
> Detected by checkpatch.pl with --strict flag.
> 
> Signed-off-by: Praveen Jayaprakash Pattar <praveen.pattar2022@gmail.com>
> ---
>  drivers/staging/rtl8723bs/hal/hal_com.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
> index 728a2171f..de1fe8a98 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_com.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_com.c
> @@ -30,7 +30,6 @@ void rtw_hal_data_deinit(struct adapter *padapter)
>  	}
>  }
>  
> -
>  void dump_chip_info(struct hal_version	chip_version)
>  {
>  	char buf[128];
> @@ -71,7 +70,6 @@ void dump_chip_info(struct hal_version	chip_version)
>  	cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "RomVer(%d)\n", chip_version.ROMVer);
>  }
>  
> -
>  #define	EEPROM_CHANNEL_PLAN_BY_HW_MASK	0x80
>  
>  /*
> @@ -378,7 +376,6 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>  
>  	} else { /* typical setting */
>  
> -
>  		/* BK,	BE,	VI,	VO,	BCN,	CMD, MGT, HIGH, HCCA */
>  		/*   1,		1,	0,	0,	0,	0,	0,	0,		0	}; */
>  		/* 0:ep_0 num, 1:ep_1 num */
> @@ -419,7 +416,6 @@ static void _ThreeOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>  
>  	} else { /* typical setting */
>  
> -
>  		/* 	BK,	BE,	VI,	VO,	BCN,	CMD, MGT, HIGH, HCCA */
>  		/*   2,		2,	1,	0,	0,	0,	0,	0,		0	}; */
>  		/* 0:H, 1:N, 2:L */
> @@ -751,7 +747,6 @@ void SetHalODMVar(
>  	}
>  }
>  
> -
>  bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8 *pInt)
>  {
>  	u16 i = 0;
> -- 
> 2.54.0
> 
>

Someone else sent this same change right before you did, sorry.