Re: [PATCH v4 2/2] usb: dwc2: host: Clear interrupts before handling them

2015-11-20 Thread Doug Anderson
Felipe, On Fri, Nov 20, 2015 at 7:40 AM, Felipe Balbi wrote: > > Hi, > > Douglas Anderson writes: >> In general it is wise to clear interrupts before processing them. If >> you don't do that, you can get: >> 1. Interrupt happens >> 2. You look at system

Re: [PATCH v4 2/2] usb: dwc2: host: Clear interrupts before handling them

2015-11-20 Thread Felipe Balbi
Hi, Douglas Anderson writes: > In general it is wise to clear interrupts before processing them. If > you don't do that, you can get: > 1. Interrupt happens > 2. You look at system state and process interrupt > 3. A new interrupt happens > 4. You clear interrupt

Re: [PATCH v4 2/2] usb: dwc2: host: Clear interrupts before handling them

2015-11-20 Thread Doug Anderson
Hi, On Fri, Nov 20, 2015 at 8:49 AM, Doug Anderson wrote: > Felipe, > > On Fri, Nov 20, 2015 at 7:40 AM, Felipe Balbi wrote: >> >> Hi, >> >> Douglas Anderson writes: >>> In general it is wise to clear interrupts before processing

[PATCH v4 2/2] usb: dwc2: host: Clear interrupts before handling them

2015-11-19 Thread Douglas Anderson
In general it is wise to clear interrupts before processing them. If you don't do that, you can get: 1. Interrupt happens 2. You look at system state and process interrupt 3. A new interrupt happens 4. You clear interrupt without processing it. This patch was actually a first attempt to fix