Fill in remaining MTD driver data for OneNAND.
Review onenand_print_device_info():
- Return device info to fill mtd device name.
- Remove verbose parameter as it become useless.
Signed-off-by: Fathi Boudra [EMAIL PROTECTED]
---
diff --git a/common/cmd_onenand.c b/common/cmd_onenand.c
index
Wolfgang Denk a écrit :
That's why I think we should have an explicit builtin command, i. e.
command with the name builtin that will always run the corresponding
builtin command, no if there is a command alias or not - similar to
the bash command
... and make sure that one can *never* define
JFFS2 command support on OneNAND.
- Fix typo.
- Fix nand_bbt_descr redefinition when NAND and OneNAND are enabled.
Since last comments:
- Add some CONFIG_CMD_ONENAND to prevent adding new code.
- Fix line too long.
- Revert mtd device types definition.
TODO: Use NAND command interface (e.g.:
In message [EMAIL PROTECTED] you wrote:
That would be REALLY cool! It would take some initial work, but the
reward would be really simple and transparent expandability for the
command set. As with the bootm command, we might end up with simpler
code (I don't think too many commands are
Wolfgang Denk wrote:
In message [EMAIL PROTECTED] you wrote:
My current best thought is to create a new boot simple (boots?
bootsm?) command that contains only the essence of bootm. I would then
change the command bootm to do a hush script run of the env variable
bootm (i.e. the command
Kumar Gala wrote:
If we really want to simplify what bootm does than I think we should
remove ft_board_setup() from lib_ppc/bootm.c and expect any actually
modification of the device tree to have already occurred.
Is this something we'd really be willing to do?
- k
I don't think so.
In message [EMAIL PROTECTED] you wrote:
My current best thought is to create a new boot simple (boots?
bootsm?) command that contains only the essence of bootm. I would then
change the command bootm to do a hush script run of the env variable
bootm (i.e. the command bootm would really
Hi,
TODO: Use NAND command interface (e.g.: part_validate_*nand()).
Do you mean merging the NAND and OneNAND versions of the jffs2
code, through the (mostly already existing) function pointer
interface?
yes, as you suggested to kyungmin: use mtd_info function pointer interface.
I'd
On Mon, Aug 4, 2008 at 9:02 PM, Becky Bruce [EMAIL PROTECTED] wrote:
Currently, they use CONFIG_NUM_CPUS, which is different than
85xx for no good reason.
--- a/include/configs/MPC8610HPCD.h
+++ b/include/configs/MPC8610HPCD.h
@@ -17,7 +17,7 @@
#define CONFIG_MPC86xx 1 /*
Wolfgang Denk wrote:
One minor flaw, I don't see how bootm the env script could run bootm
the built-in command, because it would instead recursively run bootm
the env script if scripts have higher priority and the command line
bootm would run the built-in bootm if scripts have lower
On Aug 5, 2008, at 5:19 AM, Wolfgang Denk wrote:
In message [EMAIL PROTECTED] you wrote:
My current best thought is to create a new boot simple (boots?
bootsm?) command that contains only the essence of bootm. I would
then
change the command bootm to do a hush script run of the env
In message [EMAIL PROTECTED] you wrote:
That's why I think we should have an explicit builtin command, i. e.
command with the name builtin that will always run the corresponding
builtin command, no if there is a command alias or not - similar to
the bash command
... and make sure that
On Mon, 4 Aug 2008, Scott Wood wrote:
On Mon, Aug 04, 2008 at 02:45:33PM +0200, Guennadi Liakhovetski wrote:
I _think_ this should work with all NAND chips. Otherwise we might have to
introduce a configuration variable.
Which small-page NAND chips can't handle READOOB? On large page
Version 5: This time based on nand-flash/testing head. Therefore it
couldn't be runtime-tested: largepage support in nand_spl is broken there
ATM. Otherwise all comments have been taken care of. After nand_spl is
fixed for largepage NAND devices, this will be tested, if needed, v6 will
be
Signed-off-by: Guennadi Liakhovetski [EMAIL PROTECTED]
---
drivers/mtd/jedec_flash.c | 17 +
1 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/drivers/mtd/jedec_flash.c b/drivers/mtd/jedec_flash.c
index 9845e93..020647a 100644
--- a/drivers/mtd/jedec_flash.c
+++
We have to read a whole page on large-page NAND chips, at least the chip
installed on SMDK6400 can only read whole pages with READ0. Also activate
chipselect before using the chip, and deactivate it afterwards.
Signed-off-by: Guennadi Liakhovetski [EMAIL PROTECTED]
---
nand_spl/nand_boot.c |
Notice: USB on S3C6400 currently works _only_ with switched off MMU. One could
try to enable the MMU, but map addresses 1-to-1, and disable data cache, then
it should work too and we could still profit from instruction cache.
Signed-off-by: Guennadi Liakhovetski [EMAIL PROTECTED]
---
Based on the original S3C64XX UART driver by Samsung for U-Boot 1.1.6.
Signed-off-by: Guennadi Liakhovetski [EMAIL PROTECTED]
---
drivers/serial/Makefile |1 +
drivers/serial/s3c64xx.c | 197 ++
2 files changed, 198 insertions(+), 0 deletions(-)
Kumar Gala wrote:
On Aug 5, 2008, at 5:19 AM, Wolfgang Denk wrote:
[snip]
One idea that has been spinning in my mind for some time is to make
the run command to execute the content of an environment variable
optional. Instead, we could try and handle environment variable names
similar
Based on the original S3C64XX NAND driver by Samsung for U-Boot 1.1.6.
Signed-off-by: Guennadi Liakhovetski [EMAIL PROTECTED]
---
drivers/mtd/nand/Makefile |1 +
drivers/mtd/nand/s3c64xx.c | 305
2 files changed, 306 insertions(+), 0
SMDK6400 can only boot U-Boot from NAND-flash. This patch adds a nand_spl
driver for it too. The board can also boot from the NOR flash, but due to
hardware limitations it can only address 64KiB on it, which is not enough
for U-Boot. Based on the original sources by Samsung for U-Boot 1.1.6.
Wolfgang Denk a écrit :
In message [EMAIL PROTECTED] you wrote:
That's why I think we should have an explicit builtin command, i. e.
command with the name builtin that will always run the corresponding
builtin command, no if there is a command alias or not - similar to
the bash command
...
On Aug 5, 2008, at 8:36 AM, Jerry Van Baren wrote:
Kumar Gala wrote:
On Aug 5, 2008, at 5:19 AM, Wolfgang Denk wrote:
[snip]
One idea that has been spinning in my mind for some time is to
make
the run command to execute the content of an environment
variable
optional. Instead,
Can we drop any functionality from the current bootm?
For example does powerpc still need to support bd_t based booting?
- k
-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest
here's a rough start at an outline for the bootm script based on the
code (I've only outlined the Linux/PPC boot case its seems the most
complicated). One of the first things we clearly need is a imload
command. Thoughts on the various disable_{interrupts, usb, caches} ?
- k
bootm
In message [EMAIL PROTECTED] you wrote:
Can we drop any functionality from the current bootm?
Any? You mean:
int bootm (...)
{
return 0;
}
:-)
For example does powerpc still need to support bd_t based booting?
Yes, of course we need this. There is
On Aug 5, 2008, at 9:45 AM, Wolfgang Denk wrote:
In message
[EMAIL PROTECTED] you wrote:
Can we drop any functionality from the current bootm?
Any? You mean:
int bootm (...)
{
return 0;
}
:-)
if it were only so easy :)
For example does powerpc
On Tue, Aug 05, 2008 at 03:08:04PM +0200, Guennadi Liakhovetski wrote:
It's a large-page device. And, as far as I understand the datasheet, to
read data at arbitrary offset in a page, you first have to issue a READ
PAGE (READ0) for _the_ _whole_ page, then you can use RANDOM DATA READ to
On Aug 5, 2008, at 8:36 AM, Jerry Van Baren wrote:
Kumar Gala wrote:
On Aug 5, 2008, at 5:19 AM, Wolfgang Denk wrote:
[snip]
One idea that has been spinning in my mind for some time is to
make
the run command to execute the content of an environment
variable
optional. Instead,
On Tue, Aug 05, 2008 at 03:36:33PM +0200, Guennadi Liakhovetski wrote:
Version 5: This time based on nand-flash/testing head. Therefore it
couldn't be runtime-tested: largepage support in nand_spl is broken there
ATM.
Hmm, so it is. I don't have hardware that uses the cmd_ctrl interface to
On Tue, 5 Aug 2008, Scott Wood wrote:
On Tue, Aug 05, 2008 at 03:36:33PM +0200, Guennadi Liakhovetski wrote:
Version 5: This time based on nand-flash/testing head. Therefore it
couldn't be runtime-tested: largepage support in nand_spl is broken there
ATM.
Hmm, so it is. I don't have
On Tue, Aug 05, 2008 at 03:36:43PM +0200, Guennadi Liakhovetski wrote:
/*
- * offs has to be aligned to a block address!
+ * offs has to be aligned to a page address!
*/
block = offs / CFG_NAND_BLOCK_SIZE;
+ /* Recalculate offs as an offset inside a block */
On Tue, 5 Aug 2008, Scott Wood wrote:
On Tue, Aug 05, 2008 at 03:08:04PM +0200, Guennadi Liakhovetski wrote:
It's a large-page device. And, as far as I understand the datasheet, to
read data at arbitrary offset in a page, you first have to issue a READ
PAGE (READ0) for _the_ _whole_
Guennadi Liakhovetski wrote:
On Tue, 5 Aug 2008, Scott Wood wrote:
Are you saying that your NAND chip can't read the OOB by issuing READ0
with the appropriate column address? Which chip is this, and where can I
find a manual?
At least, this is how I understood it, I might be wrong though:
boot_get_ramdisk() should not treat the case when a FIT image does not contain
a ramdisk as an error.
Signed-off-by: Peter Tyser [EMAIL PROTECTED]
---
The original code would not allow booting of a FIT image which didn't contain a
ramdisk. The bug was observed and fixed on a powerpc 85xx
Kumar Gala wrote:
On Aug 5, 2008, at 8:36 AM, Jerry Van Baren wrote:
Kumar Gala wrote:
On Aug 5, 2008, at 5:19 AM, Wolfgang Denk wrote:
[snip]
What do you think?
While this is a cleaner implementation of what I've implemented w/
ft_env_setup() it still doesn't completely solve my
[snip]
As I look at this more and more I think trying to re-encode the
control flow of the bootm command in a script is just insane.
There are too many special cases we have to deal with that we'd
just being moving from C code into the script.
My assumption is that a given
On Tue, Aug 05, 2008 at 11:06:28AM +0200, Fathi BOUDRA wrote:
-void onenand_print_device_info(int device, int verbose)
+char * onenand_print_device_info(int device)
{
int vcc, demuxed, ddp, density;
-
- if (!verbose)
- return;
+ char *dev_info;
vcc =
Added the 'fdtcmd' environment variable as a way to provide 'fdt' commands
that the user can supply to manipulate the device tree after
ft_board_setup() and before the tree is handled to the kernel.
The idea is that users may want to add or manipulate nodes w/changing the
u-boot binary. The
On Aug 5, 2008, at 11:50 AM, Kumar Gala wrote:
Added the 'fdtcmd' environment variable as a way to provide 'fdt'
commands
that the user can supply to manipulate the device tree after
ft_board_setup() and before the tree is handled to the kernel.
The idea is that users may want to add or
On Tue, 5 Aug 2008, Scott Wood wrote:
Also, remove the ctrl variable in favor of passing the constants
directly, and remove redundant (u8) casts.
Signed-off-by: Scott Wood [EMAIL PROTECTED]
---
This patch is untested, as I don't have the hardware.
Applied to u-boot-nand-flash/testing.
On Aug 5, 2008, at 7:03 AM, Andy Fleming wrote:
On Mon, Aug 4, 2008 at 9:02 PM, Becky Bruce
[EMAIL PROTECTED] wrote:
Currently, they use CONFIG_NUM_CPUS, which is different than
85xx for no good reason.
--- a/include/configs/MPC8610HPCD.h
+++ b/include/configs/MPC8610HPCD.h
@@ -17,7
In message [EMAIL PROTECTED] you wrote:
Added the 'fdtcmd' environment variable as a way to provide 'fdt' commands
that the user can supply to manipulate the device tree after
ft_board_setup() and before the tree is handled to the kernel.
The idea is that users may want to add or manipulate
Is this something we'd be willing to put into mainline?
- k
diff --git a/lib_ppc/bootm.c b/lib_ppc/bootm.c
index a872d31..3ebddd3 100644
--- a/lib_ppc/bootm.c
+++ b/lib_ppc/bootm.c
@@ -192,7 +192,8 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag, int argc, char
*argv[],
}
#ifdef
In message [EMAIL PROTECTED] you wrote:
Is this something we'd be willing to put into mainline?
Definitely not as is.
These are two unrelated changes, so please split this up into two
separate patches so we can decide independently.
Best regards,
Wolfgang Denk
--
DENX Software Engineering
I have enabled nfs booting on ppc based embedded board. I had placed my
kernel and rootfs in tftp directory, and had set the u-boot enivironment
varialbes as:
setenv bootfile /image/kernel
setenv root_path /tftpboot/image
The board was booting with this configuration...
Now I have
Kumar Gala wrote:
here's a rough start at an outline for the bootm script based on the
code (I've only outlined the Linux/PPC boot case its seems the most
complicated). One of the first things we clearly need is a imload
command. Thoughts on the various disable_{interrupts, usb,
On ADS5121 when booting linux the following errors are seen:
Unable to update property /[EMAIL PROTECTED]:bus-frequency,
err=FDT_ERR_NOTFOUND
Unable to update property /[EMAIL PROTECTED]/[EMAIL
PROTECTED]:local-mac-address, err=FDT_ERR_NOTFOUND
Unable to update property /[EMAIL
On Tue, Aug 5, 2008 at 7:24 PM, cjjoy1980 [EMAIL PROTECTED] wrote:
I have enabled nfs booting on ppc based embedded board. I had placed my
kernel and rootfs in tftp directory, and had set the u-boot enivironment
varialbes as:
setenv bootfile /image/kernel
setenv root_path
49 matches
Mail list logo