Hi Jonathon, Just one example. TXFIFOEMPTY interrupt can be asserted in both modes: Slave and DMA, but this interrupt handling required for Slave mode only and should be ignored for DMA modes. This is why this interrupt masked for DMA and unmasked for Slave mode. Thanks, Minas On 12/19/2022 8:15 PM, Jonathon Reinhart <jrreinhart@xxxxxxxxxx> wrote: >From: Jonathon Reinhart <jrreinhart@xxxxxxxxxx> >Sent: Monday, December 19, 2022 8:15 PM >To: linux-usb@xxxxxxxxxxxxxxx >Cc: Vardan Mikayelyan <mvardan@xxxxxxxxxxxx>; John Keeping ><john@xxxxxxxxxxxx>; John Youn <johnyoun@xxxxxxxxxxxx>; Felipe Balbi ><felipe.balbi@xxxxxxxxxxxxxxx> >Subject: dwc2: Application of DIEPMSK when reading DIEPINT > >Hello, > >326015887b6a: usb: dwc2: gadget: Add dwc2_gadget_read_ep_interrupts function > >Prior to this commit, the DIEPINT / DOEPINT interrupt status registers were >directly read and written-back to clear. But after this commit, the register >value is masked with DIEPMSK / DOEPMSK prior to clearing or being returned >to act upon. > >Can anyone explain the purpose of this change? In what case will the *INT >register have bits set which are not set in *MSK (and will be subsequently >ignored)? Why should those be ignored? > >Thank you, >Jonathon Reinhart
