Re: [U-Boot] U-Boot Digest, Vol 9, Issue 49

how can i download a bin file into nand flash in windows xp?
在2009-02-05,u-boot-request@lists.denx.de 写道:
Send U-Boot mailing list submissions to u-boot@lists.denx.de
To subscribe or unsubscribe via the World Wide Web, visit http://lists.denx.de/mailman/listinfo/u-boot or, via email, send a message with subject or body 'help' to u-boot-request@lists.denx.de
You can reach the person managing the list at u-boot-owner@lists.denx.de
When replying, please edit your Subject line so it is more specific than "Re: Contents of U-Boot digest..."
Today's Topics:
1. Re: [PATCH 3/9] i.MX31: support GPIO as a chip-select in the mxc_spi driver (Guennadi Liakhovetski) 2. Re: [PATCH 4/9] A driver for the S6E63D6 SPI display controller from Samsung (Guennadi Liakhovetski) 3. Re: [PATCH 1/2] flash/cfi_flash: Use virtual sector start address, not phys (Stefan Roese) 4. [PATCH] cfi_flash: Silence compilation warning (Stefan Roese) 5. Re: [PATCH 4/9] A driver for the S6E63D6 SPI display controller from Samsung (Guennadi Liakhovetski) 6. Re: [PATCH 1/2] flash/cfi_flash: Use virtual sector start address, not phys (Stefan Roese) 7. [PATCH] cfi_flash: Fix typo in cfi_flash.c (Stefan Roese) 8. [cfi-flash] Please pull git://www.denx.de/git/u-boot-cfi-flash.git (Stefan Roese) 9. Re: [PATCH 9/9] ARM: add an "eet" variant of the imx31_phycore board (Guennadi Liakhovetski)
----------------------------------------------------------------------
Message: 1 Date: Thu, 5 Feb 2009 11:03:53 +0100 (CET) From: Guennadi Liakhovetski lg@denx.de Subject: Re: [U-Boot] [PATCH 3/9] i.MX31: support GPIO as a chip-select in the mxc_spi driver To: Jean-Christophe PLAGNIOL-VILLARD plagnioj@jcrosoft.com Cc: U-Boot@lists.denx.de Message-ID: Pine.LNX.4.64.0902051102120.5553@axis700.grange Content-Type: TEXT/PLAIN; charset=US-ASCII
On Wed, 4 Feb 2009, Jean-Christophe PLAGNIOL-VILLARD wrote:
On 17:59 Wed 04 Feb , Guennadi Liakhovetski wrote:
- /*
* Some SPI devices require active chip-select over multiple
* transactions, we achieve this using a GPIO. Still, the SPI
* controller has to be configured to use one of its own chipselects.
* To use this feature you have to call spi_setup_slave() with
* cs = internal_cs | (gpio << 8), and you have to use some unused
* on this SPI controller cs between 0 and 3.
*/
- if (cs > 3) {
mxcs->gpio = cs >> 8;
cs &= 3;
mx31_gpio_direction(mxcs->gpio, MX31_GPIO_DIRECTION_OUT);
- } else
mxcs->gpio = -1;
why not add a callback for the chipselect instead as example if you have to use a gpio extender it will simplest to implent instead of hack the SPI driver
and how shall I pass this callback to
struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs, unsigned int max_hz, unsigned int mode)
?
Thanks Guennadi --- Guennadi Liakhovetski, Ph.D.
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@denx.de
------------------------------
Message: 2 Date: Thu, 5 Feb 2009 11:19:38 +0100 (CET) From: Guennadi Liakhovetski lg@denx.de Subject: Re: [U-Boot] [PATCH 4/9] A driver for the S6E63D6 SPI display controller from Samsung To: Anatolij Gustschin agust@denx.de Cc: U-Boot@lists.denx.de Message-ID: Pine.LNX.4.64.0902051117020.5553@axis700.grange Content-Type: TEXT/PLAIN; charset=US-ASCII
On Wed, 4 Feb 2009, Anatolij Gustschin wrote:
Hi Guennadi,
+struct s6e63d6 {
- unsigned int bus;
- unsigned int cs;
- struct spi_slave *slave;
+};
could we use "bus" and "cs" from struct spi_slave? "struct spi_slave" declares them already. Maybe we should drop struct s6e63d6 entirely and use struct spi_slave instead?
No, cs and bus here are input parameters, set up by the caller to s6e63d6_init() and used by the latter to setup the SPI-slave. slave is driver-"private" field, the caller doesn't need to know what is under s6e63d6.
Thanks Guennadi --- Guennadi Liakhovetski, Ph.D.
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@denx.de
------------------------------
Message: 3 Date: Thu, 5 Feb 2009 11:25:16 +0100 From: Stefan Roese sr@denx.de Subject: Re: [U-Boot] [PATCH 1/2] flash/cfi_flash: Use virtual sector start address, not phys To: u-boot@lists.denx.de Message-ID: 200902051125.16237.sr@denx.de Content-Type: text/plain; charset="iso-8859-1"
On Monday 02 February 2009, Becky Bruce wrote:
include/flash.h was commented to say that the address in flash_info->start was a physical address. However, from u-boot's point of view, and looking at most flash code, it makes more sense for this to be a virtual address. So I corrected the comment to indicate that this was a virtual address.
The only flash driver that was actually treating the address as physical was the mtd/cfi_flash driver. However, this code was using it inconsistently as it actually directly dereferenced the "start" element, while it used map_physmem to get a virtual address in other places. I changed this driver so that the code which initializes the info->start field calls map_physmem to get a virtual address, eliminating the need for further map_physmem calls. The code is now consistent.
The *only* place a physical address should be used is when defining the flash banks list that is used to initialize the flash_info struct, usually found in the board config file.
Applied to cfi-flash/master. 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@denx.de =====================================================================
------------------------------
Message: 4 Date: Thu, 5 Feb 2009 11:26:42 +0100 From: Stefan Roese sr@denx.de Subject: [U-Boot] [PATCH] cfi_flash: Silence compilation warning To: u-boot@lists.denx.de Message-ID: 1233829602-10031-1-git-send-email-sr@denx.de
Patch "flash/cfi_flash: Use virtual sector start address, not phys" introduced a small compilation warning. This patch fixes it.
Signed-off-by: Stefan Roese sr@denx.de --- drivers/mtd/cfi_flash.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c index 4cb5fb5..3a8f8e4 100644 --- a/drivers/mtd/cfi_flash.c +++ b/drivers/mtd/cfi_flash.c @@ -879,7 +879,7 @@ static int flash_write_cfibuffer (flash_info_t * info, ulong dest, uchar * cp, int cnt; int retcode; void *src = cp; - void *dst = dest; + void *dst = (void *)dest; void *dst2 = dst; int flag = 0; uint offset = 0;

Dear Null,
In message 17101379.329971233837292914.JavaMail.coremail@app150.163.com you wrote:
[500 Lines full quote deleted]
You send a 500 Lines full quote,
without a real name,
without a resonable subject,
highjacking an existing mail thread instead of creationg a new one,
posting HTML even though this is strictly forbidden on this list,
and you send to the list-request address,
... and you still expect a resonable answer?
Please start reading http://www.netmeister.org/news/learn2quote.html and http://catb.org/esr/faqs/smart-questions.html and http://www.xs4all.nl/~hanb/documents/quotingguide.html
Best regards,
Wolfgang Denk
participants (2)
-
null
-
Wolfgang Denk