public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ladislav Michl <ladis@linux-mips.org>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] NAND support [was: Port U-Boot to SBC-2410X]
Date: Wed, 31 Aug 2005 23:21:22 +0200	[thread overview]
Message-ID: <20050831212122.GA32559@orphique> (raw)
In-Reply-To: <20050831151911.3DA5635265F@atlas.denx.de>

On Wed, Aug 31, 2005 at 05:19:11PM +0200, Wolfgang Denk wrote:
> Please note that a  complete  rewrite  of  the  NAND  code  has  been
> discussed  several  times  before.  There  is  some  work in progress
> (performed  by  Ladislav  Michl   <ladis@linux-mips.org>)   whic   is
> available  in  the  "testing-NAND"  branch  of  the git repository at
> source.denx.net.

Wolfgang,

I promised you to send update over a weekend, but didn't tell you which
one ;-). Although normaly I do not have problems explaining my boss why
I'm working on something we do not necessarily need (nice clean code :)),
he has not so good understanding when I'm doing it when I should hunt
more important bugs elsewhere. My apologies to everyone...

> It makes IMHO littel sense to add patches to  the  old  code.  Please
> help improving / fixing the rewqrite instead.

Patch against "testing-NAND" branch can be found here:
ftp://ftp.linux-mips.org/pub/linux/mips/people/ladis/u-boot/u-boot-nand.diff
It also contains support for new board NetStar (it is not meant to be
included - I want to maintain it outside U-Boot, but I'd agree to put it
into "testing-NAND" branch for reference), which shows sample
implementation. Note that you have to define CONFIG_NEW_NAND_CODE to
actually compile and use new code. Original <linux/mtd/nand.h> was moved
to <linux/mtd/nand_old.h> and so has to be fixed all unconverted boards
includes (I do not have energy to do it now and test if they still compiles).
Ideally old and new code should be able to coexist. Please let me know if you
experience any problems or if you want me to split patch into smaller
chunks.

File list:
drivers/nand/*			- Linux MTD
include/linux/mtd/*		- MTD headers
include/nand.h			- compat stuff
common/cmd_nand_new.c		- user command interface to Linux MTD (*)
fs/jffs2/jffs2_nand_1pass.c
fs/jffs2/jffs2_nand_private.h	- somewhat faster jffs2 file loader (*)

(*) - needs discussion and improvements, it's not finished.

Best regards,
	ladis

  reply	other threads:[~2005-08-31 21:21 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-31 13:55 [U-Boot-Users] Port U-Boot to SBC-2410X luo.jinhua
2005-08-31 15:19 ` Wolfgang Denk
2005-08-31 21:21   ` Ladislav Michl [this message]
2005-08-31 23:05     ` [U-Boot-Users] NAND support [was: Port U-Boot to SBC-2410X] Wolfgang Denk
2005-09-01  8:21       ` Ladislav Michl
2005-09-01  1:40   ` [U-Boot-Users] Port U-Boot to SBC-2410X luo.jinhua

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=20050831212122.GA32559@orphique \
    --to=ladis@linux-mips.org \
    --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