diff options
author | Nicholas Piggin <npiggin@gmail.com> | 2020-04-27 21:08:12 +1000 |
---|---|---|
committer | Oliver O'Halloran <oohall@gmail.com> | 2020-06-11 12:52:55 +1000 |
commit | de96c5c7c2660461625f7e6daa2299c60c4d3eab (patch) | |
tree | b6481f89f6393755da9ac08c51a3b86a2368a57e /hw/npu2-opencapi.c | |
parent | 52466735b64df8836711b698bdfe00e4a18d65bc (diff) | |
download | skiboot-de96c5c7c2660461625f7e6daa2299c60c4d3eab.zip skiboot-de96c5c7c2660461625f7e6daa2299c60c4d3eab.tar.gz skiboot-de96c5c7c2660461625f7e6daa2299c60c4d3eab.tar.bz2 |
io: endian annotations and fix
Annotate io accessor pointer types with endian.
sparse caught a bug in memcpy_from_ci, which is fixed.
From: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Diffstat (limited to 'hw/npu2-opencapi.c')
-rw-r--r-- | hw/npu2-opencapi.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/hw/npu2-opencapi.c b/hw/npu2-opencapi.c index b03aad3..035c6cd 100644 --- a/hw/npu2-opencapi.c +++ b/hw/npu2-opencapi.c @@ -1427,7 +1427,7 @@ static int64_t npu2_opencapi_pcicfg_read(struct phb *phb, uint32_t bdfn, cfg_addr = SETFIELD(NPU2_CQ_CTL_CONFIG_ADDR_REGISTER_NUMBER, cfg_addr, offset & ~3u); - out_be64((uint64_t *)genid_base, cfg_addr); + out_be64((beint64_t *)genid_base, cfg_addr); sync(); switch (size) { @@ -1437,10 +1437,10 @@ static int64_t npu2_opencapi_pcicfg_read(struct phb *phb, uint32_t bdfn, break; case 2: *((uint16_t *)data) = - in_le16((volatile uint16_t *)(genid_base + 128 + (offset & 2))); + in_le16((volatile leint16_t *)(genid_base + 128 + (offset & 2))); break; case 4: - *((uint32_t *)data) = in_le32((volatile uint32_t *)(genid_base + 128)); + *((uint32_t *)data) = in_le32((volatile leint32_t *)(genid_base + 128)); break; default: return OPAL_PARAMETER; @@ -1485,7 +1485,7 @@ static int64_t npu2_opencapi_pcicfg_write(struct phb *phb, uint32_t bdfn, cfg_addr = SETFIELD(NPU2_CQ_CTL_CONFIG_ADDR_REGISTER_NUMBER, cfg_addr, offset & ~3u); - out_be64((uint64_t *)genid_base, cfg_addr); + out_be64((beint64_t *)genid_base, cfg_addr); sync(); switch (size) { @@ -1494,11 +1494,11 @@ static int64_t npu2_opencapi_pcicfg_write(struct phb *phb, uint32_t bdfn, data); break; case 2: - out_le16((volatile uint16_t *)(genid_base + 128 + (offset & 2)), + out_le16((volatile leint16_t *)(genid_base + 128 + (offset & 2)), data); break; case 4: - out_le32((volatile uint32_t *)(genid_base + 128), data); + out_le32((volatile leint32_t *)(genid_base + 128), data); break; default: return OPAL_PARAMETER; |