[PATCH 0/2] some simplifications in hw/core/fw-path-provider.c

Daniel Henrique Barboza posted 2 patches 2 years, 2 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220121202952.24763-1-danielhb413@gmail.com
hw/core/fw-path-provider.c    | 14 +++-----------
include/hw/fw-path-provider.h |  2 --
2 files changed, 3 insertions(+), 13 deletions(-)
[PATCH 0/2] some simplifications in hw/core/fw-path-provider.c
Posted by Daniel Henrique Barboza 2 years, 2 months ago
Hi,

These are somes simplifications I found interesting to make while I was
investigating Gitlab #720.

It isn't clear in the MAINTAINERS file who is responsible for this file,
so I'm ccing the QOM maintainers since it's the closest match.


Daniel Henrique Barboza (2):
  hw/core/fw-path-provider: turn fw_path_provider_get_dev_path() static
  hw/core/fw-path-provider.c: open code fw_path_provider_get_dev_path()

 hw/core/fw-path-provider.c    | 14 +++-----------
 include/hw/fw-path-provider.h |  2 --
 2 files changed, 3 insertions(+), 13 deletions(-)

-- 
2.34.1


Re: [PATCH 0/2] some simplifications in hw/core/fw-path-provider.c
Posted by Philippe Mathieu-Daudé via 2 years, 2 months ago
On 21/1/22 21:29, Daniel Henrique Barboza wrote:
> Hi,
> 
> These are somes simplifications I found interesting to make while I was
> investigating Gitlab #720.
> 
> It isn't clear in the MAINTAINERS file who is responsible for this file,
> so I'm ccing the QOM maintainers since it's the closest match.
> 
> 
> Daniel Henrique Barboza (2):
>    hw/core/fw-path-provider: turn fw_path_provider_get_dev_path() static
>    hw/core/fw-path-provider.c: open code fw_path_provider_get_dev_path()

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>