All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] "date reset" for MPC8xx
Date: Mon, 14 Apr 2003 15:41:50 +0200	[thread overview]
Message-ID: <20030414134155.14FC2C58B9@atlas.denx.de> (raw)
In-Reply-To: Your message of "Mon, 14 Apr 2003 15:16:50 +0200." <3E9AB4C2.6020500@imc-berlin.de>

Dear Steven,

in message <3E9AB4C2.6020500@imc-berlin.de> you wrote:
> 
> I think would should realy reset the MPC8xx's internal RTC when 
> calling "date reset" or "rtc_reset" instead of "nothing to do".

But on the MPC8xx's internal RTC there is no "reset" function, so  it
is IMHO appropriate to do nothing.

> Please find and consider the attached patch.
> 
> Background:
> 
> I have seen it several times, that the MPC8xx's internal RTC contains 
> an invalid date, e.g. negative times (caused by no or low battery). 

There is no "invalid date" in the Unix notation of  time_t.  Negative
times (i. e. dates < January 1, 1970,00:00:00 UTC, are valid, too.

> That might result in dates before 1970-01-01 under Linux and that in 
> turn confuses timeouts.

Oops? I am not aware of any part in U-Boot (or Linux) that  uses  the
system time for timeouts! And I cannot understand why January 1, 1970
should  be  any better than January 1, 1960 or January 1, 1950 or any
other arbitrarily chosen date.

> I've seen that udhcpc (from busybox) hangs in such cases!

FIx this, then.

> For that reason u-boot should check for valid dates and do a 
> rtc_reset() if nessesary.

No. The RTC reset funtion is only to be used for RTC chips where  the
hardware supports such a reset operation.


If you need a valid system time, please use the "date" command to set
a valid time.


Patch rejected.


Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
Any sufficiently advanced technology is indistinguishable from magic.
                                                   - Arthur C. Clarke

      reply	other threads:[~2003-04-14 13:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-14 13:16 [U-Boot-Users] "date reset" for MPC8xx Steven Scholz
2003-04-14 13:41 ` Wolfgang Denk [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030414134155.14FC2C58B9@atlas.denx.de \
    --to=wd@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.