From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Wed, 22 Feb 2012 13:49:59 +0000 Subject: [PATCH 0/7] share sa1100 rtc driver to arch-mmp In-Reply-To: References: <1329815096-6200-1-git-send-email-haojian.zhuang@marvell.com> <201202221239.54321.arnd@arndb.de> Message-ID: <201202221350.00100.arnd@arndb.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 22 February 2012, Haojian Zhuang wrote: > On Wed, Feb 22, 2012 at 8:39 PM, Arnd Bergmann wrote: > > On Tuesday 21 February 2012, Haojian Zhuang wrote: > >> Remove the hardcode definition of rtc registers, use ioremap() instead. > >> Append clk support for arch-mmp, so add dummy clk in arch-sa1100/arch-pxa. > > > > What is the status of the drivers/rtc/rtc-pxa.c driver after this series? > > Is is still required for some pxa machines or can it get removed now? > > It seems a little strange to register two platform devices for the > > rtc on each pxa machine, with the same resources. > > Yes, it's very strange to support two RTC wrappers at the same time. > So rtc-pxa is removed in pxa27x/pxa3xx after this patch series. It > only exists in pxa95x. What is the reason for this? Wouldn't it be easier to move pxa95x over to use the same driver as well? I guess that can be a series for another time, but it seems like you are almost at the point where you can remove the other driver entirely. Arnd