[PATCH] hw/loader: Remove unused rom_add_file_as() definition

Philippe Mathieu-Daudé posted 1 patch 2 years, 7 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210917190136.2246440-1-philmd@redhat.com
include/hw/loader.h | 2 --
1 file changed, 2 deletions(-)
[PATCH] hw/loader: Remove unused rom_add_file_as() definition
Posted by Philippe Mathieu-Daudé 2 years, 7 months ago
rom_add_file_as() is not used anywhere, remove it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 include/hw/loader.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/hw/loader.h b/include/hw/loader.h
index cbfc1848737..c52a1df8b68 100644
--- a/include/hw/loader.h
+++ b/include/hw/loader.h
@@ -329,8 +329,6 @@ void hmp_info_roms(Monitor *mon, const QDict *qdict);
     rom_add_blob(_f, _b, _l, _l, _a, NULL, NULL, NULL, NULL, true)
 #define rom_add_file_mr(_f, _mr, _i)            \
     rom_add_file(_f, NULL, 0, _i, false, _mr, NULL)
-#define rom_add_file_as(_f, _as, _i)            \
-    rom_add_file(_f, NULL, 0, _i, false, NULL, _as)
 #define rom_add_file_fixed_as(_f, _a, _i, _as)          \
     rom_add_file(_f, NULL, _a, _i, false, NULL, _as)
 #define rom_add_blob_fixed_as(_f, _b, _l, _a, _as)      \
-- 
2.31.1

Re: [PATCH] hw/loader: Remove unused rom_add_file_as() definition
Posted by Alistair Francis 2 years, 7 months ago
On Sat, Sep 18, 2021 at 5:02 AM Philippe Mathieu-Daudé
<philmd@redhat.com> wrote:
>
> rom_add_file_as() is not used anywhere, remove it.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Looks fine, although I'm not convinced removing this gets us anything.
It might be used in the future and doesn't add too much cruft.

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  include/hw/loader.h | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/include/hw/loader.h b/include/hw/loader.h
> index cbfc1848737..c52a1df8b68 100644
> --- a/include/hw/loader.h
> +++ b/include/hw/loader.h
> @@ -329,8 +329,6 @@ void hmp_info_roms(Monitor *mon, const QDict *qdict);
>      rom_add_blob(_f, _b, _l, _l, _a, NULL, NULL, NULL, NULL, true)
>  #define rom_add_file_mr(_f, _mr, _i)            \
>      rom_add_file(_f, NULL, 0, _i, false, _mr, NULL)
> -#define rom_add_file_as(_f, _as, _i)            \
> -    rom_add_file(_f, NULL, 0, _i, false, NULL, _as)
>  #define rom_add_file_fixed_as(_f, _a, _i, _as)          \
>      rom_add_file(_f, NULL, _a, _i, false, NULL, _as)
>  #define rom_add_blob_fixed_as(_f, _b, _l, _a, _as)      \
> --
> 2.31.1
>
>