linux-kernel - Re: [PATCH 2/2] USB: at91: using USBA_NR

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <53040592.2000707@atmel.com>
Date:	Wed, 19 Feb 2014 09:14:58 +0800
From:	Bo Shen <voice.shen@...el.com>
To:	<balbi@...com>
CC:	<nicolas.ferre@...el.com>, <plagnioj@...osoft.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	<linux-arm-kernel@...ts.infradead.org>,
	<linux-usb@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 2/2] USB: at91: using USBA_NR_DMAS for DMA channels

Hi Felipe Balbi,

On 02/19/2014 12:19 AM, Felipe Balbi wrote:
> On Fri, Jan 17, 2014 at 10:59:25AM +0800, Bo Shen wrote:
>> When the SoC is earlier than sama5d3 SoC, which have the same number
>> endpoints and DMAs. However for sama5d3 SoC, it has different number
>> for endpoints and DMAs. So, define USBA_NR_DMAs for DMA channels
>>
>> Signed-off-by: Bo Shen <voice.shen@...el.com>
>> ---
>>
>>   drivers/usb/gadget/atmel_usba_udc.c | 2 +-
>>   drivers/usb/gadget/atmel_usba_udc.h | 2 +-
>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/usb/gadget/atmel_usba_udc.c b/drivers/usb/gadget/atmel_usba_udc.c
>> index 7e67a81..5cded1c 100644
>> --- a/drivers/usb/gadget/atmel_usba_udc.c
>> +++ b/drivers/usb/gadget/atmel_usba_udc.c
>> @@ -1661,7 +1661,7 @@ static irqreturn_t usba_udc_irq(int irq, void *devid)
>>   	if (dma_status) {
>>   		int i;
>>
>> -		for (i = 1; i < USBA_NR_ENDPOINTS; i++)
>> +		for (i = 1; i < USBA_NR_DMAS; i++)
>>   			if (dma_status & (1 << i))
>>   				usba_dma_irq(udc, &udc->usba_ep[i]);
>>   	}
>> diff --git a/drivers/usb/gadget/atmel_usba_udc.h b/drivers/usb/gadget/atmel_usba_udc.h
>> index 2922db5..a70706e 100644
>> --- a/drivers/usb/gadget/atmel_usba_udc.h
>> +++ b/drivers/usb/gadget/atmel_usba_udc.h
>> @@ -210,7 +210,7 @@
>>   #define USBA_FIFO_BASE(x)	((x) << 16)
>>
>>   /* Synth parameters */
>> -#define USBA_NR_ENDPOINTS	7
>> +#define USBA_NR_DMAS		7
>
> what's the difference ? You just renamed this macro. Also, please
> clarify a bit your commit log.

As commit message said, the SoC before sama5d3, the endpoint number is 
the same as DMA channel number, so use endpoints definition for DMA 
channel number, however after sama5d3, the endpoints is not the same as 
DMA channel, so use DMA micro for DMA channels.

Best Regards,
Bo Shen


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/