mbox series

[v2,0/2] hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value

Message ID 20200721131911.27380-1-philmd@redhat.com
Headers show
Series hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value | expand

Message

Philippe Mathieu-Daudé July 21, 2020, 1:19 p.m. UTC 
Since v1:
- Addressed Markus review comments
- Added patch to clarify fw_cfg_add_from_generator (Markus)

Philippe Mathieu-Daudé (2):
  hw/nvram/fw_cfg: Simplify fw_cfg_add_from_generator() error
    propagation
  hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value

 include/hw/nvram/fw_cfg.h |  8 ++++++--
 hw/nvram/fw_cfg.c         | 13 +++++++------
 softmmu/vl.c              |  6 +-----
 3 files changed, 14 insertions(+), 13 deletions(-)

Comments

Philippe Mathieu-Daudé July 21, 2020, 2:52 p.m. UTC | #1 
On 7/21/20 3:19 PM, Philippe Mathieu-Daudé wrote:
> Since v1:
> - Addressed Markus review comments
> - Added patch to clarify fw_cfg_add_from_generator (Markus)
> 
> Philippe Mathieu-Daudé (2):
>   hw/nvram/fw_cfg: Simplify fw_cfg_add_from_generator() error
>     propagation
>   hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value
> 
>  include/hw/nvram/fw_cfg.h |  8 ++++++--
>  hw/nvram/fw_cfg.c         | 13 +++++++------
>  softmmu/vl.c              |  6 +-----
>  3 files changed, 14 insertions(+), 13 deletions(-)
> 

Thanks for the reviews, patches applied to fw_cfg-next tree.

Phil.