All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kumar Gala <kumar.gala@motorola.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] dynamic setting of CONFIG_SYS_CLK_FREQ
Date: Thu, 22 Apr 2004 14:51:16 -0500	[thread overview]
Message-ID: <6A9BF178-9496-11D8-9316-000393DBC2E8@motorola.com> (raw)
In-Reply-To: <20040422194434.A9C1BC109F@atlas.denx.de>

thanks,

I realized that sys_clk is not defined, I was going to add it for 85xx 
since none of the other defines in gd currently represent the value. 
The other option would be to have a function return the proper 
frequency (as suggested earlier in the thread). Any suggestions as to 
which option would be preferred?

I'm able to initialize gd->sys_clk (CONFIG_SYS_CLK_FREQ) as the first 
thing done in board_early_init_f.

- kumar

On Apr 22, 2004, at 2:44 PM, Wolfgang Denk wrote:

> In message <89194C60-9482-11D8-9316-000393DBC2E8@motorola.com> you 
> wrote:
>> How bad would it be to do something like:
>>
>> #define CONFIG_SYS_CLK_FREQ	(gd->sys_clk)
>
> Thisi s perfectly legal.
>
>> or is this what you meant by 'provide a suitable init fuction'
>
> No. I meant: you can do something like the above, but you  must  make
> sure  that  there  is  code  to  initialize gd->sys_clk before anyone
> attempts to access it.
>
> And BTW: sys_clk is not a member of the struct "gd".
>
> 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
>         ... Hiroshima 45 ... Chernobyl 86 ... Windows 95 ...

  reply	other threads:[~2004-04-22 19:51 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-22 16:02 [U-Boot-Users] dynamic setting of CONFIG_SYS_CLK_FREQ Kumar Gala
2004-04-22 17:14 ` Wolfgang Denk
2004-04-22 17:28   ` Kumar Gala
2004-04-22 19:44     ` Wolfgang Denk
2004-04-22 19:51       ` Kumar Gala [this message]
2004-04-25 14:03         ` Wolfgang Denk
2004-04-26  6:33           ` Pantelis Antoniou
2004-04-26  8:02             ` Wolfgang Denk
2004-04-26  8:16               ` Pantelis Antoniou
2004-04-26  9:14                 ` Wolfgang Denk
2004-04-26 10:18                   ` Pantelis Antoniou
2004-04-26 11:13                     ` Wolfgang Denk
2004-04-26 11:32                       ` Pantelis Antoniou
2004-04-26 13:03                         ` Wolfgang Denk
2004-04-26 13:25                           ` Pantelis Antoniou
2004-04-26 14:00                             ` Wolfgang Denk
2004-04-22 17:58 ` Dan Malek
  -- strict thread matches above, loose matches on Subject: below --
2004-04-22 17:52 VanBaren, Gerald

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=6A9BF178-9496-11D8-9316-000393DBC2E8@motorola.com \
    --to=kumar.gala@motorola.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 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.