summaryrefslogtreecommitdiff
path: root/src/flash
Commit message (Collapse)AuthorAgeFilesLines
* nit: do not add \n at end of LOG_ERROREric Wetzel2011-01-057-14/+14
| | | | Fixed in many other places, and submitted in response to Øyvind's invitation.
* nit: do not add \n at end of LOG_ERRORØyvind Harboe2011-01-031-4/+4
| | | | | This is done in many other places as well, patch to fix gratefully accepted!
* error handling: the error number is not part of the user interfaceØyvind Harboe2011-01-035-17/+13
| | | | | | | | Do not propagate error number to user. This is for internal programming purposes only. Error messages to the user is reported as text via LOG_ERROR(). Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cfi_protect is not implemented on Spansion flashes (many do not even have ↵Michael Schwingen2011-01-021-2/+2
| | | | | | protection bits). Demote from error to warning, so that common board code can use "flash write_image erase unlock" regardless of the flash type. Signed-off-by: Michael Schwingen <michael@schwingen.org>
* non-CFI flash code uses data from CFI structures. Make sure that timeouts ↵Michael Schwingen2011-01-022-99/+98
| | | | | | are filled in on non-CFI flashes, and print CFI information in all cases, nut just on CFI flashes. Signed-off-by: Michael Schwingen <michael@schwingen.org>
* NAND/S3CXXXX: remove private "target" copyAntonio Borneo2011-01-027-33/+16
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/ORION: remove private "target" copyAntonio Borneo2011-01-021-13/+5
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/NUC910: remove private "target" copyAntonio Borneo2011-01-021-21/+7
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/MX3: remove private "target" copyAntonio Borneo2011-01-022-22/+11
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/MX2: remove private "target" copyAntonio Borneo2011-01-022-20/+11
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/LPC3180: remove private "target" copyAntonio Borneo2011-01-022-24/+16
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/DAVINCI: remove private "target" copyAntonio Borneo2011-01-021-27/+17
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/AT91SAM9: remove private "target" copyAntonio Borneo2011-01-021-37/+22
| | | | | | | Remove "target" form private data, and use common one in struct nand_block. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/TCL: prepare for common "target" referenceAntonio Borneo2011-01-022-3/+26
| | | | | | | | | Every NAND driver keeps private copy of "target" structure. Prepare infostructure to move private "target" copy in common/shared struct nand_device. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/TCL: fix segfault on syntax errorAntonio Borneo2011-01-021-1/+1
| | | | | | | Fix segfault if incomplete command "nand device dummy" is executed. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/CORE: Comment use of alive_sleep()Antonio Borneo2011-01-021-0/+1
| | | | Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NAND/CORE: Replace decimal dot in messagesAntonio Borneo2011-01-021-66/+66
| | | | | | | | | | | | Table of NAND devices reports operating voltage. Replace comma with proper decimal dot. Øyvind: "." is correct for UK/US, but incorrect for many other languages. OpenOCD is not localized at this point, so sticking to "." for US/UK should be OK. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* flash/nand: review NAND driver interfaceAntonio Borneo2011-01-0213-47/+2
| | | | | | | | | | From struct nand_flash_controller : - remove unused field register_commands; - remove field controller_ready, exported but never referenced. Remove dead code pointed by controller_ready. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* Compilation Warnings on OS X 10.5Andrew MacIsaac2010-12-292-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I received a number of "-Wshadow" related warnings (treated as errors) while trying to build on OS X Leopard. In addition, there were two miscellaneous other warnings in the flash drivers. Attached are two patches which correct these issues and the commit messages to accompany them. My system has the following configuration (taken from uname -a): Darwin 9.8.0 Darwin Kernel Version 9.8.0: Wed Jul 15 16:55:01 PDT 2009; root:xnu-1228.15.4~1/RELEASE_I386 i386 === Werror_patch.txt Commit Message === compilation: fixes for -Wshadow warnings on OS X These changes fix -Wshadow compilation warnings on OS X 10.5.8 Compiled with the following configure command: ../configure --prefix=/usr/local --enable-maintainer-mode --enable-jlink --enable-ft2232_libftdi === flash_patch.txt Commit Message === compilation: fixes for flash driver warnings on OS X These changes fix two compilation warnings on OS X 10.5.8: ../../../../src/flash/nor/at91sam3.c:2767: warning: redundant redeclaration of 'at91sam3_flash' ../../../../src/flash/nor/at91sam3.c:101: warning: previous declaration of 'at91sam3_flash' was here and ../../../../src/flash/nor/stmsmi.c:205: warning: format not a string literal and no format arguments Compiled with the following configure command: ../configure --prefix=/usr/local --enable-maintainer-mode --enable-jlink --enable-ft2232_libftdi === Andrew
* warning: fix warning where GCC didn't catch a doubly declared global structureØyvind Harboe2010-12-291-1/+1
| | | | | | Why hasn't earlier versions of GCC caught this? Bug? Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* warnings: use more 'const' for char *Øyvind Harboe2010-12-2910-14/+14
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* stm32x : always use valid flash base addressSpencer Oliver2010-12-231-0/+3
| | | | | | | This change matches the previous behaviour of using the known flash address. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* flash: print flash bank name on flash info cmdSpencer Oliver2010-12-231-2/+2
| | | | | | | | The flash bank name is a required element in adding flash banks, however other than looking at the config file there is no way of getting the name used in openocd. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* stm32: fix reversed option byte writeSpencer Oliver2010-12-231-4/+4
| | | | | | This has been a long standing bug that has never been spotted. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* stm32: update option bytes for stm32xl familySpencer Oliver2010-12-231-8/+79
| | | | | | | add supoort for xl family boot bank option. The option byte handling will be cleaned up in a later patch. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* stm32: add stm32 xl family flash supportSpencer Oliver2010-12-231-1/+45
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* stm32: add dual flash bank supportSpencer Oliver2010-12-231-26/+37
| | | | | | | This patch adds the initial dual flash bank support for devices such as the stm32xl family. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* Fix for segfault in handle_nand_dump_command.Paul Richards2010-12-151-5/+5
|
* stm32: fix unprotectØyvind Harboe2010-12-151-4/+11
| | | | | | | | | the unprotect fn in stm32 needs to unprotect more sectors than was requested aligned to some boundary. Print warning when this happens. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* flash: clarify protect documentationØyvind Harboe2010-12-151-3/+6
| | | | | | | some chips unprotect more than the range asked for. The protect fn, must unprotect/protect minimally the range given. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* contrib: add source to the cfi flash loadersSpencer Oliver2010-12-101-6/+13
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: disable buffer writes for M29W128GSpencer Oliver2010-12-102-5/+16
| | | | | | | | | | For some reason buffer writes for the M29W128G do not work reliably, so disable them. See: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=504a3e72208fc6a65924426ff5693982590bccdc Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: allow optional buffer write supportSpencer Oliver2010-12-102-2/+12
| | | | | | | Some flash's do not support buffer writes, so we now check they are supported before trying to use them. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: calculate correct timeoutsSpencer Oliver2010-12-102-16/+29
| | | | | | | | The existing code used incorrect timeout values for the various cfi operations. We now calculate the timeouts and convert to msecs if necessary. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: prefix string hex outputSpencer Oliver2010-12-101-12/+12
| | | | | | Add hex prefix so we know output is not decimal. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: add time format to cfi query outputSpencer Oliver2010-12-101-8/+11
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cfi: whitespace and long line cleanupSpencer Oliver2010-12-101-282/+389
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* Fix sector layout for 504-KiB LPC2000 devicesRolf Meeser2010-12-041-4/+7
|
* lpc2900.c: Improve debug output for device probingRolf Meeser2010-12-011-1/+5
|
* lpc2900.c: Add support for new device LPC2926Rolf Meeser2010-12-011-1/+5
|
* LPC2900: Avoid false positive reprobing if first probing failsRolf Meeser2010-12-011-7/+11
|
* FLASH/NOR: Rename spearsmi.c to stmsmi.cAntonio Borneo2010-11-232-1/+1
| | | | Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* FLASH/NOR: rename from spearsmi to stmsmiAntonio Borneo2010-11-232-81/+81
| | | | | | | STMicroelectronics controller SMI is not SPEAr specific. Rename it and change name to every symbol in the code. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* STR750: Add SMI interface supportAntonio Borneo2010-11-231-35/+43
| | | | | | | Modified spearsmi driver to include support for STR75x Added missing initialization in tcl file for STR750 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* NOR/SPEARSMI: fix segfaultAntonio Borneo2010-11-231-0/+1
| | | | | | | If flash chip is not listed in the table, or if no flash is connected, pointer must be properly initialized. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* flash: iterating over an address range now handles multiple banksØyvind Harboe2010-11-221-1/+38
| | | | | | | e.g. flash erase_address now works across an address range that spans multiple flash chips. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* build: add common.mkSpencer Oliver2010-11-183-10/+4
| | | | | | | Rather than specifying common makefile variables move them all to a common.mk. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* FLASH/NOR: Remove useless file tms470.hAntonio Borneo2010-11-173-40/+15
| | | | Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* FLASH/NOR: Remove useless file str9xpec.hAntonio Borneo2010-11-173-79/+50
| | | | Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* FLASH/NOR: Remove useless file str9x.hAntonio Borneo2010-11-173-62/+35
| | | | Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>