diff options
| author | Andrew Jeffery <[email protected]> | 2025-06-16 13:13:46 +0000 |
|---|---|---|
| committer | Andrew Jeffery <[email protected]> | 2025-07-08 02:05:07 +0000 |
| commit | 4483e3c481bd2d026813434f3cd93381386cd1fa (patch) | |
| tree | dedaab3e04a015bcfdf1e4b61854d53816fd9b0e /rust/helpers/helpers.c | |
| parent | soc: aspeed: lpc-snoop: Use dev_err_probe() where possible (diff) | |
| download | kernel-4483e3c481bd2d026813434f3cd93381386cd1fa.tar.gz kernel-4483e3c481bd2d026813434f3cd93381386cd1fa.zip | |
soc: aspeed: lpc-snoop: Consolidate channel initialisation
Previously, channel initialisation was a bit perilous with respect to
resource cleanup in error paths. While the implementation had issues,
it at least made an effort to eliminate some of its problems by first
testing whether any channels were enabled, and bailing out if not.
Having improved the robustness of resource handling in probe() we can
now rearrange the initial channel test to be located with the subsequent
test, and rework the unrolled conditional logic to use a loop for an
improvement in readability.
Link: https://patch.msgid.link/20250616-aspeed-lpc-snoop-fixes-v2-9-3cdd59c934d3@codeconstruct.com.au
Acked-by: Jean Delvare <[email protected]>
Signed-off-by: Andrew Jeffery <[email protected]>
Diffstat (limited to 'rust/helpers/helpers.c')
0 files changed, 0 insertions, 0 deletions