]> git.hungrycats.org Git - linux/commit
ASoC: rockchip: Drop problematic guard() changes
authorNicolas Frattaroli <nicolas.frattaroli@collabora.com>
Wed, 17 Jun 2026 11:46:04 +0000 (13:46 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 17 Jun 2026 13:43:43 +0000 (14:43 +0100)
commit1a3c8e28959790ae3e06029681418278b7821a3c
treeb0c85c6c63cb82d917a8c5fdb7eb264baa2a2e60
parent2e9261761b35f0b67b7487688cd1365f535be0b3
ASoC: rockchip: Drop problematic guard() changes

This reverts commit f7fe9f707360 ("ASoC: rockchip: rockchip_sai: Use
guard() for spin locks").

This is very noisy pointless churn that was not tested by the submitter,
nor was it addressed to the driver's maintainer. It mixes unrelated
whitespace changes (eliminating the blank line between the includes -
why?) with hard to review diffs that add a whole indentation level to
the function for no benefit, while also not following kernel code style
by doing stuff like "ret == 0".

The driver is better off without these changes, and they're not worth
the time to validate whether they really do make no functional changes.

Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Link: https://patch.msgid.link/20260617-sai-revert-v1-1-e46adda2213b@collabora.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/rockchip/rockchip_sai.c