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
