date:20110122

Re: [U-Boot] [PATCH v3] arm926ejs: timer: Replace bss variable by gdr

2011-01-22 Thread Reinhard Meyer

Dear Albert ARIBAUD,
> However a general rework of ARM timer code is in order so that all SoCs
> and CPUs share the same set of gd variables with the same names and the
> same logic; and when we get that, this code shall move along.
>
> About this rework, as the saying goes... "Patches Welcome ©". :)

There were several suggestions about that in the past (including from me)
that involve rework everywhere HZ related timeouts are used. I still
prefer a method as follows (because it does not need repeated mul/div 
calculations
nor necessarily 64 bit arithmetic):

u32 timeout = timeout_init(100); /* 100ms timeout */

do {...} while (!timed_out(timeout));

Internally it would be like:

timeout_init(x):
  return fast_tick + (x * fast_tick_rate) / CONFIG_SYS_HZ;
  /* this might need 64 bit precision in some implementations */

time_out(x):
  return ((i32)(x - fast_tick)) < 0;

If the tick were really high speed (and then 64 bits),
fast_tick could be derived by shifting the tick some bits to the right.

But, as long as we cannot agree on something, there will be no time spent
to make patches...

Best Regards,
Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [v4 patch 6/6] SMDK6400: Fix SMDK6400 SDRAM init

2011-01-22 Thread Minkyu Kang

Dear seedshope,

On 22 January 2011 00:34, seedshope  wrote:
> Since SDRAM init function have already change, So the SDRAM
> initial function must be change.
>
> Signed-off-by: seedshope 
> ---
>  board/samsung/smdk6400/smdk6400.c |   10 +-
>  1 files changed, 9 insertions(+), 1 deletions(-)
>
> diff --git a/board/samsung/smdk6400/smdk6400.c 
> b/board/samsung/smdk6400/smdk6400.c
> index 35aa40b..1d03b7a 100644
> --- a/board/samsung/smdk6400/smdk6400.c
> +++ b/board/samsung/smdk6400/smdk6400.c
> @@ -78,10 +78,18 @@ int board_init(void)
>        return 0;
>  }
>
> -int dram_init(void)
> +void dram_init_banksize(void)
>  {
> +       DECLARE_GLOBAL_DATA_PTR;

Please remove it.

> +
>        gd->bd->bi_dram[0].start = PHYS_SDRAM_1;
>        gd->bd->bi_dram[0].size = PHYS_SDRAM_1_SIZE;

Thanks
Minkyu Kang
-- 
from. prom.
www.promsoft.net
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [v4 patch 6/6] SMDK6400: Fix SMDK6400 SDRAM init

2011-01-22 Thread Thomas Langer

Hi seedshope,

seedshope  gmail.com> writes:

> -int dram_init(void)
> +void dram_init_banksize(void)
>  {
> + DECLARE_GLOBAL_DATA_PTR;

This declaration should be done on file scope, not in a function.

> +
>   gd->bd->bi_dram[0].start = PHYS_SDRAM_1;
>   gd->bd->bi_dram[0].size = PHYS_SDRAM_1_SIZE;
> +}
> +

Best regards,
Thomas

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3] arm926ejs: timer: Replace bss variable by gdr

2011-01-22 Thread Alexander Holler

Am 22.01.2011 08:46, schrieb Albert ARIBAUD:
> Le 22/01/2011 06:39, Alexander Holler a écrit :
>> Hello,
>>
>> Am 21.01.2011 09:56, schrieb Heiko Schocher:
>>
>>> -static ulong timestamp;
>>> -static ulong lastdec;
>>> +DECLARE_GLOBAL_DATA_PTR;
>>> +
>>> +#define timestamp gd->tbl
>>> +#define lastdec gd->lastinc
>>
>> I'm the only one who doesn't like such defines? They might be handy for
>> quick fixes, but in regard to style and readablity I don't like them.
>> When looking at teh code where they will used, you won't see the actual
>> place where they are stored. And in more complex expression they might
>> become dangerous to use because they hide the operator "->".
>
> I accept the patch because it un-breaks support for ARM cpus, and I
> prefer a working fix to a perfect fix in this specific, transitional,
> situation.

My experience is that such quick fixes (or workarounds) usually 
manifests (because they become forgotten) and later on might even be 
copied to other places. ;)

Anyway, I'll have to thank for that patch, because it fixes at least one 
of the problems I have while trying to chainload a 2010.12 from a 
2010.12 on a kirkwood system.

Regards, and again, thanks for the patch, even if I found it ugly,

Alexander

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Бог благословитl

2011-01-22 Thread azeeza.b

миссис Azeeza Basara, родом из Катара. Мой муж покойный д-р Омар Джалал Basara.
Он работе посольства Катара в Судане и Марокко в течение двенадцати лет. И ушел
в отставку в нефтяной торговый до его смерти. Я решил не получить себе участие
в любой брак после его смерти или получить ребенка вне моего семейного дома. Я
страдал от рака матки в течение четырех лет после умер мой муж. Вместе с с
нарушениями слуха в связи с лекарственной реакции. У нас не было детей, после
восемнадцати лет в мирное брака. Недавно мой врач сказал мне, что моя жизнь не
будет превышать трех месяцев, в связи с болезнью рака моя. У меня есть депозит
в размере $ 7.6Musd в банк в Англии. И поэтому, я искренне хочет распространять
фонда для менее привилегированных в вашей стране (бывшего Советского Союза). Я
очень хочу, чтобы использовать его для гуманитарной деятельности. Я не ребенок,
который унаследует эти деньги от меня. А родственники моего мужа была плохо для
меня, поскольку моя болезнь моя, больных раком. Я ни хочет, ни банку заполучить
мои деньги после моей смерти.

Как только я получаю ваш ответ, я сообщу мой адвокат, чтобы помочь вам
снять деньги из банка, который вы будете использовать в улучшение человечества
в вашем регионе (стране). Пожалуйста, уверяют меня, что вы будете искренне
помочь мне в этом. Я ожидает Вашего ответа в скором времени с этой электронной
почтой (ab76...@hotmail.com)
Azeeza Basara.

44 matches

Mail list logo