From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 03/10] 83xx/85xx/86xx: LBC register cleanup
Date: Wed, 02 Jun 2010 14:16:30 -0500 [thread overview]
Message-ID: <4C06AE0E.7070301@freescale.com> (raw)
In-Reply-To: <1275502329-23457-4-git-send-email-beckyb@kernel.crashing.org>
On 06/02/2010 01:12 PM, Becky Bruce wrote:
> +#ifdef CONFIG_NAND_FSL_ELBC
> + u8 res9[0x8];
> + u32 fmr; /* Flash Mode Register */
> + u32 fir; /* Flash Instruction Register */
> + u32 fcr; /* Flash Command Register */
> + u32 fbar; /* Flash Block Addr Register */
> + u32 fpar; /* Flash Page Addr Register */
> + u32 fbcr; /* Flash Byte Count Register */
> + u8 res10[0xF08];
> +#else
> + u8 res9[0xF28];
> +#endif
Shouldn't this be CONFIG_FSL_ELBC (or really, CONFIG_SYS_FSL_ELBC, as it
describes the hardware and not user config)? The existence of these
fields isn't dependent on whether we're enabling the NAND driver (who
knows, some board code might want to do something directly with it, or
someone could just want the NAND SPL for some strange reason, or...).
-Scott
next prev parent reply other threads:[~2010-06-02 19:16 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-02 18:11 [U-Boot] [PATCH V2 0/10] PPC LBC/TLB cleanups; reginfo command for 85xx Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 01/10] powerpc: Update configs to properly set FSL_ELBC Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 02/10] drivers/usb/host/ohci-hcd: undef readl/writel before redefining Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 03/10] 83xx/85xx/86xx: LBC register cleanup Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 04/10] mpc85xx: tlb.c cleanups Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 05/10] mpc85xx: Add print_tlbcam() function Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 06/10] drivers/misc/fsl_law.c: Rearrange code to avoid duplication Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 07/10] fsl_law.c: Add print_laws() for FSL_CORENET platforms Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 08/10] mpc85xx: Add reginfo command Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 09/10] powerpc 83xx/85xx: Merge lbc upmconfig code Becky Bruce
2010-06-02 18:12 ` [U-Boot] [PATCH V2 10/10] MAKEALL: Add missing powerpc 36-bit targets Becky Bruce
2010-06-02 19:21 ` Scott Wood
2010-06-02 19:25 ` Becky Bruce
2010-06-02 20:08 ` Wolfgang Denk
2010-06-02 19:16 ` Scott Wood [this message]
2010-06-02 19:32 ` [U-Boot] [PATCH V2 03/10] 83xx/85xx/86xx: LBC register cleanup Becky Bruce
2010-06-02 20:55 ` [U-Boot] [PATCH V3 " Becky Bruce
2010-06-02 20:57 ` Becky Bruce
2010-06-04 0:37 ` [U-Boot] [PATCH V2 " Kim Phillips
2010-06-04 20:22 ` Scott Wood
2010-06-08 16:15 ` Becky Bruce
2010-06-08 17:09 ` Kim Phillips
2010-06-08 17:28 ` Becky Bruce
2010-06-08 18:43 ` Kim Phillips
2010-06-08 18:49 ` Becky Bruce
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=4C06AE0E.7070301@freescale.com \
--to=scottwood@freescale.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