Hi Geert, > > + u32 dma_irqs = INFO1_DTRANEND0 | > > + (priv->quirks && priv->quirks->old_info1_layout ? > > + INFO1_DTRANEND1_OLD : INFO1_DTRANEND1); > > Perhaps it makes sense to store the dma_irqs mask in priv->quirks, > or even in priv, to simplify this code (repeated below)? I tried that, yet didn't find it prettier. I am not strict here, though, so can change if desired. Thanks for the review! Wolfram
Attachment:
signature.asc
Description: PGP signature