U-Boot Archive on 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox