]> www.infradead.org Git - users/jedix/linux-maple.git/commit
soc: qcom: pmic_glink: Drop redundant pg assignment before taking lock
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Sat, 11 Jan 2025 17:23:26 +0000 (18:23 +0100)
committerBjorn Andersson <andersson@kernel.org>
Wed, 5 Feb 2025 03:16:01 +0000 (21:16 -0600)
commit1ac98d6484c4a53e9b3ed1f59d03a2dd2d287b70
tree6e79f6e069e24ad0c769e5b2907335b641782734
parent2014c95afecee3e76ca4a56956a936e23283f05b
soc: qcom: pmic_glink: Drop redundant pg assignment before taking lock

Commit e9f826b0459f ("soc: qcom: pmic_glink: simplify locking with
guard()") was on top of a fix [1] which was moving the 'pg =
__pmic_glink' assignment into the critical section.  Unfortunately the
actual fix was not applied and instead rebased version of the next patch
got in.  The resulting code is in general correct, but now there is a
duplicated assignment 'pg = __pmic_glink'.

[1] https://lore.kernel.org/all/20240822164815.230167-1-krzysztof.kozlowski@linaro.org/

Fixes: e9f826b0459f ("soc: qcom: pmic_glink: simplify locking with guard()")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20250111172326.101779-1-krzysztof.kozlowski@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/soc/qcom/pmic_glink.c