Re: [Outreachy kernel] [PATCH v2 3/5] staging: media: bcm2048: remove unnecessary blank lines

[Sudip Mukherjee <sudipm.mukherjee@gmail.com>]

On Fri, Oct 30, 2015 at 01:21:05PM -0700, Amarjargal Gundjalam wrote:
> This patch fixes the checkpatch issues:
> 
> CHECK: Please don't use multiple blank lines
> CHECK: Blank lines aren't necessary after an open brace '{'
> 
> Signed-off-by: Amarjargal Gundjalam <amarjargal.gundjalam@gmail.com>
> ---
>  drivers/staging/media/bcm2048/radio-bcm2048.c |   10 +---------
>  1 file changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c
> index e65c83c..f8578fb 100644
> --- a/drivers/staging/media/bcm2048/radio-bcm2048.c
> +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
> @@ -179,7 +179,6 @@
>  #define BCM2048_DEFAULT_TIMEOUT		1500
>  #define BCM2048_AUTO_SEARCH_TIMEOUT	3000
>  
> -
>  #define BCM2048_FREQDEV_UNIT		10000
>  #define BCM2048_FREQV4L2_MULTI		625
>  #define dev_to_v4l2(f)	((f * BCM2048_FREQDEV_UNIT) / BCM2048_FREQV4L2_MULTI)
> @@ -1436,10 +1435,8 @@ static void bcm2048_parse_rds_pi(struct bcm2048_device *bdev)
>  	u16 pi;
>  
>  	for (i = 0; i < bdev->fifo_size; i += BCM2048_RDS_FIFO_DUPLE_SIZE) {
> -
>  		/* Block A match, only data without crc errors taken */
>  		if (bdev->rds_info.radio_text[i] == BCM2048_RDS_BLOCK_A) {
> -
>  			pi = (bdev->rds_info.radio_text[i + 1] << 8) +
>  				bdev->rds_info.radio_text[i + 2];
>  
> @@ -1494,8 +1491,7 @@ static int bcm2048_parse_rt_match_b(struct bcm2048_device *bdev, int i)
>  		return -EIO;
>  
>  	if ((bdev->rds_info.radio_text[i] & BCM2048_RDS_BLOCK_MASK) ==
> -		BCM2048_RDS_BLOCK_B) {
> -
> +	    BCM2048_RDS_BLOCK_B) {

This is an unrelated change from your commit message. This should have
been included in your 1/5 patch.

regards
sudip