Re: [PATCH] RTC: M41T80: Century Bit support
From: Maciej W. Rozycki
Date: Tue May 20 2008 - 15:53:27 EST
On Mon, 19 May 2008, Atsushi Nemoto wrote:
> > That's what I did initially as it is as obvious as you can get. But as I
> > noted, CFE, the firmware used with the SWARM, does not set CEB even though
> > it takes CB into account. The approach is not useful therefore at least
> > for one major user of the device.
>
> Oh I had missed that the firmware does not set CEB. Hmm...
As I said, this is one of the lesser problems with CFE, sigh...
> > Hmm, not terribly useful as few of us if anybody live back in the 20th
> > century. ;-) I think let's scrap the patch in this case and let our
> > grandchildren solve the problem -- a proposal has been published and can
> > be reused as needed.
>
> OK for me. Lets hear how our grandchildren complain on this fault. :)
Well, Andrew has applied this patch to the -mm tree now and after a bit
of thinking I have made up my mind and decided we should keep the patch.
This is an I2C device which means it will always only be explicitly
requested by platform code. This is an opportunity for the platform to
express the will and the way to use the century bit.
I'll cook-up a follow-on patch as soon as the SWARM bits propagate to
upstream, to add a set of flags that will let platforms specify the
desired way the century bit is meant to be handled and which this driver
will honour. Please feel free to keep the interpretation within your pet
board intact -- the flags will be capable to express your needs.
Andrew, could you please hold the patch in -mm till the flags have been
added?
Maciej
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/