Change in coreboot[master]: libpayload: add read64() and write64()

[Thomas Heijligen (Code Review)]

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34206 )

Change subject: libpayload: add read64() and write64()
......................................................................

libpayload: add read64() and write64()

Change-Id: I6febf13ed54a7707f5a99d3d2715c36e18517c12
Signed-off-by: Thomas Heijligen 
---
M payloads/libpayload/include/x86/arch/io.h
1 file changed, 10 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/34206/1

diff --git a/payloads/libpayload/include/x86/arch/io.h b/payloads/libpayload/include/x86/arch/io.h
index c417ce0..46836d9 100644
--- a/payloads/libpayload/include/x86/arch/io.h
+++ b/payloads/libpayload/include/x86/arch/io.h
@@ -64,6 +64,11 @@
    return *((volatile uint32_t *)(addr));
 }

+static inline __attribute__((always_inline)) uint64_t read64(const volatile void *addr)
+{
+	return *((volatile uint64_t *)(addr));
+}
+
 static inline __attribute__((always_inline)) void write8(volatile void *addr, uint8_t value)
 {
    *((volatile uint8_t *)(addr)) = value;
@@ -79,6 +84,11 @@
    *((volatile uint32_t *)(addr)) = value;
 }

+static inline __attribute__((always_inline)) void write64(volatile void *addr, uint64_t value)
+{
+	*((volatile uint64_t *)(addr)) = value;
+}
+
 static inline unsigned int inl(int port)
 {
    unsigned long val;

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34206 )

Change subject: libpayload: add read64() and write64()
......................................................................

Patch Set 2: Code-Review+1

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34206 )

Change subject: libpayload: add read64() and write64()
......................................................................

Patch Set 2: Code-Review+2

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/34206 )

Change subject: libpayload: add read64() and write64()
......................................................................

libpayload: add read64() and write64()

Change-Id: I6febf13ed54a7707f5a99d3d2715c36e18517c12
Signed-off-by: Thomas Heijligen 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34206
Reviewed-by: Paul Menzel 
Reviewed-by: Patrick Rudolph 
Tested-by: build bot (Jenkins) 
---
M payloads/libpayload/include/x86/arch/io.h
1 file changed, 10 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Patrick Rudolph: Looks good to me, approved

diff --git a/payloads/libpayload/include/x86/arch/io.h b/payloads/libpayload/include/x86/arch/io.h
index c417ce0..46836d9 100644
--- a/payloads/libpayload/include/x86/arch/io.h
+++ b/payloads/libpayload/include/x86/arch/io.h
@@ -64,6 +64,11 @@
    return *((volatile uint32_t *)(addr));
 }

+static inline __attribute__((always_inline)) uint64_t read64(const volatile void *addr)
+{
+	return *((volatile uint64_t *)(addr));
+}
+
 static inline __attribute__((always_inline)) void write8(volatile void *addr, uint8_t value)
 {
    *((volatile uint8_t *)(addr)) = value;
@@ -79,6 +84,11 @@
    *((volatile uint32_t *)(addr)) = value;
 }

+static inline __attribute__((always_inline)) void write64(volatile void *addr, uint64_t value)
+{
+	*((volatile uint64_t *)(addr)) = value;
+}
+
 static inline unsigned int inl(int port)
 {
    unsigned long val;