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 /hw/arm/aspeed.c | |
| parent | ecabf8ba94961a23f473e7060f8589e93967cfd2 (diff) | |
| download | focaccia-qemu-8d3aaefdb3bb13df7c57091a4319f3afe3c351ac.tar.gz focaccia-qemu-8d3aaefdb3bb13df7c57091a4319f3afe3c351ac.zip | |
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 'hw/arm/aspeed.c')
0 files changed, 0 insertions, 0 deletions