mbox series

[v2,0/3] fw_cfg: fix boot bootsplash and reboot-timeout error checking

Message ID 1542777026-2788-1-git-send-email-liq3ea@gmail.com
Headers show
Series fw_cfg: fix boot bootsplash and reboot-timeout error checking | expand

Message

Li Qiang Nov. 21, 2018, 5:10 a.m. UTC 
And also do some code cleanup.
A lot of thanks to Markus's review and advice.

v2: fix some small issue per Markus's review.

Li Qiang (3):
  fw_cfg: fix -boot bootsplash error checking
  fw_cfg: fix -boot reboot-timeout error checking
  fw_cfg: make qemu_extra_params_fw locally

 hw/nvram/fw_cfg.c       | 63 +++++++++++++++++++++----------------------------
 include/sysemu/sysemu.h |  1 -
 vl.c                    |  4 ++--
 3 files changed, 29 insertions(+), 39 deletions(-)

Comments

Gerd Hoffmann Dec. 10, 2018, 2:33 p.m. UTC | #1 
On Tue, Nov 20, 2018 at 09:10:23PM -0800, Li Qiang wrote:
> And also do some code cleanup.
> A lot of thanks to Markus's review and advice.
> 
> v2: fix some small issue per Markus's review.
> 
> Li Qiang (3):
>   fw_cfg: fix -boot bootsplash error checking
>   fw_cfg: fix -boot reboot-timeout error checking
>   fw_cfg: make qemu_extra_params_fw locally

Series is
Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
Philippe Mathieu-Daudé Dec. 11, 2018, 4:19 p.m. UTC | #2 
On 11/21/18 6:10 AM, Li Qiang wrote:
> And also do some code cleanup.
> A lot of thanks to Markus's review and advice.
> 
> v2: fix some small issue per Markus's review.
> 
> Li Qiang (3):
>   fw_cfg: fix -boot bootsplash error checking
>   fw_cfg: fix -boot reboot-timeout error checking
>   fw_cfg: make qemu_extra_params_fw locally
> 
>  hw/nvram/fw_cfg.c       | 63 +++++++++++++++++++++----------------------------
>  include/sysemu/sysemu.h |  1 -
>  vl.c                    |  4 ++--
>  3 files changed, 29 insertions(+), 39 deletions(-)
> 

Thanks, I've queued this series for when 4.0 opens up.

Regards,

Phil.