ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

pci/shpc: change shpc_get_status() return type to uint8_t 

The result of the function is always one byte. The result is always
assigned to uint8_t variable. Also, shpc_get_status() should be
symmetric to shpc_set_status() which has uint8_t value argument.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Reviewed-by: Anton Kuchin <antonkuchin@yandex-team.ru>
Message-Id: <20230216180356.156832-3-vsementsov@yandex-team.ru>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
Vladimir Sementsov-Ogievskiy 2023-02-16 21:03:40 +03:00 committed by Michael S. Tsirkin
parent 93af1274ea
commit 94c84780ce
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hw/pci/shpc.c
View File

@ -123,10 +123,13 @@
#define SHPC_PCI_TO_IDX(pci_slot) ((pci_slot) - 1) #define SHPC_PCI_TO_IDX(pci_slot) ((pci_slot) - 1)
#define SHPC_IDX_TO_PHYSICAL(slot) ((slot) + 1) #define SHPC_IDX_TO_PHYSICAL(slot) ((slot) + 1)
static uint16_t shpc_get_status(SHPCDevice *shpc, int slot, uint16_t msk) static uint8_t shpc_get_status(SHPCDevice *shpc, int slot, uint16_t msk)
{ {
uint8_t *status = shpc->config + SHPC_SLOT_STATUS(slot); uint8_t *status = shpc->config + SHPC_SLOT_STATUS(slot);
return (pci_get_word(status) & msk) >> ctz32(msk); uint16_t result = (pci_get_word(status) & msk) >> ctz32(msk);
assert(result <= UINT8_MAX);
return result;
} }
static void shpc_set_status(SHPCDevice *shpc, static void shpc_set_status(SHPCDevice *shpc,