[PATCH 0/2] hw/mem/pc-dimm: Trivial code changes

Wainer dos Santos Moschetta posted 2 patches 5 years, 8 months ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200310180510.19489-1-wainersm@redhat.com
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>
hw/mem/pc-dimm.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
[PATCH 0/2] hw/mem/pc-dimm: Trivial code changes
Posted by Wainer dos Santos Moschetta 5 years, 8 months ago
Improve one error message and fix one code style warning.

Wainer dos Santos Moschetta (2):
  hw/mem/pc-dimm: Print slot number on error at pc_dimm_pre_plug()
  hw/mem/pc-dimm: Fix line over 80 characters warning

 hw/mem/pc-dimm.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

-- 
2.24.1


Re: [PATCH 0/2] hw/mem/pc-dimm: Trivial code changes
Posted by Laurent Vivier 5 years, 6 months ago
Le 10/03/2020 à 19:05, Wainer dos Santos Moschetta a écrit :
> Improve one error message and fix one code style warning.
> 
> Wainer dos Santos Moschetta (2):
>   hw/mem/pc-dimm: Print slot number on error at pc_dimm_pre_plug()
>   hw/mem/pc-dimm: Fix line over 80 characters warning
> 
>  hw/mem/pc-dimm.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 

Applied to my trivial-patches branch.

Thanks,
Laurent

Re: [PATCH 0/2] hw/mem/pc-dimm: Trivial code changes
Posted by Stefano Garzarella 5 years, 8 months ago
On Tue, Mar 10, 2020 at 03:05:08PM -0300, Wainer dos Santos Moschetta wrote:
> Improve one error message and fix one code style warning.
> 
> Wainer dos Santos Moschetta (2):
>   hw/mem/pc-dimm: Print slot number on error at pc_dimm_pre_plug()
>   hw/mem/pc-dimm: Fix line over 80 characters warning
> 
>  hw/mem/pc-dimm.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>