On 11/8/2016 1:16 AM, Felipe Balbi wrote:
> 
> Hi,
> 
> John Youn <johny...@synopsys.com> writes:
>> From: Vahram Aharonyan <vahr...@synopsys.com>
>>
>> Update endpoint related register descriptions to add DOEPMSK_BNAMSK.
>>
>> Add BNA interrupt's unmasking during gadget initalization if descriptor
> 
> what does BNA stand for?

Buffer not available. I'll clarify in commit message.

> 
>> DMA mode is selected.
>>
>> Signed-off-by: Vahram Aharonyan <vahr...@synopsys.com>
>> Signed-off-by: John Youn <johny...@synopsys.com>
>> ---
>>  drivers/usb/dwc2/gadget.c | 7 +++++++
>>  drivers/usb/dwc2/hw.h     | 1 +
>>  2 files changed, 8 insertions(+)
>>
>> diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c
>> index 4013518..3ab4d60 100644
>> --- a/drivers/usb/dwc2/gadget.c
>> +++ b/drivers/usb/dwc2/gadget.c
>> @@ -2316,6 +2316,9 @@ static void dwc2_hsotg_epint(struct dwc2_hsotg *hsotg, 
>> unsigned int idx,
>>      if (ints & DXEPINT_BACK2BACKSETUP)
>>              dev_dbg(hsotg->dev, "%s: B2BSetup/INEPNakEff\n", __func__);
>>  
>> +    if (ints & DXEPINT_BNAINTR)
>> +            dev_dbg(hsotg->dev, "%s: BNA interrupt\n", __func__);
> 
> Why are you enabling interrupts *just* to print out a debugging message?
> Is it really worth the CPU time?
> 

It's implemented later. I'll see if I can reorganize these patches a
bit.

John

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

Reply via email to