From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Date: Fri, 27 Mar 2015 00:59:58 +0100 From: Boris Brezillon To: Ben Dooks Cc: linux-kernel@lists.codethink.co.uk, Guenter Roeck , Nicolas Ferre , Wim Van Sebroeck , linux-watchdog@vger.kernel.org, Andrew Victor , Jean-Christophe Plagniol-Villard Subject: Re: [PATCH] watchdog: at91sam9: use endian agnostic IO Message-ID: <20150327005958.3edc8276@bbrezillon> In-Reply-To: <1427380454-2002-1-git-send-email-ben.dooks@codethink.co.uk> References: <1427380454-2002-1-git-send-email-ben.dooks@codethink.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-ID: On Thu, 26 Mar 2015 14:34:14 +0000 Ben Dooks wrote: > Use endian agnostic IO functions for the watchdog driver for when it > is enabled on ATSAMA5D36 devices running in big endian. > > Signed-off-by: Ben Dooks Acked-by: Boris Brezillon > --- > CC: Boris Brezillon > CC: Guenter Roeck > CC: Nicolas Ferre > CC: Wim Van Sebroeck > CC: linux-watchdog@vger.kernel.org > CC: Andrew Victor > CC: Nicolas Ferre > CC: Jean-Christophe Plagniol-Villard > --- > drivers/watchdog/at91sam9_wdt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/at91sam9_wdt.c b/drivers/watchdog/at91sam9_wdt.c > index 1443b3c..e4698f7 100644 > --- a/drivers/watchdog/at91sam9_wdt.c > +++ b/drivers/watchdog/at91sam9_wdt.c > @@ -40,9 +40,9 @@ > #define DRV_NAME "AT91SAM9 Watchdog" > > #define wdt_read(wdt, field) \ > - __raw_readl((wdt)->base + (field)) > + readl_relaxed((wdt)->base + (field)) > #define wdt_write(wtd, field, val) \ > - __raw_writel((val), (wdt)->base + (field)) > + writel_relaxed((val), (wdt)->base + (field)) > > /* AT91SAM9 watchdog runs a 12bit counter @ 256Hz, > * use this to convert a watchdog -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com