RE: [PATCH 6/6] musb: dma: use optimal transfer element for sdma

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

 



Hi,
> Ajay Kumar Gupta wrote:
> 
> > Use optimal values of transfer element based on buffer address in system
> > DMA programming. This would improve the performance.
> >
> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@xxxxxx>
> > ---
> >  drivers/usb/musb/musbhsdma.c |   29 ++++++++++++++++++++++++++---
> >  1 files changed, 26 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/usb/musb/musbhsdma.c b/drivers/usb/musb/musbhsdma.c
> > index d29e487..39c1801 100644
> > --- a/drivers/usb/musb/musbhsdma.c
> > +++ b/drivers/usb/musb/musbhsdma.c
> > @@ -52,11 +52,34 @@ static void musb_sdma_channel_program(struct musb
> *musb,
> >  		struct musb_dma_channel *musb_channel,
> >  		dma_addr_t dma_addr, u32 len)
> >  {
> > +	u16 frame = len;
> > +	int data_type = OMAP_DMA_DATA_TYPE_S8;
> > +
> > +	switch (dma_addr & 0x3) {
> > +	case 0:
> > +		if ((len % 4) == 0) {
> > +			data_type = OMAP_DMA_DATA_TYPE_S32;
> > +			frame = len / 4;
> > +			break;
> > +		}
> > +	case 2:
> > +		if ((len % 2) == 0) {
> > +			data_type = OMAP_DMA_DATA_TYPE_S16;
> > +			frame = len / 2;
> > +			break;
> > +		}
> > +	case 1:
> > +	case 3:
> > +	default:
> > +		data_type = OMAP_DMA_DATA_TYPE_S8;
> > +		frame = len;
> > +			break;
> 
>     The *break* is overindented.
Ok.

-Ajay
> 
> WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux