All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
@ 2009-05-15 22:15 Matthias Kaehlcke
  2009-05-15 22:30 ` Scott Wood
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-05-15 22:15 UTC (permalink / raw)
  To: u-boot

KB9202: Add NAND support

This is a forward port of the patch submitted by Christian from
Kwikbyte in 06/2007
(http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>

--

--- u-boot-2009.03.org/board/kb9202/Makefile	2009-03-21 22:04:41.000000000 +0100
+++ u-boot-2009.03/board/kb9202/Makefile	2009-05-15 23:30:11.000000000 +0200
@@ -28,7 +28,7 @@
 
 LIB	= $(obj)lib$(BOARD).a
 
-COBJS	:= kb9202.o
+COBJS	:= kb9202.o nand.o
 
 SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)
 OBJS	:= $(addprefix $(obj),$(COBJS))
--- /dev/null	2009-05-14 23:36:38.280017984 +0200
+++ u-boot-2009.03/board/kb9202/nand.c	2009-05-15 23:30:11.000000000 +0200
@@ -0,0 +1,158 @@
+/*
+ * (C) Copyright 2006 KwikByte kb9200_dev at kwikbyte.com
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#ifdef CONFIG_CMD_NAND
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#ifndef	CONFIG_KB9202B_ATL
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+#else
+#define KB9202_NAND_NCE (1 << 26) /* EN* on PB26 */
+#define KB9202_NAND_BUSY (1 << 27) /* RB* on PB27 */
+#endif
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			AT91C_BASE_PIOC->PIO_CODR = KB9202_NAND_NCE;
+		else
+			AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned	value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->options &= ~(NAND_BUSWIDTH_16);
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	AT91C_BASE_PMC->PMC_PCER = ((unsigned) 1 << AT91C_ID_PIOC);
+#ifdef	CONFIG_KB9202B_ATL
+	AT91C_BASE_PMC->PMC_PCER = ((unsigned) 1 << AT91C_ID_PIOB);
+#endif
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	((AT91PS_SMC2)AT91C_BASE_SMC2)->SMC2_CSR[3] =
+		AT91C_SMC2_WSEN |
+		(4 & AT91C_SMC2_NWS) |
+		((1 << 8) & AT91C_SMC2_TDF) |
+		AT91C_SMC2_DBW_8 |
+		((1 << 24) & AT91C_SMC2_RWSETUP) |
+		((1 << 29) & AT91C_SMC2_RWHOLD);
+
+	/* enable internal NAND controller */
+	value = *(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	*(AT91C_EBI_CSA) = value;
+
+	/* enable SMOE/SMWE */
+	AT91C_BASE_PIOC->PIO_ASR = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+	AT91C_BASE_PIOC->PIO_PDR = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+	AT91C_BASE_PIOC->PIO_OER = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+
+	/* set NCE to high */
+	AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;
+
+	/* disable output on pin connected to the busy line of the NAND */
+	AT91C_BASE_PIOC->PIO_ODR = KB9202_NAND_BUSY;
+
+	/* enable the PIO to control NCE and BUSY */
+	AT91C_BASE_PIOC->PIO_PER = KB9202_NAND_NCE | KB9202_NAND_BUSY;
+
+	/* enable output for NCE */
+	AT91C_BASE_PIOC->PIO_OER = KB9202_NAND_NCE;
+
+	return (0);
+}
+
+#endif /* CONFIG_CMD_NAND */
--- u-boot-2009.03.org/include/configs/kb9202.h	2009-05-15 23:31:55.000000000 +0200
+++ u-boot-2009.03/include/configs/kb9202.h	2009-05-15 23:30:11.000000000 +0200
@@ -62,6 +62,11 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +76,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -111,6 +116,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI
@@ -140,7 +148,11 @@
 #define PHYS_FLASH_SIZE			0x200000
 #endif
 
+#ifdef CONFIG_KB9202B_ATL
+#define CFG_MAX_FLASH_BANKS    0
+#else
 #define CONFIG_SYS_MAX_FLASH_BANKS		1
+#endif
 #define CONFIG_SYS_MAX_FLASH_SECT		256
 
 #define	CONFIG_HARD_I2C

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

      El trabajo es el refugio de los que no tienen nada que hacer
                            (Oscar Wilde)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-05-15 22:15 [U-Boot] [PATCH 2/2] KB9202: Add NAND support Matthias Kaehlcke
@ 2009-05-15 22:30 ` Scott Wood
  2009-05-16 14:11   ` Matthias Kaehlcke
  2009-06-03 17:42   ` Matthias Kaehlcke
  0 siblings, 2 replies; 21+ messages in thread
From: Scott Wood @ 2009-05-15 22:30 UTC (permalink / raw)
  To: u-boot

Matthias Kaehlcke wrote:
> +/*
> + * Board-specific function to access the device ready signal.
> + */
> +static int kb9202_nand_ready(struct mtd_info *mtd)
> +{
> +	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
> +}

Use I/O accessors.

> +int board_nand_init(struct nand_chip *nand)
> +{
> +	unsigned	value;
[snip]
>> +	/* enable internal NAND controller */
>> +	value = *(AT91C_EBI_CSA);
>> +	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
>> +	*(AT91C_EBI_CSA) = value;

This is a hardware register.  Surely it has a defined width?

> +	/* setup nand flash access (allow ample margin) */
> +	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
> +	((AT91PS_SMC2)AT91C_BASE_SMC2)->SMC2_CSR[3] =
> +		AT91C_SMC2_WSEN |
> +		(4 & AT91C_SMC2_NWS) |
> +		((1 << 8) & AT91C_SMC2_TDF) |
> +		AT91C_SMC2_DBW_8 |
> +		((1 << 24) & AT91C_SMC2_RWSETUP) |
> +		((1 << 29) & AT91C_SMC2_RWHOLD);

What is AT91PS_SMC2?  Please don't hide pointers inside typedefs. 
Please define symbols to have the proper type in the first place, rather 
than casting@the point of use.

> +#ifdef CONFIG_KB9202B_ATL
> +#define CFG_MAX_FLASH_BANKS    0
> +#else

Is this really the recommended way of turning off flash support?

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-05-15 22:30 ` Scott Wood
@ 2009-05-16 14:11   ` Matthias Kaehlcke
  2009-05-18 17:38     ` Scott Wood
  2009-06-03 17:42   ` Matthias Kaehlcke
  1 sibling, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-05-16 14:11 UTC (permalink / raw)
  To: u-boot

Hi Scott

El Fri, May 15, 2009 at 05:30:48PM -0500 Scott Wood ha dit:

> Matthias Kaehlcke wrote:
>> +/*
>> + * Board-specific function to access the device ready signal.
>> + */
>> +static int kb9202_nand_ready(struct mtd_info *mtd)
>> +{
>> +	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
>> +}
>
> Use I/O accessors.
>
>> +int board_nand_init(struct nand_chip *nand)
>> +{
>> +	unsigned	value;
> [snip]
>>> +	/* enable internal NAND controller */
>>> +	value = *(AT91C_EBI_CSA);
>>> +	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
>>> +	*(AT91C_EBI_CSA) = value;
>
> This is a hardware register.  Surely it has a defined width?
>
>> +	/* setup nand flash access (allow ample margin) */
>> +	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
>> +	((AT91PS_SMC2)AT91C_BASE_SMC2)->SMC2_CSR[3] =
>> +		AT91C_SMC2_WSEN |
>> +		(4 & AT91C_SMC2_NWS) |
>> +		((1 << 8) & AT91C_SMC2_TDF) |
>> +		AT91C_SMC2_DBW_8 |
>> +		((1 << 24) & AT91C_SMC2_RWSETUP) |
>> +		((1 << 29) & AT91C_SMC2_RWHOLD);
>
> What is AT91PS_SMC2?  Please don't hide pointers inside typedefs. Please 
> define symbols to have the proper type in the first place, rather than 
> casting at the point of use.
>
>> +#ifdef CONFIG_KB9202B_ATL
>> +#define CFG_MAX_FLASH_BANKS    0
>> +#else
>
> Is this really the recommended way of turning off flash support?

thanks for your review and your comments.

i'm new to u-boot development, could you point me to the correct way
of turning of flash support? i found this in the original patch
provided by kwikbyte and just ported it to the current u-boot version.

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

              You can't separate peace from freedom because no
               one can be at peace unless he has his freedom
                              (Malcolm X)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-05-16 14:11   ` Matthias Kaehlcke
@ 2009-05-18 17:38     ` Scott Wood
  2009-05-19  7:25       ` Matthias Kaehlcke
  0 siblings, 1 reply; 21+ messages in thread
From: Scott Wood @ 2009-05-18 17:38 UTC (permalink / raw)
  To: u-boot

Matthias Kaehlcke wrote:
>>> +#ifdef CONFIG_KB9202B_ATL
>>> +#define CFG_MAX_FLASH_BANKS    0
>>> +#else
>> Is this really the recommended way of turning off flash support?
> 
> thanks for your review and your comments.
> 
> i'm new to u-boot development, could you point me to the correct way
> of turning of flash support? i found this in the original patch
> provided by kwikbyte and just ported it to the current u-boot version.

To disable NOR flash support, do not define CONFIG_FLASH_CFI_* or 
CONFIG_CMD_FLASH, and do define CONFIG_SYS_NO_FLASH.

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-05-18 17:38     ` Scott Wood
@ 2009-05-19  7:25       ` Matthias Kaehlcke
  0 siblings, 0 replies; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-05-19  7:25 UTC (permalink / raw)
  To: u-boot

El Mon, May 18, 2009 at 12:38:35PM -0500 Scott Wood ha dit:

> Matthias Kaehlcke wrote:
>>>> +#ifdef CONFIG_KB9202B_ATL
>>>> +#define CFG_MAX_FLASH_BANKS    0
>>>> +#else
>>> Is this really the recommended way of turning off flash support?
>>
>> thanks for your review and your comments.
>>
>> i'm new to u-boot development, could you point me to the correct way
>> of turning of flash support? i found this in the original patch
>> provided by kwikbyte and just ported it to the current u-boot version.
>
> To disable NOR flash support, do not define CONFIG_FLASH_CFI_* or  
> CONFIG_CMD_FLASH, and do define CONFIG_SYS_NO_FLASH.

thanks for the pointer! actually i made an attempt with
CONFIG_SYS_NO_FLASH, but this seems disable flash support at all (NOR
and NAND), the linker complained about unresolved symbols in the JFFS
code.

at the moment my development system is down due to some hardware
fault. as soon as i recover it i'll prepare a patch that addresses the
concerns you have raised

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

    Yo soy como soy y t? eres como eres, construyamos un mundo donde yo
    pueda ser sin dejar de ser yo, donde t? puedas ser sin dejar de ser
    t?, y donde ni yo ni t? obliguemos al otro a ser como yo o como t?
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-05-15 22:30 ` Scott Wood
  2009-05-16 14:11   ` Matthias Kaehlcke
@ 2009-06-03 17:42   ` Matthias Kaehlcke
  2009-06-03 19:50     ` Scott Wood
  2009-06-04  6:53     ` Stefan Roese
  1 sibling, 2 replies; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-03 17:42 UTC (permalink / raw)
  To: u-boot

hi,

El Fri, May 15, 2009 at 05:30:48PM -0500 Scott Wood ha dit:

> Matthias Kaehlcke wrote:
>> +/*
>> + * Board-specific function to access the device ready signal.
>> + */
>> +static int kb9202_nand_ready(struct mtd_info *mtd)
>> +{
>> +	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
>> +}
>
> Use I/O accessors.
>
>> +int board_nand_init(struct nand_chip *nand)
>> +{
>> +	unsigned	value;
> [snip]
>>> +	/* enable internal NAND controller */
>>> +	value = *(AT91C_EBI_CSA);
>>> +	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
>>> +	*(AT91C_EBI_CSA) = value;
>
> This is a hardware register.  Surely it has a defined width?
>
>> +	/* setup nand flash access (allow ample margin) */
>> +	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
>> +	((AT91PS_SMC2)AT91C_BASE_SMC2)->SMC2_CSR[3] =
>> +		AT91C_SMC2_WSEN |
>> +		(4 & AT91C_SMC2_NWS) |
>> +		((1 << 8) & AT91C_SMC2_TDF) |
>> +		AT91C_SMC2_DBW_8 |
>> +		((1 << 24) & AT91C_SMC2_RWSETUP) |
>> +		((1 << 29) & AT91C_SMC2_RWHOLD);
>
> What is AT91PS_SMC2?  Please don't hide pointers inside typedefs. Please 
> define symbols to have the proper type in the first place, rather than 
> casting at the point of use.

here is a version of the patch that addresses the issues you pointed
out. sorry for the late response, my development system died and i
needed some time to get a new machine and restore the setup

--

This is a forward port of the patch submitted by Christian from
Kwikbyte in 06/2007
(http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
---
 board/kb9202/Makefile    |    2 +-
 board/kb9202/nand.c      |  153 ++++++++++++++++++++++++++++++++++++++++++++++
 include/configs/kb9202.h |   10 +++-
 3 files changed, 163 insertions(+), 2 deletions(-)
 create mode 100644 board/kb9202/nand.c

diff --git a/board/kb9202/Makefile b/board/kb9202/Makefile
index 363f665..0ba4fcc 100644
--- a/board/kb9202/Makefile
+++ b/board/kb9202/Makefile
@@ -28,7 +28,7 @@ include $(TOPDIR)/config.mk
 
 LIB	= $(obj)lib$(BOARD).a
 
-COBJS	:= kb9202.o
+COBJS	:= kb9202.o nand.o
 
 SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)
 OBJS	:= $(addprefix $(obj),$(COBJS))
diff --git a/board/kb9202/nand.c b/board/kb9202/nand.c
new file mode 100644
index 0000000..fa63aa1
--- /dev/null
+++ b/board/kb9202/nand.c
@@ -0,0 +1,153 @@
+/*
+ * (C) Copyright 2006 KwikByte kb9200_dev at kwikbyte.com
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#ifdef CONFIG_CMD_NAND
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			AT91C_BASE_PIOC->PIO_CODR = KB9202_NAND_NCE;
+		else
+			AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	const unsigned int value = readl(AT91C_PIOC_PDSR);
+
+	return ((value & KB9202_NAND_BUSY) != 0);
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+	struct _AT91S_SMC2 *at91s_smc2 = AT91C_BASE_SMC2;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->options &= ~(NAND_BUSWIDTH_16);
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	AT91C_BASE_PMC->PMC_PCER = ((unsigned) 1 << AT91C_ID_PIOC);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	at91s_smc2->SMC2_CSR[3] =
+		AT91C_SMC2_WSEN |
+		(4 & AT91C_SMC2_NWS) |
+		((1 << 8) & AT91C_SMC2_TDF) |
+		AT91C_SMC2_DBW_8 |
+		((1 << 24) & AT91C_SMC2_RWSETUP) |
+		((1 << 29) & AT91C_SMC2_RWHOLD);
+
+	/* enable internal NAND controller */
+	value = *(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	*(AT91C_EBI_CSA) = value;
+
+	/* enable SMOE/SMWE */
+	AT91C_BASE_PIOC->PIO_ASR = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+	AT91C_BASE_PIOC->PIO_PDR = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+	AT91C_BASE_PIOC->PIO_OER = AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE;
+
+	/* set NCE to high */
+	AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;
+
+	/* disable output on pin connected to the busy line of the NAND */
+	AT91C_BASE_PIOC->PIO_ODR = KB9202_NAND_BUSY;
+
+	/* enable the PIO to control NCE and BUSY */
+	AT91C_BASE_PIOC->PIO_PER = KB9202_NAND_NCE | KB9202_NAND_BUSY;
+
+	/* enable output for NCE */
+	AT91C_BASE_PIOC->PIO_OER = KB9202_NAND_NCE;
+
+	return (0);
+}
+
+#endif /* CONFIG_CMD_NAND */
diff --git a/include/configs/kb9202.h b/include/configs/kb9202.h
index 8651e53..cd4c577 100644
--- a/include/configs/kb9202.h
+++ b/include/configs/kb9202.h
@@ -62,6 +62,11 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +76,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -112,6 +117,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI
-- 
1.6.3.1

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

    Me lo contaron y lo olvid?, lo vi y lo entend?, lo hice y lo aprend?
                              (Confucio)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-03 17:42   ` Matthias Kaehlcke
@ 2009-06-03 19:50     ` Scott Wood
  2009-06-04 17:45       ` Matthias Kaehlcke
  2009-06-04  6:53     ` Stefan Roese
  1 sibling, 1 reply; 21+ messages in thread
From: Scott Wood @ 2009-06-03 19:50 UTC (permalink / raw)
  To: u-boot

Matthias Kaehlcke wrote:
> hi,
> 
> El Fri, May 15, 2009 at 05:30:48PM -0500 Scott Wood ha dit:
> 
>> Matthias Kaehlcke wrote:
>>> +/*
>>> + * Board-specific function to access the device ready signal.
>>> + */
>>> +static int kb9202_nand_ready(struct mtd_info *mtd)
>>> +{
>>> +	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
>>> +}
>> Use I/O accessors.
[snip]
> +		if (ctrl & NAND_NCE)
> +			AT91C_BASE_PIOC->PIO_CODR = KB9202_NAND_NCE;
> +		else
> +			AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;

You're still not using I/O accessors in many places.

> +#ifdef CONFIG_CMD_NAND

Put this in the makefile instead.

> +static int kb9202_nand_ready(struct mtd_info *mtd)
> +{
> +	const unsigned int value = readl(AT91C_PIOC_PDSR);
> +
> +	return ((value & KB9202_NAND_BUSY) != 0);
> +}

static int kb9202_nand_ready(struct mtd_info *mtd)
{
	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
}

> +int board_nand_init(struct nand_chip *nand)
> +{
> +	unsigned int value;
> +	struct _AT91S_SMC2 *at91s_smc2 = AT91C_BASE_SMC2;
> +
> +	nand->ecc.mode = NAND_ECC_SOFT;
> +	nand->options &= ~(NAND_BUSWIDTH_16);

Unnecessary parens.

> +	nand->cmd_ctrl = kb9202_nand_hwcontrol;
> +	nand->dev_ready = kb9202_nand_ready;
> +
> +	/* in case running outside of bootloader */
> +	AT91C_BASE_PMC->PMC_PCER = ((unsigned) 1 << AT91C_ID_PIOC);

Unnecessary cast and parens.

> +	at91s_smc2->SMC2_CSR[3] =
> +		AT91C_SMC2_WSEN |
> +		(4 & AT91C_SMC2_NWS) |
> +		((1 << 8) & AT91C_SMC2_TDF) |
> +		AT91C_SMC2_DBW_8 |
> +		((1 << 24) & AT91C_SMC2_RWSETUP) |
> +		((1 << 29) & AT91C_SMC2_RWHOLD);

Are those instances of (constant1 & constant2) ever going to evaluate to 
anything but constant1?  Can we get rid of the magic numbers?

Otherwise, ACK.

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-03 17:42   ` Matthias Kaehlcke
  2009-06-03 19:50     ` Scott Wood
@ 2009-06-04  6:53     ` Stefan Roese
  2009-06-04 17:49       ` Matthias Kaehlcke
  1 sibling, 1 reply; 21+ messages in thread
From: Stefan Roese @ 2009-06-04  6:53 UTC (permalink / raw)
  To: u-boot

On Wednesday 03 June 2009 19:42:00 Matthias Kaehlcke wrote:

<snip>

> here is a version of the patch that addresses the issues you pointed
> out. sorry for the late response, my development system died and i
> needed some time to get a new machine and restore the setup
>
> --
>
> This is a forward port of the patch submitted by Christian from
> Kwikbyte in 06/2007
> (http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)
>
> Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> ---
>  board/kb9202/Makefile    |    2 +-
>  board/kb9202/nand.c      |  153
> ++++++++++++++++++++++++++++++++++++++++++++++ include/configs/kb9202.h |  
> 10 +++-
>  3 files changed, 163 insertions(+), 2 deletions(-)
>  create mode 100644 board/kb9202/nand.c

I suggest to move this driver to drivers/mtd/nand instead.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-03 19:50     ` Scott Wood
@ 2009-06-04 17:45       ` Matthias Kaehlcke
  0 siblings, 0 replies; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-04 17:45 UTC (permalink / raw)
  To: u-boot

El Wed, Jun 03, 2009 at 02:50:40PM -0500 Scott Wood ha dit:

> Matthias Kaehlcke wrote:
>> hi,
>>
>> El Fri, May 15, 2009 at 05:30:48PM -0500 Scott Wood ha dit:
>>
>>> Matthias Kaehlcke wrote:
>>>> +/*
>>>> + * Board-specific function to access the device ready signal.
>>>> + */
>>>> +static int kb9202_nand_ready(struct mtd_info *mtd)
>>>> +{
>>>> +	return (((AT91C_BASE_PIOC->PIO_PDSR) & KB9202_NAND_BUSY) != 0);
>>>> +}
>>> Use I/O accessors.
> [snip]
>> +		if (ctrl & NAND_NCE)
>> +			AT91C_BASE_PIOC->PIO_CODR = KB9202_NAND_NCE;
>> +		else
>> +			AT91C_BASE_PIOC->PIO_SODR = KB9202_NAND_NCE;
>
> You're still not using I/O accessors in many places.
>
>> +#ifdef CONFIG_CMD_NAND
>
> Put this in the makefile instead.
>
>> +static int kb9202_nand_ready(struct mtd_info *mtd)
>> +{
>> +	const unsigned int value = readl(AT91C_PIOC_PDSR);
>> +
>> +	return ((value & KB9202_NAND_BUSY) != 0);
>> +}
>
> static int kb9202_nand_ready(struct mtd_info *mtd)
> {
> 	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
> }
>
>> +int board_nand_init(struct nand_chip *nand)
>> +{
>> +	unsigned int value;
>> +	struct _AT91S_SMC2 *at91s_smc2 = AT91C_BASE_SMC2;
>> +
>> +	nand->ecc.mode = NAND_ECC_SOFT;
>> +	nand->options &= ~(NAND_BUSWIDTH_16);
>
> Unnecessary parens.
>
>> +	nand->cmd_ctrl = kb9202_nand_hwcontrol;
>> +	nand->dev_ready = kb9202_nand_ready;
>> +
>> +	/* in case running outside of bootloader */
>> +	AT91C_BASE_PMC->PMC_PCER = ((unsigned) 1 << AT91C_ID_PIOC);
>
> Unnecessary cast and parens.
>
>> +	at91s_smc2->SMC2_CSR[3] =
>> +		AT91C_SMC2_WSEN |
>> +		(4 & AT91C_SMC2_NWS) |
>> +		((1 << 8) & AT91C_SMC2_TDF) |
>> +		AT91C_SMC2_DBW_8 |
>> +		((1 << 24) & AT91C_SMC2_RWSETUP) |
>> +		((1 << 29) & AT91C_SMC2_RWHOLD);
>
> Are those instances of (constant1 & constant2) ever going to evaluate to  
> anything but constant1?  Can we get rid of the magic numbers?
>
> Otherwise, ACK.

thanks for your comments!

i hope this version of the patch addresses all your concerns

--

This is a forward port of the patch submitted by Christian from
Kwikbyte in 06/2007
(http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
---
 board/kb9202/Makefile                        |    1 +
 board/kb9202/nand.c                          |  151 ++++++++++++++++++++++++++
 include/asm-arm/arch-at91rm9200/AT91RM9200.h |    2 +
 include/configs/kb9202.h                     |   10 ++-
 4 files changed, 163 insertions(+), 1 deletions(-)
 create mode 100644 board/kb9202/nand.c

diff --git a/board/kb9202/Makefile b/board/kb9202/Makefile
index 363f665..d78454d 100644
--- a/board/kb9202/Makefile
+++ b/board/kb9202/Makefile
@@ -29,6 +29,7 @@ include $(TOPDIR)/config.mk
 LIB	= $(obj)lib$(BOARD).a
 
 COBJS	:= kb9202.o
+COBJS-$(CONFIG_CMD_NAND)   += nand.o
 
 SRCS	:= $(SOBJS:.o=.S) $(COBJS:.o=.c)
 OBJS	:= $(addprefix $(obj),$(COBJS))
diff --git a/board/kb9202/nand.c b/board/kb9202/nand.c
new file mode 100644
index 0000000..7db8d7c
--- /dev/null
+++ b/board/kb9202/nand.c
@@ -0,0 +1,151 @@
+/*
+ * (C) Copyright 2006
+ * KwikByte <kb9200_dev@kwikbyte.com>
+ *
+ * (C) Copyright 2009
+ * Matthias Kaehlcke <matthias@kaehlcke.net>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+#define KB9202_SMC2_NWS (1 << 2)
+#define KB9202_SMC2_TDF (1 << 8)
+#define KB9202_SMC2_RWSETUP (1 << 24)
+#define KB9202_SMC2_RWHOLD (1 << 29)
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			writel(KB9202_NAND_NCE, AT91C_PIOC_CODR);
+		else
+			writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->options &= ~NAND_BUSWIDTH_16;
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	writel(1 << AT91C_ID_PIOC, AT91C_PMC_PCER);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
+				 AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
+				 AT91C_SMC_CSR3);
+
+	/* enable internal NAND controller */
+	value = readl(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	writel(value, AT91C_EBI_CSA);
+
+	/* enable SMOE/SMWE */
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_ASR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_PDR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_OER);
+
+	/* set NCE to high */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+
+	/* disable output on pin connected to the busy line of the NAND */
+	writel(KB9202_NAND_BUSY, AT91C_PIOC_ODR);
+
+	/* enable the PIO to control NCE and BUSY */
+	writel(KB9202_NAND_NCE | KB9202_NAND_BUSY, AT91C_PIOC_PER);
+
+	/* enable output for NCE */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_OER);
+
+	return (0);
+}
diff --git a/include/asm-arm/arch-at91rm9200/AT91RM9200.h b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
index 00bae1c..2e40565 100644
--- a/include/asm-arm/arch-at91rm9200/AT91RM9200.h
+++ b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
@@ -737,6 +737,7 @@ typedef struct _AT91S_PDC
 #define AT91C_SMC2_ACSS_STANDARD	((unsigned int) 0x0 << 16) /* (SMC2) Standard, asserted at the beginning of the access and deasserted at the end. */
 #define AT91C_SMC2_DBW_8	((unsigned int) 0x2 << 13) /* (SMC2) 8-bit. */
 #define AT91C_SMC2_WSEN		((unsigned int) 0x1 <<  7) /* (SMC2) Wait State Enable */
+#define AT91C_PIOC_OER		((AT91_REG *)	0xFFFFF810) /* (PIOC) PIO Output Enable Register */
 #define AT91C_PIOC_ASR		((AT91_REG *)	0xFFFFF870) /* (PIOC) Select A Register */
 #define AT91C_PIOC_SODR		((AT91_REG *)	0xFFFFF830) /* (PIOC) Set Output Data Register */
 #define AT91C_PIOC_CODR		((AT91_REG *)	0xFFFFF834) /* (PIOC) Clear Output Data Register */
@@ -790,6 +791,7 @@ typedef struct _AT91S_PDC
 #define AT91C_MC_AASR		0xFFFFFF08
 #define AT91C_EBI_CFGR		0xFFFFFF64
 #define AT91C_SMC_CSR0		0xFFFFFF70
+#define AT91C_SMC_CSR3		0xFFFFFF7C
 
 /* clocks */
 #define AT91C_PLLAR		0xFFFFFC28
diff --git a/include/configs/kb9202.h b/include/configs/kb9202.h
index 8651e53..cd4c577 100644
--- a/include/configs/kb9202.h
+++ b/include/configs/kb9202.h
@@ -62,6 +62,11 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +76,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -112,6 +117,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI
-- 
1.6.3.1

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

             If liberty means anything@all, it means the
           right to tell people what they do not want to hear
                            (George Orwell)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-04  6:53     ` Stefan Roese
@ 2009-06-04 17:49       ` Matthias Kaehlcke
  2009-06-04 18:08         ` Stefan Roese
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-04 17:49 UTC (permalink / raw)
  To: u-boot

El Thu, Jun 04, 2009 at 08:53:10AM +0200 Stefan Roese ha dit:

> On Wednesday 03 June 2009 19:42:00 Matthias Kaehlcke wrote:
> 
> <snip>
> 
> > here is a version of the patch that addresses the issues you pointed
> > out. sorry for the late response, my development system died and i
> > needed some time to get a new machine and restore the setup
> >
> > --
> >
> > This is a forward port of the patch submitted by Christian from
> > Kwikbyte in 06/2007
> > (http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)
> >
> > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > ---
> >  board/kb9202/Makefile    |    2 +-
> >  board/kb9202/nand.c      |  153
> > ++++++++++++++++++++++++++++++++++++++++++++++ include/configs/kb9202.h |  
> > 10 +++-
> >  3 files changed, 163 insertions(+), 2 deletions(-)
> >  create mode 100644 board/kb9202/nand.c
> 
> I suggest to move this driver to drivers/mtd/nand instead.

i thought drivers/mtd/nand is for generic nand drivers, mine - at least
atm - is kb9202 specific. if it's really preferred to have board
specific drivers in drivers/mtd/nand i'll the adapt the patch happily.

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

    Yo soy como soy y t? eres como eres, construyamos un mundo donde yo
    pueda ser sin dejar de ser yo, donde t? puedas ser sin dejar de ser
    t?, y donde ni yo ni t? obliguemos al otro a ser como yo o como t?
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-04 17:49       ` Matthias Kaehlcke
@ 2009-06-04 18:08         ` Stefan Roese
  2009-06-04 18:50           ` Matthias Kaehlcke
  0 siblings, 1 reply; 21+ messages in thread
From: Stefan Roese @ 2009-06-04 18:08 UTC (permalink / raw)
  To: u-boot

On Thursday 04 June 2009 19:49:25 Matthias Kaehlcke wrote:
> > > This is a forward port of the patch submitted by Christian from
> > > Kwikbyte in 06/2007
> > > (http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)
> > >
> > > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > > ---
> > >  board/kb9202/Makefile    |    2 +-
> > >  board/kb9202/nand.c      |  153
> > > ++++++++++++++++++++++++++++++++++++++++++++++ include/configs/kb9202.h
> > > | 10 +++-
> > >  3 files changed, 163 insertions(+), 2 deletions(-)
> > >  create mode 100644 board/kb9202/nand.c
> >
> > I suggest to move this driver to drivers/mtd/nand instead.
>
> i thought drivers/mtd/nand is for generic nand drivers,

No, it's for all NAND driver using the common NAND MTD infrastructure. Same as 
in Linux.

> mine - at least
> atm - is kb9202 specific. if it's really preferred to have board
> specific drivers in drivers/mtd/nand i'll the adapt the patch happily.

Yes, please move it. It's better to collect all NAND related drivers in one 
place.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-04 18:08         ` Stefan Roese
@ 2009-06-04 18:50           ` Matthias Kaehlcke
  2009-06-04 19:08             ` Wolfgang Denk
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-04 18:50 UTC (permalink / raw)
  To: u-boot

El Thu, Jun 04, 2009 at 08:08:04PM +0200 Stefan Roese ha dit:

> On Thursday 04 June 2009 19:49:25 Matthias Kaehlcke wrote:
> > > > This is a forward port of the patch submitted by Christian from
> > > > Kwikbyte in 06/2007
> > > > (http://lists.denx.de/pipermail/u-boot/2007-June/022068.html)
> > > >
> > > > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > > > ---
> > > >  board/kb9202/Makefile    |    2 +-
> > > >  board/kb9202/nand.c      |  153
> > > > ++++++++++++++++++++++++++++++++++++++++++++++ include/configs/kb9202.h
> > > > | 10 +++-
> > > >  3 files changed, 163 insertions(+), 2 deletions(-)
> > > >  create mode 100644 board/kb9202/nand.c
> > >
> > > I suggest to move this driver to drivers/mtd/nand instead.
> >
> > i thought drivers/mtd/nand is for generic nand drivers,
> 
> No, it's for all NAND driver using the common NAND MTD infrastructure. Same as 
> in Linux.
> 
> > mine - at least
> > atm - is kb9202 specific. if it's really preferred to have board
> > specific drivers in drivers/mtd/nand i'll the adapt the patch happily.
> 
> Yes, please move it. It's better to collect all NAND related drivers in one 
> place.

thanks for your explication, below is the driver in the corresponding
directory

--

Add NAND support for the KwikByte KB9202

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
---
 drivers/mtd/nand/Makefile                    |    1 +
 drivers/mtd/nand/kb9202_nand.c               |  151 ++++++++++++++++++++++++++
 include/asm-arm/arch-at91rm9200/AT91RM9200.h |    2 +
 include/configs/kb9202.h                     |   11 ++-
 4 files changed, 164 insertions(+), 1 deletions(-)
 create mode 100644 drivers/mtd/nand/kb9202_nand.c

diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index 471cd6b..d2f3e61 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -44,6 +44,7 @@ COBJS-$(CONFIG_NAND_NOMADIK) += nomadik.o
 COBJS-$(CONFIG_NAND_S3C2410) += s3c2410_nand.c
 COBJS-$(CONFIG_NAND_S3C64XX) += s3c64xx.o
 COBJS-$(CONFIG_NAND_OMAP_GPMC) += omap_gpmc.o
+COBJS-$(CONFIG_NAND_KB9202) += kb9202_nand.o
 endif
 
 COBJS	:= $(COBJS-y)
diff --git a/drivers/mtd/nand/kb9202_nand.c b/drivers/mtd/nand/kb9202_nand.c
new file mode 100644
index 0000000..7db8d7c
--- /dev/null
+++ b/drivers/mtd/nand/kb9202_nand.c
@@ -0,0 +1,151 @@
+/*
+ * (C) Copyright 2006
+ * KwikByte <kb9200_dev@kwikbyte.com>
+ *
+ * (C) Copyright 2009
+ * Matthias Kaehlcke <matthias@kaehlcke.net>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+#define KB9202_SMC2_NWS (1 << 2)
+#define KB9202_SMC2_TDF (1 << 8)
+#define KB9202_SMC2_RWSETUP (1 << 24)
+#define KB9202_SMC2_RWHOLD (1 << 29)
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			writel(KB9202_NAND_NCE, AT91C_PIOC_CODR);
+		else
+			writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->options &= ~NAND_BUSWIDTH_16;
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	writel(1 << AT91C_ID_PIOC, AT91C_PMC_PCER);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
+				 AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
+				 AT91C_SMC_CSR3);
+
+	/* enable internal NAND controller */
+	value = readl(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	writel(value, AT91C_EBI_CSA);
+
+	/* enable SMOE/SMWE */
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_ASR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_PDR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_OER);
+
+	/* set NCE to high */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+
+	/* disable output on pin connected to the busy line of the NAND */
+	writel(KB9202_NAND_BUSY, AT91C_PIOC_ODR);
+
+	/* enable the PIO to control NCE and BUSY */
+	writel(KB9202_NAND_NCE | KB9202_NAND_BUSY, AT91C_PIOC_PER);
+
+	/* enable output for NCE */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_OER);
+
+	return (0);
+}
diff --git a/include/asm-arm/arch-at91rm9200/AT91RM9200.h b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
index 00bae1c..aaab2c2 100644
--- a/include/asm-arm/arch-at91rm9200/AT91RM9200.h
+++ b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
@@ -737,6 +737,7 @@ typedef struct _AT91S_PDC
 #define AT91C_SMC2_ACSS_STANDARD	((unsigned int) 0x0 << 16) /* (SMC2) Standard, asserted at the beginning of the access and deasserted@the end. */
 #define AT91C_SMC2_DBW_8	((unsigned int) 0x2 << 13) /* (SMC2) 8-bit. */
 #define AT91C_SMC2_WSEN		((unsigned int) 0x1 <<  7) /* (SMC2) Wait State Enable */
+#define AT91C_PIOC_OER		((AT91_REG *)	0xFFFFF810) /* (PIOC) PIO Output Enable Register */
 #define AT91C_PIOC_ASR		((AT91_REG *)	0xFFFFF870) /* (PIOC) Select A Register */
 #define AT91C_PIOC_SODR		((AT91_REG *)	0xFFFFF830) /* (PIOC) Set Output Data Register */
 #define AT91C_PIOC_CODR		((AT91_REG *)	0xFFFFF834) /* (PIOC) Clear Output Data Register */
@@ -780,6 +781,7 @@ typedef struct _AT91S_PDC
 #define AT91C_PIOB_PER		((AT91_REG *)	0xFFFFF600) /* (PIOB) PIO Enable Register */
 #define AT91C_PIOB_ODR		((AT91_REG *)	0xFFFFF614) /* (PIOB) Output Disable Registerr */
 #define AT91C_PIOB_PDSR		((AT91_REG *)	0xFFFFF63C) /* (PIOB) Pin Data Status Register */
+#define AT91C_SMC_CSR3		((AT91_REG *)	0xFFFFFF7C) /* (SMC) Chip Select Register 3 */
 
 #else
 /* flash */
diff --git a/include/configs/kb9202.h b/include/configs/kb9202.h
index 8651e53..c51845e 100644
--- a/include/configs/kb9202.h
+++ b/include/configs/kb9202.h
@@ -62,6 +62,12 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define CONFIG_NAND_KB9202
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +77,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -112,6 +118,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

      The assumption that what currently exists must necessarily
        exist is the acid that corrodes all visionary thinking
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2] KB9202: Add NAND support
  2009-06-04 18:50           ` Matthias Kaehlcke
@ 2009-06-04 19:08             ` Wolfgang Denk
  2009-06-11 18:46               ` [U-Boot] [PATCH 2/2 v4] " Matthias Kaehlcke
  0 siblings, 1 reply; 21+ messages in thread
From: Wolfgang Denk @ 2009-06-04 19:08 UTC (permalink / raw)
  To: u-boot

Dear Matthias Kaehlcke,

In message <20090604185024.GB4398@darwin> you wrote:
> 
> thanks for your explication, below is the driver in the corresponding
> directory
> 
> --

Suchg comment is supposed to go *below* the "--" line, not above.

> Add NAND support for the KwikByte KB9202

And such a commit message must go *above* the "--" line, not below.

Also, it would be nice if you added some version information (like
"[PATCH 2/2 v3]" in the Subject: line, and summarize the changes
between versions below the "--" line.

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Old programmers never die, they just become managers.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v4] KB9202: Add NAND support
  2009-06-04 19:08             ` Wolfgang Denk
@ 2009-06-11 18:46               ` Matthias Kaehlcke
  2009-06-23 21:19                 ` Scott Wood
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-11 18:46 UTC (permalink / raw)
  To: u-boot

Add NAND support for the KwikByte KB9202

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>

--

Changes:

- moved driver to drivers/mtd/nand/
- use i/o accessors
- don't check for ATL custom board
- removed unnecessary cast
- don't use magic numbers

 drivers/mtd/nand/Makefile                    |    1 +
 drivers/mtd/nand/kb9202_nand.c               |  151 ++++++++++++++++++++++++++
 include/asm-arm/arch-at91rm9200/AT91RM9200.h |    2 +
 include/configs/kb9202.h                     |   11 ++-
 4 files changed, 164 insertions(+), 1 deletions(-)
 create mode 100644 drivers/mtd/nand/kb9202_nand.c

diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index 471cd6b..d2f3e61 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -44,6 +44,7 @@ COBJS-$(CONFIG_NAND_NOMADIK) += nomadik.o
 COBJS-$(CONFIG_NAND_S3C2410) += s3c2410_nand.c
 COBJS-$(CONFIG_NAND_S3C64XX) += s3c64xx.o
 COBJS-$(CONFIG_NAND_OMAP_GPMC) += omap_gpmc.o
+COBJS-$(CONFIG_NAND_KB9202) += kb9202_nand.o
 endif
 
 COBJS	:= $(COBJS-y)
diff --git a/drivers/mtd/nand/kb9202_nand.c b/drivers/mtd/nand/kb9202_nand.c
new file mode 100644
index 0000000..7db8d7c
--- /dev/null
+++ b/drivers/mtd/nand/kb9202_nand.c
@@ -0,0 +1,151 @@
+/*
+ * (C) Copyright 2006
+ * KwikByte <kb9200_dev@kwikbyte.com>
+ *
+ * (C) Copyright 2009
+ * Matthias Kaehlcke <matthias@kaehlcke.net>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+#define KB9202_SMC2_NWS (1 << 2)
+#define KB9202_SMC2_TDF (1 << 8)
+#define KB9202_SMC2_RWSETUP (1 << 24)
+#define KB9202_SMC2_RWHOLD (1 << 29)
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			writel(KB9202_NAND_NCE, AT91C_PIOC_CODR);
+		else
+			writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->options &= ~NAND_BUSWIDTH_16;
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	writel(1 << AT91C_ID_PIOC, AT91C_PMC_PCER);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
+				 AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
+				 AT91C_SMC_CSR3);
+
+	/* enable internal NAND controller */
+	value = readl(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	writel(value, AT91C_EBI_CSA);
+
+	/* enable SMOE/SMWE */
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_ASR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_PDR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_OER);
+
+	/* set NCE to high */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+
+	/* disable output on pin connected to the busy line of the NAND */
+	writel(KB9202_NAND_BUSY, AT91C_PIOC_ODR);
+
+	/* enable the PIO to control NCE and BUSY */
+	writel(KB9202_NAND_NCE | KB9202_NAND_BUSY, AT91C_PIOC_PER);
+
+	/* enable output for NCE */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_OER);
+
+	return (0);
+}
diff --git a/include/asm-arm/arch-at91rm9200/AT91RM9200.h b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
index 00bae1c..aaab2c2 100644
--- a/include/asm-arm/arch-at91rm9200/AT91RM9200.h
+++ b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
@@ -737,6 +737,7 @@ typedef struct _AT91S_PDC
 #define AT91C_SMC2_ACSS_STANDARD	((unsigned int) 0x0 << 16) /* (SMC2) Standard, asserted at the beginning of the access and deasserted@the end. */
 #define AT91C_SMC2_DBW_8	((unsigned int) 0x2 << 13) /* (SMC2) 8-bit. */
 #define AT91C_SMC2_WSEN		((unsigned int) 0x1 <<  7) /* (SMC2) Wait State Enable */
+#define AT91C_PIOC_OER		((AT91_REG *)	0xFFFFF810) /* (PIOC) PIO Output Enable Register */
 #define AT91C_PIOC_ASR		((AT91_REG *)	0xFFFFF870) /* (PIOC) Select A Register */
 #define AT91C_PIOC_SODR		((AT91_REG *)	0xFFFFF830) /* (PIOC) Set Output Data Register */
 #define AT91C_PIOC_CODR		((AT91_REG *)	0xFFFFF834) /* (PIOC) Clear Output Data Register */
@@ -780,6 +781,7 @@ typedef struct _AT91S_PDC
 #define AT91C_PIOB_PER		((AT91_REG *)	0xFFFFF600) /* (PIOB) PIO Enable Register */
 #define AT91C_PIOB_ODR		((AT91_REG *)	0xFFFFF614) /* (PIOB) Output Disable Registerr */
 #define AT91C_PIOB_PDSR		((AT91_REG *)	0xFFFFF63C) /* (PIOB) Pin Data Status Register */
+#define AT91C_SMC_CSR3		((AT91_REG *)	0xFFFFFF7C) /* (SMC) Chip Select Register 3 */
 
 #else
 /* flash */
diff --git a/include/configs/kb9202.h b/include/configs/kb9202.h
index 8651e53..c51845e 100644
--- a/include/configs/kb9202.h
+++ b/include/configs/kb9202.h
@@ -62,6 +62,12 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define CONFIG_NAND_KB9202
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +77,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -112,6 +118,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

               If sharing a thing in no way diminishes it,
               it is not rightly owned if it is not shared
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v4] KB9202: Add NAND support
  2009-06-11 18:46               ` [U-Boot] [PATCH 2/2 v4] " Matthias Kaehlcke
@ 2009-06-23 21:19                 ` Scott Wood
  2009-06-24 15:23                   ` [U-Boot] [PATCH 2/2 v5] " Matthias Kaehlcke
  0 siblings, 1 reply; 21+ messages in thread
From: Scott Wood @ 2009-06-23 21:19 UTC (permalink / raw)
  To: u-boot

On Thu, Jun 11, 2009 at 08:46:54PM +0200, Matthias Kaehlcke wrote:
> Add NAND support for the KwikByte KB9202
> 
> Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> 
> --

The git separator is three dashes, not two.

> Changes:
> 
> - moved driver to drivers/mtd/nand/
> - use i/o accessors
> - don't check for ATL custom board
> - removed unnecessary cast
> - don't use magic numbers
> 
>  drivers/mtd/nand/Makefile                    |    1 +
>  drivers/mtd/nand/kb9202_nand.c               |  151 ++++++++++++++++++++++++++
>  include/asm-arm/arch-at91rm9200/AT91RM9200.h |    2 +
>  include/configs/kb9202.h                     |   11 ++-

I get conflicts in kb9202.h.  Is this against an arch tree, or does it
need to be respun?

> diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
> index 471cd6b..d2f3e61 100644
> --- a/drivers/mtd/nand/Makefile
> +++ b/drivers/mtd/nand/Makefile
> @@ -44,6 +44,7 @@ COBJS-$(CONFIG_NAND_NOMADIK) += nomadik.o
>  COBJS-$(CONFIG_NAND_S3C2410) += s3c2410_nand.c
>  COBJS-$(CONFIG_NAND_S3C64XX) += s3c64xx.o
>  COBJS-$(CONFIG_NAND_OMAP_GPMC) += omap_gpmc.o
> +COBJS-$(CONFIG_NAND_KB9202) += kb9202_nand.o

Wolfgang likes these things kept in alphabetical order.

> +int board_nand_init(struct nand_chip *nand)
> +{
> +	unsigned int value;
> +
> +	nand->ecc.mode = NAND_ECC_SOFT;
> +	nand->options &= ~NAND_BUSWIDTH_16;

Why would you need to clear this?  It's the board driver that sets it in
the first place (if applicable).

> +	/* setup nand flash access (allow ample margin) */
> +	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
> +	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
> +				 AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
> +				 AT91C_SMC_CSR3);
> +

Line length.  Did you perhaps use a tab size other than 8?

Otherwise, ACK.

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v5] KB9202: Add NAND support
  2009-06-23 21:19                 ` Scott Wood
@ 2009-06-24 15:23                   ` Matthias Kaehlcke
  2009-06-24 20:51                     ` Scott Wood
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-06-24 15:23 UTC (permalink / raw)
  To: u-boot

Add NAND support for the KwikByte KB9202

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>

---

El Tue, Jun 23, 2009 at 04:19:40PM -0500 Scott Wood ha dit:

> I get conflicts in kb9202.h.  Is this against an arch tree, or does it
> need to be respun?

The previous patches were against v2009-03, as i had problems
building the current git head for the kb9202 when i started working on
the patch. Did you apply the first patch of this series when you got
the conflict? This patch is based on the current HEAD.

Changes regarding v4:
 * put KB9202 entry in the Makefile at the correct position
 (alphabetical order)
 * removed superfluous clear of NAND_BUSWIDTH_16 flag
 * fixed line length/broken indentation

 drivers/mtd/nand/Makefile                    |    1 +
 drivers/mtd/nand/kb9202_nand.c               |  150 ++++++++++++++++++++++++++
 include/asm-arm/arch-at91rm9200/AT91RM9200.h |    2 +
 include/configs/kb9202.h                     |   11 ++-
 4 files changed, 163 insertions(+), 1 deletions(-)
 create mode 100644 drivers/mtd/nand/kb9202_nand.c

diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index 71dd5b9..d2bd558 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -40,6 +40,7 @@ COBJS-$(CONFIG_DRIVER_NAND_BFIN) += bfin_nand.o
 COBJS-$(CONFIG_NAND_DAVINCI) += davinci_nand.o
 COBJS-$(CONFIG_NAND_FSL_ELBC) += fsl_elbc_nand.o
 COBJS-$(CONFIG_NAND_FSL_UPM) += fsl_upm.o
+COBJS-$(CONFIG_NAND_KB9202) += kb9202_nand.o
 COBJS-$(CONFIG_NAND_MPC5121_NFC) += mpc5121_nfc.o
 COBJS-$(CONFIG_NAND_NOMADIK) += nomadik.o
 COBJS-$(CONFIG_NAND_S3C2410) += s3c2410_nand.c
diff --git a/drivers/mtd/nand/kb9202_nand.c b/drivers/mtd/nand/kb9202_nand.c
new file mode 100644
index 0000000..b8f46fa
--- /dev/null
+++ b/drivers/mtd/nand/kb9202_nand.c
@@ -0,0 +1,150 @@
+/*
+ * (C) Copyright 2006
+ * KwikByte <kb9200_dev@kwikbyte.com>
+ *
+ * (C) Copyright 2009
+ * Matthias Kaehlcke <matthias@kaehlcke.net>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+#define KB9202_SMC2_NWS (1 << 2)
+#define KB9202_SMC2_TDF (1 << 8)
+#define KB9202_SMC2_RWSETUP (1 << 24)
+#define KB9202_SMC2_RWHOLD (1 << 29)
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			writel(KB9202_NAND_NCE, AT91C_PIOC_CODR);
+		else
+			writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	writel(1 << AT91C_ID_PIOC, AT91C_PMC_PCER);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
+		AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
+		AT91C_SMC_CSR3);
+
+	/* enable internal NAND controller */
+	value = readl(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	writel(value, AT91C_EBI_CSA);
+
+	/* enable SMOE/SMWE */
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_ASR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_PDR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_OER);
+
+	/* set NCE to high */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+
+	/* disable output on pin connected to the busy line of the NAND */
+	writel(KB9202_NAND_BUSY, AT91C_PIOC_ODR);
+
+	/* enable the PIO to control NCE and BUSY */
+	writel(KB9202_NAND_NCE | KB9202_NAND_BUSY, AT91C_PIOC_PER);
+
+	/* enable output for NCE */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_OER);
+
+	return (0);
+}
diff --git a/include/asm-arm/arch-at91rm9200/AT91RM9200.h b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
index 00bae1c..aaab2c2 100644
--- a/include/asm-arm/arch-at91rm9200/AT91RM9200.h
+++ b/include/asm-arm/arch-at91rm9200/AT91RM9200.h
@@ -737,6 +737,7 @@ typedef struct _AT91S_PDC
 #define AT91C_SMC2_ACSS_STANDARD	((unsigned int) 0x0 << 16) /* (SMC2) Standard, asserted at the beginning of the access and deasserted@the end. */
 #define AT91C_SMC2_DBW_8	((unsigned int) 0x2 << 13) /* (SMC2) 8-bit. */
 #define AT91C_SMC2_WSEN		((unsigned int) 0x1 <<  7) /* (SMC2) Wait State Enable */
+#define AT91C_PIOC_OER		((AT91_REG *)	0xFFFFF810) /* (PIOC) PIO Output Enable Register */
 #define AT91C_PIOC_ASR		((AT91_REG *)	0xFFFFF870) /* (PIOC) Select A Register */
 #define AT91C_PIOC_SODR		((AT91_REG *)	0xFFFFF830) /* (PIOC) Set Output Data Register */
 #define AT91C_PIOC_CODR		((AT91_REG *)	0xFFFFF834) /* (PIOC) Clear Output Data Register */
@@ -780,6 +781,7 @@ typedef struct _AT91S_PDC
 #define AT91C_PIOB_PER		((AT91_REG *)	0xFFFFF600) /* (PIOB) PIO Enable Register */
 #define AT91C_PIOB_ODR		((AT91_REG *)	0xFFFFF614) /* (PIOB) Output Disable Registerr */
 #define AT91C_PIOB_PDSR		((AT91_REG *)	0xFFFFF63C) /* (PIOB) Pin Data Status Register */
+#define AT91C_SMC_CSR3		((AT91_REG *)	0xFFFFFF7C) /* (SMC) Chip Select Register 3 */
 
 #else
 /* flash */
diff --git a/include/configs/kb9202.h b/include/configs/kb9202.h
index 8651e53..c51845e 100644
--- a/include/configs/kb9202.h
+++ b/include/configs/kb9202.h
@@ -62,6 +62,12 @@
 #ifdef CONFIG_KB9202B
 #define	CONFIG_BOOTARGS	"console=ttyS0,115200 noinitrd root=/dev/mtdblock0 rootfstype=jffs2 mem=64M"
 #define	CONFIG_BOOTCOMMAND	"bootm 0x10000000"
+#define CONFIG_NAND_KB9202
+#define NAND_MAX_CHIPS 1
+#define CONFIG_SYS_MAX_NAND_DEVICE 1
+#define CONFIG_SYS_NAND_BASE 0x40000000
+#define CONFIG_JFFS2_NAND
+#define CONFIG_JFFS2_CMDLINE
 #endif
 
 
@@ -71,7 +77,7 @@
 /*
  * Size of malloc() pool
  */
-#define CONFIG_SYS_MALLOC_LEN	(128*1024)
+#define CONFIG_SYS_MALLOC_LEN (1024*1024)
 #define CONFIG_SYS_GBL_DATA_SIZE	128	/* size in bytes reserved for initial data */
 
 #define CONFIG_BAUDRATE 115200
@@ -112,6 +118,9 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_DHCP
+#ifdef CONFIG_KB9202B
+#define CONFIG_CMD_NAND
+#endif
 #define CONFIG_CMD_JFFS2
 
 #undef CONFIG_CMD_BDI

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona


  The salvation of mankind lies only in making everything the concern of all
                         (Alexander Solzhenitsyn)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v5] KB9202: Add NAND support
  2009-06-24 15:23                   ` [U-Boot] [PATCH 2/2 v5] " Matthias Kaehlcke
@ 2009-06-24 20:51                     ` Scott Wood
  2009-07-12 19:57                       ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 1 reply; 21+ messages in thread
From: Scott Wood @ 2009-06-24 20:51 UTC (permalink / raw)
  To: u-boot

On Wed, Jun 24, 2009 at 05:23:05PM +0200, Matthias Kaehlcke wrote:
> Add NAND support for the KwikByte KB9202
> 
> Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> 
> ---
> 
> El Tue, Jun 23, 2009 at 04:19:40PM -0500 Scott Wood ha dit:
> 
> > I get conflicts in kb9202.h.  Is this against an arch tree, or does it
> > need to be respun?
> 
> The previous patches were against v2009-03, as i had problems
> building the current git head for the kb9202 when i started working on
> the patch. Did you apply the first patch of this series when you got
> the conflict? This patch is based on the current HEAD.

No - if it depends on other patches that go through an arch tree, then this
patch should go through that tree as well.

Acked-by: Scott Wood <scottwood@freescale.com>

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v5] KB9202: Add NAND support
  2009-06-24 20:51                     ` Scott Wood
@ 2009-07-12 19:57                       ` Jean-Christophe PLAGNIOL-VILLARD
  2009-07-16 18:45                         ` Scott Wood
  0 siblings, 1 reply; 21+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2009-07-12 19:57 UTC (permalink / raw)
  To: u-boot

On 15:51 Wed 24 Jun     , Scott Wood wrote:
> On Wed, Jun 24, 2009 at 05:23:05PM +0200, Matthias Kaehlcke wrote:
> > Add NAND support for the KwikByte KB9202
> > 
> > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > 
> > ---
> > 
> > El Tue, Jun 23, 2009 at 04:19:40PM -0500 Scott Wood ha dit:
> > 
> > > I get conflicts in kb9202.h.  Is this against an arch tree, or does it
> > > need to be respun?
> > 
> > The previous patches were against v2009-03, as i had problems
> > building the current git head for the kb9202 when i started working on
> > the patch. Did you apply the first patch of this series when you got
> > the conflict? This patch is based on the current HEAD.
> 
> No - if it depends on other patches that go through an arch tree, then this
> patch should go through that tree as well.
> 
> Acked-by: Scott Wood <scottwood@freescale.com>
Scott I've not follow this patch but for the arm part it's looks fine so I'll
let you handle

Best Regards,
J.

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v5] KB9202: Add NAND support
  2009-07-12 19:57                       ` Jean-Christophe PLAGNIOL-VILLARD
@ 2009-07-16 18:45                         ` Scott Wood
  2009-07-16 19:19                           ` [U-Boot] [PATCH 2/2 v6] " Matthias Kaehlcke
  0 siblings, 1 reply; 21+ messages in thread
From: Scott Wood @ 2009-07-16 18:45 UTC (permalink / raw)
  To: u-boot

On Sun, Jul 12, 2009 at 09:57:13PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 15:51 Wed 24 Jun     , Scott Wood wrote:
> > On Wed, Jun 24, 2009 at 05:23:05PM +0200, Matthias Kaehlcke wrote:
> > > Add NAND support for the KwikByte KB9202
> > > 
> > > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > > 
> > > ---
> > > 
> > > El Tue, Jun 23, 2009 at 04:19:40PM -0500 Scott Wood ha dit:
> > > 
> > > > I get conflicts in kb9202.h.  Is this against an arch tree, or does it
> > > > need to be respun?
> > > 
> > > The previous patches were against v2009-03, as i had problems
> > > building the current git head for the kb9202 when i started working on
> > > the patch. Did you apply the first patch of this series when you got
> > > the conflict? This patch is based on the current HEAD.
> > 
> > No - if it depends on other patches that go through an arch tree, then this
> > patch should go through that tree as well.
> > 
> > Acked-by: Scott Wood <scottwood@freescale.com>
> Scott I've not follow this patch but for the arm part it's looks fine so I'll
> let you handle

It still does not apply to the current tree, so I cannot accept it as is. 

The NAND tree is not an appropriate route for unrelated changes to a
board config file (such as CONFIG_BOOTARGS/CONFIG_BOOTCOMMAND) to go --
and even if it were, I don't have an easily applyable version of patch
1/2 (the mailing list archives mangle things).

Matthias, can you resend a patch that just adds the NAND driver (for me),
and a separate patch that just touches the board config (for the arch
tree)?

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v6] KB9202: Add NAND support
  2009-07-16 18:45                         ` Scott Wood
@ 2009-07-16 19:19                           ` Matthias Kaehlcke
  2009-08-06 21:47                             ` Scott Wood
  0 siblings, 1 reply; 21+ messages in thread
From: Matthias Kaehlcke @ 2009-07-16 19:19 UTC (permalink / raw)
  To: u-boot

Add KB9202 NAND driver

Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>

---

El Thu, Jul 16, 2009 at 01:45:57PM -0500 Scott Wood ha dit:

> On Sun, Jul 12, 2009 at 09:57:13PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 15:51 Wed 24 Jun     , Scott Wood wrote:
> > > On Wed, Jun 24, 2009 at 05:23:05PM +0200, Matthias Kaehlcke wrote:
> > > > Add NAND support for the KwikByte KB9202
> > > > 
> > > > Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> > > > 
> > > > ---
> > > > 
> > > > El Tue, Jun 23, 2009 at 04:19:40PM -0500 Scott Wood ha dit:
> > > > 
> > > > > I get conflicts in kb9202.h.  Is this against an arch tree, or does it
> > > > > need to be respun?
> > > > 
> > > > The previous patches were against v2009-03, as i had problems
> > > > building the current git head for the kb9202 when i started working on
> > > > the patch. Did you apply the first patch of this series when you got
> > > > the conflict? This patch is based on the current HEAD.
> > > 
> > > No - if it depends on other patches that go through an arch tree, then this
> > > patch should go through that tree as well.
> > > 
> > > Acked-by: Scott Wood <scottwood@freescale.com>
> > Scott I've not follow this patch but for the arm part it's looks fine so I'll
> > let you handle
> 
> It still does not apply to the current tree, so I cannot accept it as is. 
> 
> The NAND tree is not an appropriate route for unrelated changes to a
> board config file (such as CONFIG_BOOTARGS/CONFIG_BOOTCOMMAND) to go --
> and even if it were, I don't have an easily applyable version of patch
> 1/2 (the mailing list archives mangle things).
> 
> Matthias, can you resend a patch that just adds the NAND driver (for me),
> and a separate patch that just touches the board config (for the arch
> tree)?

ok, here is a patch that only adds the NAND driver. i'll prepare a
board config patch after my first patch has hit mainline to avoid
further confusion :)

changes regarding v5:

 * only add the NAND driver, without touching the board configuration

 drivers/mtd/nand/Makefile      |    1 +
 drivers/mtd/nand/kb9202_nand.c |  150 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 151 insertions(+), 0 deletions(-)
 create mode 100644 drivers/mtd/nand/kb9202_nand.c

diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index a5680e8..aab3b70 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -40,6 +40,7 @@ COBJS-$(CONFIG_DRIVER_NAND_BFIN) += bfin_nand.o
 COBJS-$(CONFIG_NAND_DAVINCI) += davinci_nand.o
 COBJS-$(CONFIG_NAND_FSL_ELBC) += fsl_elbc_nand.o
 COBJS-$(CONFIG_NAND_FSL_UPM) += fsl_upm.o
+COBJS-$(CONFIG_NAND_KB9202) += kb9202_nand.o
 COBJS-$(CONFIG_NAND_KIRKWOOD) += kirkwood_nand.o
 COBJS-$(CONFIG_NAND_MPC5121_NFC) += mpc5121_nfc.o
 COBJS-$(CONFIG_NAND_NOMADIK) += nomadik.o
diff --git a/drivers/mtd/nand/kb9202_nand.c b/drivers/mtd/nand/kb9202_nand.c
new file mode 100644
index 0000000..b8f46fa
--- /dev/null
+++ b/drivers/mtd/nand/kb9202_nand.c
@@ -0,0 +1,150 @@
+/*
+ * (C) Copyright 2006
+ * KwikByte <kb9200_dev@kwikbyte.com>
+ *
+ * (C) Copyright 2009
+ * Matthias Kaehlcke <matthias@kaehlcke.net>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/AT91RM9200.h>
+#include <asm/arch/hardware.h>
+
+#include <nand.h>
+
+/*
+ *      hardware specific access to control-lines
+ */
+
+#define MASK_ALE        (1 << 22)       /* our ALE is A22 */
+#define MASK_CLE        (1 << 21)       /* our CLE is A21 */
+
+#define KB9202_NAND_NCE (1 << 28) /* EN* on D28 */
+#define KB9202_NAND_BUSY (1 << 29) /* RB* on D29 */
+
+#define KB9202_SMC2_NWS (1 << 2)
+#define KB9202_SMC2_TDF (1 << 8)
+#define KB9202_SMC2_RWSETUP (1 << 24)
+#define KB9202_SMC2_RWHOLD (1 << 29)
+
+/*
+ *	Board-specific function to access device control signals
+ */
+static void kb9202_nand_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl)
+{
+	struct nand_chip *this = mtd->priv;
+
+	if (ctrl & NAND_CTRL_CHANGE) {
+		ulong IO_ADDR_W = (ulong) this->IO_ADDR_W;
+
+		/* clear ALE and CLE bits */
+		IO_ADDR_W &= ~(MASK_ALE | MASK_CLE);
+
+		if (ctrl & NAND_CLE)
+			IO_ADDR_W |= MASK_CLE;
+
+		if (ctrl & NAND_ALE)
+			IO_ADDR_W |= MASK_ALE;
+
+		this->IO_ADDR_W = (void *) IO_ADDR_W;
+
+		if (ctrl & NAND_NCE)
+			writel(KB9202_NAND_NCE, AT91C_PIOC_CODR);
+		else
+			writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+	}
+
+	if (cmd != NAND_CMD_NONE)
+		writeb(cmd, this->IO_ADDR_W);
+}
+
+
+/*
+ * Board-specific function to access the device ready signal.
+ */
+static int kb9202_nand_ready(struct mtd_info *mtd)
+{
+	return readl(AT91C_PIOC_PDSR) & KB9202_NAND_BUSY;
+}
+
+
+/*
+ * Board-specific NAND init.  Copied from include/linux/mtd/nand.h for reference.
+ *
+ * struct nand_chip - NAND Private Flash Chip Data
+ * @IO_ADDR_R:		[BOARDSPECIFIC] address to read the 8 I/O lines of the flash device
+ * @IO_ADDR_W:		[BOARDSPECIFIC] address to write the 8 I/O lines of the flash device
+ * @hwcontrol:		[BOARDSPECIFIC] hardwarespecific function for accesing control-lines
+ * @dev_ready:		[BOARDSPECIFIC] hardwarespecific function for accesing device ready/busy line
+ *			If set to NULL no access to ready/busy is available and the ready/busy information
+ *			is read from the chip status register
+ * @enable_hwecc:	[BOARDSPECIFIC] function to enable (reset) hardware ecc generator. Must only
+ *			be provided if a hardware ECC is available
+ * @eccmode:		[BOARDSPECIFIC] mode of ecc, see defines
+ * @chip_delay:		[BOARDSPECIFIC] chip dependent delay for transfering data from array to read regs (tR)
+ * @options:		[BOARDSPECIFIC] various chip options. They can partly be set to inform nand_scan about
+ *			special functionality. See the defines for further explanation
+*/
+/*
+ * This routine initializes controller and GPIOs.
+ */
+int board_nand_init(struct nand_chip *nand)
+{
+	unsigned int value;
+
+	nand->ecc.mode = NAND_ECC_SOFT;
+	nand->cmd_ctrl = kb9202_nand_hwcontrol;
+	nand->dev_ready = kb9202_nand_ready;
+
+	/* in case running outside of bootloader */
+	writel(1 << AT91C_ID_PIOC, AT91C_PMC_PCER);
+
+	/* setup nand flash access (allow ample margin) */
+	/* 4 wait states, 1 setup, 1 hold, 1 float for 8-bit device */
+	writel(AT91C_SMC2_WSEN | KB9202_SMC2_NWS | KB9202_SMC2_TDF |
+		AT91C_SMC2_DBW_8 | KB9202_SMC2_RWSETUP | KB9202_SMC2_RWHOLD,
+		AT91C_SMC_CSR3);
+
+	/* enable internal NAND controller */
+	value = readl(AT91C_EBI_CSA);
+	value |= AT91C_EBI_CS3A_SMC_SmartMedia;
+	writel(value, AT91C_EBI_CSA);
+
+	/* enable SMOE/SMWE */
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_ASR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_PDR);
+	writel(AT91C_PC1_BFRDY_SMOE | AT91C_PC3_BFBAA_SMWE, AT91C_PIOC_OER);
+
+	/* set NCE to high */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_SODR);
+
+	/* disable output on pin connected to the busy line of the NAND */
+	writel(KB9202_NAND_BUSY, AT91C_PIOC_ODR);
+
+	/* enable the PIO to control NCE and BUSY */
+	writel(KB9202_NAND_NCE | KB9202_NAND_BUSY, AT91C_PIOC_PER);
+
+	/* enable output for NCE */
+	writel(KB9202_NAND_NCE, AT91C_PIOC_OER);
+
+	return (0);
+}

