From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/4] arm920t/at91/timer: replace bss variables by gd
Date: Tue, 30 Nov 2010 09:03:57 +0100 [thread overview]
Message-ID: <4CF4AFED.1010609@gmail.com> (raw)
In-Reply-To: <4CF4A503.50104@emk-elektronik.de>
Dear Reinhard Meyer,
Am 30.11.2010 08:17, schrieb Reinhard Meyer:
> Dear Andreas Bie?mann,
>> Reuse the gd->tbl/tbu values for timestamp/lastinc bss values in
>> arm920t/at91/timer driver.
> I see your dilemma here.
>
> tbu/tbl were introduced by me to form a true 64 bit monotonous incrementing value
> (like on most powerPC).
> You use tbl as the last (16 bit) value of the 16 bit hardware timer and
> tbu as the actual, only 32 bits worth time value.
> If the rest of the timer functions handle this correctly (I doubt that, but I cannot look at
> that right now), that "abuse" might be OK.
> But I rather have a field, say "u32 last_hw_val" (or a better name) added to the GD inside the
> AT91FAMILY define and have tbu/tbl really be a functional 64 bit value.
[snip]
I think get your point.
To get this bss issue fixed for v2010.12 I'd like to add another value
to GD to hold the last hw timer value. My current usage of tbu should
therefore go to tbl, to have a virutal 64 bit value just counting 32
bit, is that right?
Later on I also see some work in that arm920t/at91 SoC 'driver' section.
E.g. we really do need something like arm926ejs/at91/clock to have the
usart driver merged. And I might have some other changes to the timer
part of arm920t/at91. I found out the at91rm9200 do also have a PIT ...
but that is for later discussion.
regards
Andreas Bie?mann
next prev parent reply other threads:[~2010-11-30 8:03 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-30 6:37 [U-Boot] [PATCH 0/4] get at91rm9200ek working with ARM relocation Andreas Bießmann
2010-11-30 6:37 ` [U-Boot] [PATCH 1/4] at91rm9200ek: add configure target for RAM boot Andreas Bießmann
2010-11-30 6:37 ` [U-Boot] [PATCH 2/4] MAKEALL: fix AT91 Andreas Bießmann
2010-11-30 6:37 ` [U-Boot] [PATCH 3/4] arm920t: fix linker skript for -pie linking Andreas Bießmann
2010-12-08 22:52 ` Wolfgang Denk
2010-12-09 7:24 ` Andreas Bießmann
2010-12-09 7:33 ` Albert ARIBAUD
2010-12-09 9:45 ` Wolfgang Denk
2010-12-09 10:32 ` Wolfgang Denk
2010-12-09 10:51 ` Andreas Bießmann
2010-11-30 6:37 ` [U-Boot] [PATCH 4/4] arm920t/at91/timer: replace bss variables by gd Andreas Bießmann
2010-11-30 7:17 ` Reinhard Meyer
2010-11-30 8:03 ` Andreas Bießmann [this message]
2010-11-30 8:16 ` Wolfgang Denk
2010-11-30 8:48 ` Andreas Bießmann
2010-11-30 9:14 ` [U-Boot] TIMER cleanup RFC, was: " Reinhard Meyer
2010-11-30 15:11 ` J. William Campbell
2010-11-30 15:48 ` Reinhard Meyer
2010-11-30 17:29 ` J. William Campbell
2010-11-30 18:06 ` [U-Boot] [PATCH 0/4] get at91rm9200ek working with ARM relocation Andreas Bießmann
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=4CF4AFED.1010609@gmail.com \
--to=andreas.devel@googlemail.com \
--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