On Mon, Oct 04, 2021 at 09:55:11AM +0300, Dan Carpenter wrote: > On Sun, Oct 03, 2021 at 04:57:51PM -0300, Gaston Gonzalez wrote: > > Remove unnecessary blank lines. > > > > Reported by checkpatch.pl > > > > Signed-off-by: Gaston Gonzalez <gascoar@xxxxxxxxx> > > --- > > [ snip ] > > > @@ -3207,7 +3206,6 @@ enum vchiq_status vchiq_bulk_transfer(unsigned int handle, > > } > > > > state = service->state; > > - > > queue = (dir == VCHIQ_BULK_TRANSMIT) ? > > &service->bulk_tx : &service->bulk_rx; > > > > > I don't care one way or the other about the blank line, but it's > not a checkpatch warning. > Thanks for the review Dan. Of course you are right... This was not supposed to be here, I'm sorry. Please drop the series, I'll a send a v2. Thanks, Gaston > regards, > dan carpenter >