On Thu, Dec 14, 2023 at 05:58:13PM +0800, Kent Gibson wrote: > Reduce the time holding the gpio_lock by snapshotting the desc flags, > rather than testing them individually while holding the lock. > > Accept that the calculation of the used field is inherently racy, and > only check the availability of the line from pinctrl if other checks > pass, so avoiding the check for lines that are otherwise in use. ... > - spin_lock_irqsave(&gpio_lock, flags); Shouldn't this be covered by patch 1 (I mean conversion to scoped_guard() instead of spinlock)? -- With Best Regards, Andy Shevchenko