[PATCH v4 0/3] Add npcm7xx emc model

Doug Evans via posted 3 patches 4 years, 9 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210213002520.1374134-1-dje@google.com
Maintainers: Havard Skinnemoen <hskinnemoen@google.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Tyrone Ting <kfting@nuvoton.com>, Thomas Huth <thuth@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Jason Wang <jasowang@redhat.com>
There is a newer version of this series
docs/system/arm/nuvoton.rst    |   3 +-
hw/arm/npcm7xx.c               |  50 +-
hw/net/meson.build             |   1 +
hw/net/npcm7xx_emc.c           | 857 ++++++++++++++++++++++++++++++++
hw/net/trace-events            |  17 +
include/hw/arm/npcm7xx.h       |   2 +
include/hw/net/npcm7xx_emc.h   | 286 +++++++++++
tests/qtest/meson.build        |   1 +
tests/qtest/npcm7xx_emc-test.c | 862 +++++++++++++++++++++++++++++++++
9 files changed, 2076 insertions(+), 3 deletions(-)
create mode 100644 hw/net/npcm7xx_emc.c
create mode 100644 include/hw/net/npcm7xx_emc.h
create mode 100644 tests/qtest/npcm7xx_emc-test.c
[PATCH v4 0/3] Add npcm7xx emc model
Posted by Doug Evans via 4 years, 9 months ago
This is a 10/100 ethernet device that has several features.
Only the ones needed by the Linux driver have been implemented.
See npcm7xx_emc.c for a list of unimplemented features.

Doug Evans (3):
  hw/net: Add npcm7xx emc model
  hw/arm: Add npcm7xx emc model
  tests/qtests: Add npcm7xx emc model test

 docs/system/arm/nuvoton.rst    |   3 +-
 hw/arm/npcm7xx.c               |  50 +-
 hw/net/meson.build             |   1 +
 hw/net/npcm7xx_emc.c           | 857 ++++++++++++++++++++++++++++++++
 hw/net/trace-events            |  17 +
 include/hw/arm/npcm7xx.h       |   2 +
 include/hw/net/npcm7xx_emc.h   | 286 +++++++++++
 tests/qtest/meson.build        |   1 +
 tests/qtest/npcm7xx_emc-test.c | 862 +++++++++++++++++++++++++++++++++
 9 files changed, 2076 insertions(+), 3 deletions(-)
 create mode 100644 hw/net/npcm7xx_emc.c
 create mode 100644 include/hw/net/npcm7xx_emc.h
 create mode 100644 tests/qtest/npcm7xx_emc-test.c

-- 
2.30.0.478.g8a0d178c01-goog

Differences from v3:

1/3 hw/net: Add npcm7xx emc model
- no change

2/3 hw/arm: Add npcm7xx emc model
- no change

3/3 tests/qtests: Add npcm7xx emc model test
- handle big endian hosts, tested on sparc64

Differences from v2:

1/3 hw/net: Add npcm7xx emc model
- move call to qemu_set_irq
- remove use of C99 mixed decls/statements
- add use of g_autofree

2/3 hw/arm: Add npcm7xx emc model
- none, patch ok as is

3/3 tests/qtests: Add npcm7xx emc model test
- remove use of C99 mixed decls/statements

Re: [PATCH v4 0/3] Add npcm7xx emc model
Posted by Peter Maydell 4 years, 9 months ago
On Sat, 13 Feb 2021 at 00:25, Doug Evans <dje@google.com> wrote:
>
> This is a 10/100 ethernet device that has several features.
> Only the ones needed by the Linux driver have been implemented.
> See npcm7xx_emc.c for a list of unimplemented features.
>
> Doug Evans (3):
>   hw/net: Add npcm7xx emc model
>   hw/arm: Add npcm7xx emc model
>   tests/qtests: Add npcm7xx emc model test
>


Applied to target-arm.next, thanks.

-- PMM

Re: [PATCH v4 0/3] Add npcm7xx emc model
Posted by Peter Maydell 4 years, 9 months ago
On Tue, 16 Feb 2021 at 14:28, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Sat, 13 Feb 2021 at 00:25, Doug Evans <dje@google.com> wrote:
> >
> > This is a 10/100 ethernet device that has several features.
> > Only the ones needed by the Linux driver have been implemented.
> > See npcm7xx_emc.c for a list of unimplemented features.
> >
> > Doug Evans (3):
> >   hw/net: Add npcm7xx emc model
> >   hw/arm: Add npcm7xx emc model
> >   tests/qtests: Add npcm7xx emc model test
> >
>
>
> Applied to target-arm.next, thanks.

Dropped again: it failed some oddball config in the gitlab CI:
https://gitlab.com/qemu-project/qemu/-/jobs/1034174731#L12

-- PMM

Re: [PATCH v4 0/3] Add npcm7xx emc model
Posted by Doug Evans 4 years, 9 months ago
On Wed, Feb 17, 2021 at 3:01 AM Peter Maydell <peter.maydell@linaro.org>
wrote:

> On Tue, 16 Feb 2021 at 14:28, Peter Maydell <peter.maydell@linaro.org>
> wrote:
> >
> > On Sat, 13 Feb 2021 at 00:25, Doug Evans <dje@google.com> wrote:
> > >
> > > This is a 10/100 ethernet device that has several features.
> > > Only the ones needed by the Linux driver have been implemented.
> > > See npcm7xx_emc.c for a list of unimplemented features.
> > >
> > > Doug Evans (3):
> > >   hw/net: Add npcm7xx emc model
> > >   hw/arm: Add npcm7xx emc model
> > >   tests/qtests: Add npcm7xx emc model test
> > >
> >
> >
> > Applied to target-arm.next, thanks.
>
> Dropped again: it failed some oddball config in the gitlab CI:
> https://gitlab.com/qemu-project/qemu/-/jobs/1034174731#L12
>
> -- PMM
>


Yeah, the test needs to watch for --disable-slirp.
Fixed in v5:
https://lists.nongnu.org/archive/html/qemu-devel/2021-02/msg06030.html