[PATCH v1 1/1] parisc: Add ioread64_lo_hi() and iowrite64_lo_hi()

[PATCH v1 1/1] parisc: Add ioread64_lo_hi() and iowrite64_lo_hi()

[Andy Shevchenko]

It's a followup to the previous commit f15309d7ad5d ("parisc: Add
ioread64_hi_lo() and iowrite64_hi_lo()") which does only half of
the job. Add the rest, so we won't get a new kernel test robot
reports.

Fixes: f15309d7ad5d ("parisc: Add ioread64_hi_lo() and iowrite64_hi_lo()")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 arch/parisc/lib/iomap.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/arch/parisc/lib/iomap.c b/arch/parisc/lib/iomap.c
index 367f6397bda7..860385058085 100644
--- a/arch/parisc/lib/iomap.c
+++ b/arch/parisc/lib/iomap.c
@@ -346,6 +346,16 @@ u64 ioread64be(const void __iomem *addr)
 	return *((u64 *)addr);
 }
 
+u64 ioread64_lo_hi(const void __iomem *addr)
+{
+	u32 low, high;
+
+	low = ioread32(addr);
+	high = ioread32(addr + sizeof(u32));
+
+	return low + ((u64)high << 32);
+}
+
 u64 ioread64_hi_lo(const void __iomem *addr)
 {
 	u32 low, high;
@@ -419,6 +429,12 @@ void iowrite64be(u64 datum, void __iomem *addr)
 	}
 }
 
+void iowrite64_lo_hi(u64 val, void __iomem *addr)
+{
+	iowrite32(val, addr);
+	iowrite32(val >> 32, addr + sizeof(u32));
+}
+
 void iowrite64_hi_lo(u64 val, void __iomem *addr)
 {
 	iowrite32(val >> 32, addr + sizeof(u32));
@@ -530,6 +546,7 @@ EXPORT_SYMBOL(ioread32);
 EXPORT_SYMBOL(ioread32be);
 EXPORT_SYMBOL(ioread64);
 EXPORT_SYMBOL(ioread64be);
+EXPORT_SYMBOL(ioread64_lo_hi);
 EXPORT_SYMBOL(ioread64_hi_lo);
 EXPORT_SYMBOL(iowrite8);
 EXPORT_SYMBOL(iowrite16);
@@ -538,6 +555,7 @@ EXPORT_SYMBOL(iowrite32);
 EXPORT_SYMBOL(iowrite32be);
 EXPORT_SYMBOL(iowrite64);
 EXPORT_SYMBOL(iowrite64be);
+EXPORT_SYMBOL(iowrite64_lo_hi);
 EXPORT_SYMBOL(iowrite64_hi_lo);
 EXPORT_SYMBOL(ioread8_rep);
 EXPORT_SYMBOL(ioread16_rep);
-- 
2.34.1

Re: [PATCH v1 1/1] parisc: Add ioread64_lo_hi() and iowrite64_lo_hi()

[Helge Deller]

On 2/7/22 16:16, Andy Shevchenko wrote:
> It's a followup to the previous commit f15309d7ad5d ("parisc: Add
> ioread64_hi_lo() and iowrite64_hi_lo()") which does only half of
> the job. Add the rest, so we won't get a new kernel test robot
> reports.
>
> Fixes: f15309d7ad5d ("parisc: Add ioread64_hi_lo() and iowrite64_hi_lo()")
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

applied to the parisc for-next tree.

Thanks!
Helge

> ---
>  arch/parisc/lib/iomap.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/arch/parisc/lib/iomap.c b/arch/parisc/lib/iomap.c
> index 367f6397bda7..860385058085 100644
> --- a/arch/parisc/lib/iomap.c
> +++ b/arch/parisc/lib/iomap.c
> @@ -346,6 +346,16 @@ u64 ioread64be(const void __iomem *addr)
>  	return *((u64 *)addr);
>  }
>
> +u64 ioread64_lo_hi(const void __iomem *addr)
> +{
> +	u32 low, high;
> +
> +	low = ioread32(addr);
> +	high = ioread32(addr + sizeof(u32));
> +
> +	return low + ((u64)high << 32);
> +}
> +
>  u64 ioread64_hi_lo(const void __iomem *addr)
>  {
>  	u32 low, high;
> @@ -419,6 +429,12 @@ void iowrite64be(u64 datum, void __iomem *addr)
>  	}
>  }
>
> +void iowrite64_lo_hi(u64 val, void __iomem *addr)
> +{
> +	iowrite32(val, addr);
> +	iowrite32(val >> 32, addr + sizeof(u32));
> +}
> +
>  void iowrite64_hi_lo(u64 val, void __iomem *addr)
>  {
>  	iowrite32(val >> 32, addr + sizeof(u32));
> @@ -530,6 +546,7 @@ EXPORT_SYMBOL(ioread32);
>  EXPORT_SYMBOL(ioread32be);
>  EXPORT_SYMBOL(ioread64);
>  EXPORT_SYMBOL(ioread64be);
> +EXPORT_SYMBOL(ioread64_lo_hi);
>  EXPORT_SYMBOL(ioread64_hi_lo);
>  EXPORT_SYMBOL(iowrite8);
>  EXPORT_SYMBOL(iowrite16);
> @@ -538,6 +555,7 @@ EXPORT_SYMBOL(iowrite32);
>  EXPORT_SYMBOL(iowrite32be);
>  EXPORT_SYMBOL(iowrite64);
>  EXPORT_SYMBOL(iowrite64be);
> +EXPORT_SYMBOL(iowrite64_lo_hi);
>  EXPORT_SYMBOL(iowrite64_hi_lo);
>  EXPORT_SYMBOL(ioread8_rep);
>  EXPORT_SYMBOL(ioread16_rep);
>