Hi Ladislav, > (...) (And in ideal world > firmware (such as U-Boot, RedBoot, etc.) should set this register). Thanks for pointing the obvious out to me. You convinced me, this simply doesn't belong to the kernel. So your patch is not needed. I would still welcome a patch documenting the fact that the DS1339 is compatible with the DS1337 and is supported by the ds1337 driver. Thanks, -- Jean Delvare