mbox

[PULL,0/1] PCI host devaddr property fix for 5.2

Message ID 20201124151500.2945356-1-ehabkost@redhat.com (mailing list archive)
State New, archived
Headers show

Pull-request

git://github.com/ehabkost/qemu.git tags/machine-next-for-5.2-pull-request

Message

Eduardo Habkost Nov. 24, 2020, 3:14 p.m. UTC
The following changes since commit d536d9578ec3ac5029a70b8126cb84bb6f2124a4:

  Merge remote-tracking branch 'remotes/jasowang/tags/net-pull-request' into staging (2020-11-24 10:59:12 +0000)

are available in the Git repository at:

  git://github.com/ehabkost/qemu.git tags/machine-next-for-5.2-pull-request

for you to fetch changes up to 28afbc1f11f5ae33b69deb162a551110717eec94:

  Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()" (2020-11-24 10:06:54 -0500)

----------------------------------------------------------------
PCI host devaddr property fix for 5.2

----------------------------------------------------------------

Michael S. Tsirkin (1):
  Revert "hw/core/qdev-properties: Use qemu_strtoul() in
    set_pci_host_devaddr()"

 hw/core/qdev-properties-system.c | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

Comments

no-reply@patchew.org Nov. 24, 2020, 3:21 p.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20201124151500.2945356-1-ehabkost@redhat.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Message-id: 20201124151500.2945356-1-ehabkost@redhat.com
Subject: [PULL 0/1] PCI host devaddr property fix for 5.2

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/20201124151500.2945356-1-ehabkost@redhat.com -> patchew/20201124151500.2945356-1-ehabkost@redhat.com
Switched to a new branch 'test'
d681dc3 Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()"

=== OUTPUT BEGIN ===
ERROR: consider using qemu_strtoul in preference to strtoul
#42: FILE: hw/core/qdev-properties-system.c:876:
+    val = strtoul(p, &e, 16);

ERROR: consider using qemu_strtoul in preference to strtoul
#51: FILE: hw/core/qdev-properties-system.c:883:
+    val = strtoul(p, &e, 16);

ERROR: consider using qemu_strtoul in preference to strtoul
#61: FILE: hw/core/qdev-properties-system.c:891:
+        val = strtoul(p, &e, 16);

ERROR: consider using qemu_strtoul in preference to strtoul
#73: FILE: hw/core/qdev-properties-system.c:902:
+    val = strtoul(p, &e, 10);

total: 4 errors, 0 warnings, 56 lines checked

Commit d681dc33d138 (Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()") has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20201124151500.2945356-1-ehabkost@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Peter Maydell Nov. 24, 2020, 10:40 p.m. UTC | #2
On Tue, 24 Nov 2020 at 15:16, Eduardo Habkost <ehabkost@redhat.com> wrote:
>
> The following changes since commit d536d9578ec3ac5029a70b8126cb84bb6f2124a4:
>
>   Merge remote-tracking branch 'remotes/jasowang/tags/net-pull-request' into staging (2020-11-24 10:59:12 +0000)
>
> are available in the Git repository at:
>
>   git://github.com/ehabkost/qemu.git tags/machine-next-for-5.2-pull-request
>
> for you to fetch changes up to 28afbc1f11f5ae33b69deb162a551110717eec94:
>
>   Revert "hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()" (2020-11-24 10:06:54 -0500)
>
> ----------------------------------------------------------------
> PCI host devaddr property fix for 5.2
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
for any user-visible changes.

-- PMM