• Arnd Bergmann's avatar
    regulator: core: fix regulator_lock_supply regression · bb41897e
    Arnd Bergmann authored
    As noticed by Geert Uytterhoeven, my patch to avoid a harmless build warning
    in regulator_lock_supply() was total crap and introduced a real bug:
    
    > [ BUG: bad unlock balance detected! ]
    > kworker/u4:0/6 is trying to release lock (&rdev->mutex) at:
    > [<c0247b84>] regulator_set_voltage+0x38/0x50
    
    we still lock the regulator supplies, but not the actual regulators,
    so we are missing a lock, and the unlock is unbalanced.
    
    This rectifies it by first locking the regulator device itself before
    using the same loop as before to lock its supplies.
    Reported-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Fixes: 716fec9d1965 ("[SUBMITTED] regulator: core: avoid unused variable warning")
    Signed-off-by: default avatarMark Brown <broonie@kernel.org>
    bb41897e
core.c 114 KB