Re: [Outreachy kernel] Re: [PATCH v2 1/2] staging: kpc2000: kpc_dma: rearrange lines exceeding 100 columns

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 26, 2020 at 09:34:53AM +0530, Deepak R Varma wrote:
> > -		dev_dbg(&eng->pldev->dev, "Handling completed descriptor %p (acd = %p)\n", cur, cur->acd);
> > +		dev_dbg(&eng->pldev->dev, "Handling completed descriptor %p (acd = %p)\n",
> > +			cur,
> > +			cur->acd);

Why do you put 'cur' and 'cur->acd' on different lines?

> > -	rv = request_irq(eng->irq, ndd_irq_handler, IRQF_SHARED, KP_DRIVER_NAME_DMA_CONTROLLER, eng);
> > +	rv = request_irq(eng->irq,
> > +			 ndd_irq_handler,
> > +			 IRQF_SHARED,
> > +			 KP_DRIVER_NAME_DMA_CONTROLLER,
> > +			 eng);

Likewise.  I'd do:

	rv = request_irq(eng->irq, ndd_irq_handler, IRQF_SHARED,
			KP_DRIVER_NAME_DMA_CONTROLLER, eng);

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux