Messages in this thread | | | Date | Thu, 25 Jul 2019 12:55:02 +0300 | From | Peter De Schrijver <> | Subject | Re: [PATCH V6 01/21] irqchip: tegra: Do not disable COP IRQ during suspend |
| |
On Mon, Jul 22, 2019 at 12:54:51PM +0300, Dmitry Osipenko wrote: > > All Tegra SoCs support SC7, hence the 'supports_sc7' and the comment > doesn't sound correct to me. Something like 'firmware_sc7' should suit > better here. > > > + writel_relaxed(~0ul, ictlr + ICTLR_COP_IER_CLR); > > Secondly, I'm also not sure why COP interrupts need to be disabled for > pre-T210 at all, since COP is unused. This looks to me like it was > cut-n-pasted from downstream kernel without a good reason and could be > simply removed.
I don't think we can rely on the fact that COP is unused. People can write their own code to run on COP.
Peter.
| |