On Mon, Feb 17, 2025 at 04:00:37PM +0530, Ravi Kumar kairi wrote: > From: Ravi Kumar Kairi <kumarkairiravi@xxxxxxxxx> > > removed few empty lines that were causing checkpatch script warnings > > Signed-off-by: Ravi Kumar Kairi <kumarkairiravi@xxxxxxxxx> > --- > drivers/staging/gpib/agilent_82350b/agilent_82350b.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/staging/gpib/agilent_82350b/agilent_82350b.c b/drivers/staging/gpib/agilent_82350b/agilent_82350b.c > index 7936c7285d..b6871eb3c8 100644 > --- a/drivers/staging/gpib/agilent_82350b/agilent_82350b.c > +++ b/drivers/staging/gpib/agilent_82350b/agilent_82350b.c > @@ -176,7 +176,6 @@ static int agilent_82350b_accel_write(gpib_board_t *board, uint8_t *buffer, size > > event_status = read_and_clear_event_status(board); > > - > #ifdef EXPERIMENTAL > pr_info("ag_ac_wr: wait for previous BO to complete if any\n"); > retval = wait_event_interruptible(board->wait, > @@ -193,8 +192,6 @@ static int agilent_82350b_accel_write(gpib_board_t *board, uint8_t *buffer, size > *bytes_written += num_bytes; > if (retval < 0) > return retval; > - > - Why did you remove both lines here? One is fine. And your subject line needs an extra ' ' in it, right? thanks, greg k-h