[V3 3/6] RTC: sa1100: support sa1100, pxa and mmp soc families

Haojian Zhuang haojian.zhuang at gmail.com
Sun Jan 15 19:38:00 EST 2012


On Sat, Jan 14, 2012 at 5:27 PM, Russell King - ARM Linux
<linux at arm.linux.org.uk> wrote:
> On Wed, Nov 30, 2011 at 02:30:46PM +0800, Jett.Zhou wrote:
>> Since the regmap of rtc on sa1100, pxa and mmp Marvell soc families are
>> almost the same, so re-arch the rtc-sa1100 to support them.
>
> Err.  This looks wrong.  What's the clk stuff doing in here?  It seems
> to be completely unused by the driver, and then both PXA and SA1100
> end up having to provide a 'dummy' clk structure to satisfy this
> additional useless code.
>
> If it's a case that "every driver must use a clk" then there's a policy
> error somewhere.

Since this RTC driver could be shared among sa1100, pxa, mmp. In
sa1100 and pxa3xx, we don't have this clock. So we use dummy clock
instead. For pxa95x and mmp series, we have this clock.

Best Regards
Haojian



More information about the linux-arm-kernel mailing list