diff options
| author | Philippe Mathieu-Daudé <philmd@redhat.com> | 2021-12-17 22:43:05 +0100 |
|---|---|---|
| committer | Philippe Mathieu-Daudé <philmd@redhat.com> | 2021-12-30 17:16:32 +0100 |
| commit | 41d5e8da3d5e0a143a9fb397c9f34707ec544997 (patch) | |
| tree | 136741187a2433305f87f44a711e97a27276dd99 /hw/net/imx_fec.c | |
| parent | d5a9f352896fe43183ef01072b374e89a3488315 (diff) | |
| download | focaccia-qemu-41d5e8da3d5e0a143a9fb397c9f34707ec544997.tar.gz focaccia-qemu-41d5e8da3d5e0a143a9fb397c9f34707ec544997.zip | |
hw/scsi/megasas: Use uint32_t for reply queue head/tail values
While the reply queue values fit in 16-bit, they are accessed as 32-bit: 661: s->reply_queue_head = ldl_le_pci_dma(pcid, s->producer_pa); 662: s->reply_queue_head %= MEGASAS_MAX_FRAMES; 663: s->reply_queue_tail = ldl_le_pci_dma(pcid, s->consumer_pa); 664: s->reply_queue_tail %= MEGASAS_MAX_FRAMES; Having: 41:#define MEGASAS_MAX_FRAMES 2048 /* Firmware limit at 65535 */ In order to update the ld/st*_pci_dma() API to pass the address of the value to access, it is simpler to have the head/tail declared as 32-bit values. Replace the uint16_t by uint32_t, wasting 4 bytes in the MegasasState structure. Acked-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> Message-Id: <20211223115554.3155328-20-philmd@redhat.com>
Diffstat (limited to 'hw/net/imx_fec.c')
0 files changed, 0 insertions, 0 deletions