diff options
| author | Jamin Lin <jamin_lin@aspeedtech.com> | 2025-09-25 13:05:31 +0800 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2025-09-29 18:00:20 +0200 |
| commit | 8d3aaefdb3bb13df7c57091a4319f3afe3c351ac (patch) | |
| tree | 26f567d1779e2e0dbdc8c8ea8cdefca5d9b31e80 /scripts/lib/kdoc/kdoc_files.py | |
| parent | ecabf8ba94961a23f473e7060f8589e93967cfd2 (diff) | |
| download | qemu-8d3aaefdb3bb13df7c57091a4319f3afe3c351ac.zip qemu-8d3aaefdb3bb13df7c57091a4319f3afe3c351ac.tar.gz qemu-8d3aaefdb3bb13df7c57091a4319f3afe3c351ac.tar.bz2 | |
hw/arm/aspeed_ast27x0-fc: Drop dead return checks
1. object_property_set_link() can return false only when it fails, and it
sets an error when it fails. Since passing &error_abort causes an abort,
the function never returns false, and the return statement is effectively
dead code.
2. object_property_set_int() is considered as a routine which shouldn't fail.
So the common practice in models is to pass &error_abort and ignore the returned value.
https://patchwork.kernel.org/project/qemu-devel/patch/20250717034054.1903991-3-jamin_lin@aspeedtech.com/#26540626
No functional change.
Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Link: https://lore.kernel.org/qemu-devel/20250925050535.2657256-6-jamin_lin@aspeedtech.com
Signed-off-by: Cédric Le Goater <clg@redhat.com>
Diffstat (limited to 'scripts/lib/kdoc/kdoc_files.py')
0 files changed, 0 insertions, 0 deletions