[PATCH v2 1/4] ARM: CSR: add rtc i/o bridge interface for SiRFprimaII

Arnd Bergmann arnd at arndb.de
Tue Aug 23 12:01:20 EDT 2011


On Tuesday 23 August 2011, Barry Song wrote:
> From: Zhiwu Song <zhiwu.song at csr.com>
> 
> The module is a bridge between the RTC clock domain and the CPU interface
> clock domain. ARM access the register of SYSRTC, GPSRTC and PWRC through
> this module.
> 
> Signed-off-by: Zhiwu Song <zhiwu.song at csr.com>
> Signed-off-by: Barry Song <Baohua.Song at csr.com>
> ---

Acked-by: Arnd Bergmann <arnd at arndb.de>

Looks basically good now. Aside from the trivial things that Jamie
already pointed out, there is one more that I noticed:

> +
> +void __iomem *sirfsoc_rtciobrg_base;
> +static DEFINE_SPINLOCK(rtciobrg_lock);

sirfsoc_rtciobrg_base should be static as well. No other code can
properly access it anyway since the lock is static, too.

	Arnd



More information about the linux-arm-kernel mailing list