Hi Marc, Am Samstag, 10. Februar 2018, 15:35:01 CET schrieb Marc Zyngier: > The rockchip DRM driver is quite careful to disable interrupts > when taking a lock that is also taken in interrupt context, > which is a good thing. > > What is a bit over the top is to use spin_lock_irqsave when > already in interrupt context, as you cannot take another > interrupt again, and disabling interrupt is just pure > overhead. > > Switching to the non _irqsave version in interrupt context is > more logical, and less heavy handed. > > Signed-off-by: Marc Zyngier <marc.zyngier at arm.com> please note, that we had a maintainer swap for the Rockchip drm-component, where Sandy replaced Mark [0] ... with me acting as sort-of (and not yet up to speed) backup. So I guess Rockchip drm patches should also include Sandy Huang <hjc at rock-chips.com> Heiko [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=b415b79529391abd29b6a30b93697821f01e082f
