Re: [PATCH] New command checktime

[Robert Millan]

On Wed, Aug 06, 2008 at 11:09:44PM +0800, Bean wrote:
> On Wed, Aug 6, 2008 at 7:35 PM, Robert Millan <address@hidden> wrote:
> > Can we make this portable by accessing 0x70 / 0x71 I/O ports instead?
> 
> Hi,
> 
> Good point. The new patch uses cmos to get datetime setting, it should
> be generic among i386 platforms.

Yep, this works on Coreboot as well (with the obvious i386-coreboot.rmk
adjustment).  Nice work!

> +      grub_printf ("%d-%02d-%02d %02d:%02d:%02d %d\n",
> +                   year, month, day, hour, min, sec, dow);

Btw, the dow part looks weird when printed.  Without looking at the code,
it's difficult to tell what that final number means.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."