[rtc-linux] [PATCH 1/2] rtc: rtc-lpc32xx: Introduce RTC driver for the LPC32XX SoC

[Wan ZongShun]

2010/8/11 Kevin Wells <kevin.wells at nxp.com>:
> Hi Zongshun,
>
> Thanks for helping review this.
>
>> Subject: Re: [rtc-linux] [PATCH 1/2] rtc: rtc-lpc32xx: Introduce RTC driver
>> for the LPC32XX SoC
>>
>> Hi Kevin ,
>>
>> This is really a natty rtc patch:).
>
> From the urban dictionary...
> "natty"
> 1) originating from rastafarian culture. meaning 'good','cool' and
> most importantly, 'elite'.
> 2) A slang for Natural Light beer, a favorite of poor college students
> the world over.
>

Hmm,I means the first, this patch looks very good to me. :)

> I hope it's the first one, but I guess the 2nd isn't that bad if
> you’re a college student o)
> I'll merge both patches into a single patch for v2 and repost in a
> few days.
>
>> > +       /* Disable alarm during update */
>> > +       tmp = rtc_readl(rtc, RTC_CTRL);
>> > +       rtc_writel(rtc, RTC_CTRL, tmp & ~RTC_MATCH0_EN);
>> > +
>> > +       rtc->alarm_enabled = wkalrm->enabled = 1;
>> > +       if (wkalrm->enabled) {
>> > +               rtc_writel(rtc, RTC_MATCH0, alarmsecs);
>> > +               rtc_writel(rtc, RTC_INTSTAT, RTC_MATCH0_INT_STS);
>> > +               rtc_writel(rtc, RTC_CTRL, tmp | RTC_MATCH0_EN);
>> > +       }
>>
>> I think this 'wkalrm->enabled ' will always be set '1', so 'if'
>> condition will be alway true?
>
> Thanks for catching this. The '= 1' assignment shouldn't be there.
>



-- 
*linux-arm-kernel mailing list
mail addr:linux-arm-kernel at lists.infradead.org
you can subscribe by:
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

* linux-arm-NUC900 mailing list
mail addr:NUC900 at googlegroups.com
main web: https://groups.google.com/group/NUC900
you can subscribe it by sending me mail:
mcuos.com at gmail.com