[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/8] dp8393x: Replace 0x40 magic value by SONIC_REG16_COUN
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v3 2/8] dp8393x: Replace 0x40 magic value by SONIC_REG16_COUNT definition |
Date: |
Sat, 10 Jul 2021 19:53:02 +0200 |
Typo 'SONIC_REG_COUNT' in subject.
On Sat, Jul 10, 2021 at 7:50 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/net/dp8393x.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c
> index 9118364aa33..d1e147a82a6 100644
> --- a/hw/net/dp8393x.c
> +++ b/hw/net/dp8393x.c
> @@ -85,6 +85,7 @@ static const char *reg_names[] = {
> #define SONIC_MPT 0x2e
> #define SONIC_MDT 0x2f
> #define SONIC_DCR2 0x3f
> +#define SONIC_REG_COUNT 0x40
>
> #define SONIC_CR_HTX 0x0001
> #define SONIC_CR_TXP 0x0002
> @@ -158,7 +159,7 @@ struct dp8393xState {
>
> /* Registers */
> uint8_t cam[16][6];
> - uint16_t regs[0x40];
> + uint16_t regs[SONIC_REG_COUNT];
>
> /* Temporaries */
> uint8_t tx_buffer[0x10000];
> @@ -972,7 +973,7 @@ static void dp8393x_realize(DeviceState *dev, Error
> **errp)
>
> address_space_init(&s->as, s->dma_mr, "dp8393x");
> memory_region_init_io(&s->mmio, OBJECT(dev), &dp8393x_ops, s,
> - "dp8393x-regs", 0x40 << s->it_shift);
> + "dp8393x-regs", SONIC_REG_COUNT << s->it_shift);
>
> s->nic = qemu_new_nic(&net_dp83932_info, &s->conf,
> object_get_typename(OBJECT(dev)), dev->id, s);
> @@ -987,7 +988,7 @@ static const VMStateDescription vmstate_dp8393x = {
> .minimum_version_id = 0,
> .fields = (VMStateField []) {
> VMSTATE_BUFFER_UNSAFE(cam, dp8393xState, 0, 16 * 6),
> - VMSTATE_UINT16_ARRAY(regs, dp8393xState, 0x40),
> + VMSTATE_UINT16_ARRAY(regs, dp8393xState, SONIC_REG_COUNT),
> VMSTATE_END_OF_LIST()
> }
> };
> --
> 2.31.1
>
- [PATCH v3 0/8] dp8393x: fixes and improvements, Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 1/8] dp8393x: Replace address_space_rw(is_write=1) by address_space_write(), Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 3/8] dp8393x: Only shift the device registers mapping by 1 bit, Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 2/8] dp8393x: Replace 0x40 magic value by SONIC_REG16_COUNT definition, Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 4/8] dp8393x: Store CAM registers as 16-bit, Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 5/8] dp8393x: Migrate registers as array of uint16, Philippe Mathieu-Daudé, 2021/07/10
- [PATCH v3 6/8] dp8393x: Store CRC using device configured endianess, Philippe Mathieu-Daudé, 2021/07/10
- [NOTFORMERGE PATCH v3 7/8] dp8393x: Rewrite dp8393x_get() / dp8393x_put(), Philippe Mathieu-Daudé, 2021/07/10
- [NOTFORMERGE PATCH v3 8/8] dp8393x: don't force 32-bit register access, Philippe Mathieu-Daudé, 2021/07/10