diff options
author | Gabe Black <gabeblack@google.com> | 2019-05-01 20:12:05 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2019-05-29 04:23:37 +0000 |
commit | d7c4cad240fd4f378d7362da5e9e44b9f0dd80d3 (patch) | |
tree | ca7d39cefd3df3ce87cbc6588a391faab03957f9 /src/mem/secure_port_proxy.hh | |
parent | 6a69fc18e743006db013797531d9ebcdb352a88d (diff) | |
download | gem5-d7c4cad240fd4f378d7362da5e9e44b9f0dd80d3.tar.xz |
mem, arm: Replace the pointer type in PortProxy with void *.
The void * type is for pointers which point to an unknown type. We
should use that when handling anonymous buffers in the PortProxy
functions, instead of uint8_t * which points to bytes.
Importantly, C/C++ doesn't require you to do any casting to turn an
arbitrary pointer type into a void *. This will get rid of lots of
tedious, verbose casting throughout the code base.
Change-Id: Id1adecc283c866d8e24524efd64f37b079088bd9
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18571
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Diffstat (limited to 'src/mem/secure_port_proxy.hh')
-rw-r--r-- | src/mem/secure_port_proxy.hh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mem/secure_port_proxy.hh b/src/mem/secure_port_proxy.hh index 032331210..9a2200c9e 100644 --- a/src/mem/secure_port_proxy.hh +++ b/src/mem/secure_port_proxy.hh @@ -73,8 +73,8 @@ class SecurePortProxy : public PortProxy SecurePortProxy(MasterPort &port, unsigned int cache_line_size) : PortProxy(port, cache_line_size) {} - bool tryReadBlob(Addr addr, uint8_t *p, int size) const override; - bool tryWriteBlob(Addr addr, const uint8_t *p, int size) const override; + bool tryReadBlob(Addr addr, void *p, int size) const override; + bool tryWriteBlob(Addr addr, const void *p, int size) const override; bool tryMemsetBlob(Addr addr, uint8_t val, int size) const override; }; |