-- 
Matthias Kaehlcke
Embedded Linux Engineer
Barcelona

  Control over the use of one's ideas really constitutes control over other
  people's lives; and it is usually used to make their lives more difficult.
                          (Richard Stallman)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [U-Boot] [PATCH 2/2 v6] KB9202: Add NAND support
  2009-07-16 19:19                           ` [U-Boot] [PATCH 2/2 v6] " Matthias Kaehlcke
@ 2009-08-06 21:47                             ` Scott Wood
  0 siblings, 0 replies; 21+ messages in thread
From: Scott Wood @ 2009-08-06 21:47 UTC (permalink / raw)
  To: u-boot

On Thu, Jul 16, 2009 at 09:19:29PM +0200, Matthias Kaehlcke wrote:
> Add KB9202 NAND driver
> 
> Signed-off-by: Matthias Kaehlcke <matthias@kaehlcke.net>
> 

Applied to u-boot-nand-flash/next.

-Scott

^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2009-08-06 21:47 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-15 22:15 [U-Boot] [PATCH 2/2] KB9202: Add NAND support Matthias Kaehlcke
2009-05-15 22:30 ` Scott Wood
2009-05-16 14:11   ` Matthias Kaehlcke
2009-05-18 17:38     ` Scott Wood
2009-05-19  7:25       ` Matthias Kaehlcke
2009-06-03 17:42   ` Matthias Kaehlcke
2009-06-03 19:50     ` Scott Wood
2009-06-04 17:45       ` Matthias Kaehlcke
2009-06-04  6:53     ` Stefan Roese
2009-06-04 17:49       ` Matthias Kaehlcke
2009-06-04 18:08         ` Stefan Roese
2009-06-04 18:50           ` Matthias Kaehlcke
2009-06-04 19:08             ` Wolfgang Denk
2009-06-11 18:46               ` [U-Boot] [PATCH 2/2 v4] " Matthias Kaehlcke
2009-06-23 21:19                 ` Scott Wood
2009-06-24 15:23                   ` [U-Boot] [PATCH 2/2 v5] " Matthias Kaehlcke
2009-06-24 20:51                     ` Scott Wood
2009-07-12 19:57                       ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-16 18:45                         ` Scott Wood
2009-07-16 19:19                           ` [U-Boot] [PATCH 2/2 v6] " Matthias Kaehlcke
2009-08-06 21:47                             ` Scott Wood

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.