From: Sughosh Ganu <urwithsughosh@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/3 V3] Make board_init_f under nand_boot.c a weak function
Date: Wed, 27 Oct 2010 11:28:09 +0530 [thread overview]
Message-ID: <20101027055809.GE10068@Hardy> (raw)
In-Reply-To: <20101026140706.745b026a@udp111988uds.am.freescale.net>
hi Scott,
On Tue Oct 26, 2010 at 02:07:06PM -0500, Scott Wood wrote:
> On Tue, 26 Oct 2010 23:27:44 +0530
> Sughosh Ganu <urwithsughosh@gmail.com> wrote:
>
> >
> > #if defined(CONFIG_ARM) && !defined(CONFIG_SYS_ARM_WITHOUT_RELOC)
> > -void board_init_f (ulong bootflag)
> > +void __board_init_f(ulong bootflag)
> > {
> > - relocate_code (CONFIG_SYS_TEXT_BASE - TOTAL_MALLOC_LEN, NULL,
> > - CONFIG_SYS_TEXT_BASE);
> > + relocate_code(CONFIG_SYS_TEXT_BASE - TOTAL_MALLOC_LEN, NULL,
> > + CONFIG_SYS_TEXT_BASE);
> > }
> > +void board_init_f(ulong bootflag)__attribute__((weak,
> > + alias("__board_init_f")));
> > #endif
> >
> > /*
>
> Eventually all boards should just provide their own board_init_f,
> which could just consist of a call to a common board init helper
> function. Or possibly a preprocessor define could be used to indicate
> that the common function should be used.
I had prososed that in my previous mail. I see that smdk6400 is the
only arm board using this. I think we can make this change to move
the board_init_f to the respective board directory right away. The
change should not impact any other boards. If fine with you, i will
respin my patch accordingly.
-sughosh
prev parent reply other threads:[~2010-10-27 5:58 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-26 17:57 [U-Boot] [PATCH 2/3 V3] Make board_init_f under nand_boot.c a weak function Sughosh Ganu
2010-10-26 19:07 ` Scott Wood
2010-10-27 5:58 ` Sughosh Ganu [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=20101027055809.GE10068@Hardy \
--to=urwithsughosh@gmail.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