power: reset: at91: fix __le32 cast in reset code
authorBen Dooks (Codethink) <ben.dooks@codethink.co.uk>
Tue, 15 Oct 2019 15:54:14 +0000 (16:54 +0100)
committerSebastian Reichel <sre@kernel.org>
Sun, 20 Oct 2019 13:04:13 +0000 (15:04 +0200)
The writel() takes standard integers, not __le32 so
fix the following sparse warnings by removing the
cpu_to_le32() calls.

drivers/power/reset/at91-reset.c:134:9: warning: cast from restricted __le32
drivers/power/reset/at91-reset.c:143:9: warning: cast from restricted __le32

This has made no code changes, the md5sums pre and post applying
this patch are the same. The at91 should be natively little endian
anyway.

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/reset/at91-reset.c

index 44ca983..d94e326 100644 (file)
@@ -131,7 +131,7 @@ static int at91sam9g45_restart(struct notifier_block *this, unsigned long mode,
 static int sama5d3_restart(struct notifier_block *this, unsigned long mode,
                           void *cmd)
 {
-       writel(cpu_to_le32(AT91_RSTC_KEY | AT91_RSTC_PERRST | AT91_RSTC_PROCRST),
+       writel(AT91_RSTC_KEY | AT91_RSTC_PERRST | AT91_RSTC_PROCRST,
               at91_rstc_base);
 
        return NOTIFY_DONE;
@@ -140,9 +140,7 @@ static int sama5d3_restart(struct notifier_block *this, unsigned long mode,
 static int samx7_restart(struct notifier_block *this, unsigned long mode,
                         void *cmd)
 {
-       writel(cpu_to_le32(AT91_RSTC_KEY | AT91_RSTC_PROCRST),
-              at91_rstc_base);
-
+       writel(AT91_RSTC_KEY | AT91_RSTC_PROCRST, at91_rstc_base);
        return NOTIFY_DONE;
 }