The ioreadX() helpers have inconsistent interface.  On some architectures
void *__iomem address argument is a pointer to const, on some not.

Implementations of ioreadX() do not modify the memory under the
address so they can be converted to a "const" version for const-safety
and consistency among architectures.

Signed-off-by: Krzysztof Kozlowski <k...@kernel.org>
Acked-by: Alexey Brodkin <abrod...@synopsys.com>

---

Changes since v1:
1. Add Alexey's ack.
---
 arch/arc/plat-axs10x/axs10x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arc/plat-axs10x/axs10x.c b/arch/arc/plat-axs10x/axs10x.c
index 63ea5a606ecd..180c260a8221 100644
--- a/arch/arc/plat-axs10x/axs10x.c
+++ b/arch/arc/plat-axs10x/axs10x.c
@@ -84,7 +84,7 @@ static void __init axs10x_print_board_ver(unsigned int creg, 
const char *str)
                unsigned int val;
        } board;
 
-       board.val = ioread32((void __iomem *)creg);
+       board.val = ioread32((const void __iomem *)creg);
        pr_info("AXS: %s FPGA Date: %u-%u-%u\n", str, board.d, board.m,
                board.y);
 }
@@ -95,7 +95,7 @@ static void __init axs10x_early_init(void)
        char mb[32];
 
        /* Determine motherboard version */
-       if (ioread32((void __iomem *) CREG_MB_CONFIG) & (1 << 28))
+       if (ioread32((const void __iomem *) CREG_MB_CONFIG) & (1 << 28))
                mb_rev = 3;     /* HT-3 (rev3.0) */
        else
                mb_rev = 2;     /* HT-2 (rev2.0) */
-- 
2.17.1


_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc

Reply via email to