Merged.
It would be nice with some followup patches:
- some comments on how this code can/should be refactored rather
than copy pasted as new devices are added
- a patch to get rid of the non-sensical .h files in drivers/nor/*.h and
just put the private data into the .c file.
--
Øyvind
Antonio Borneo wrote:
Are you aware of any target, other than SPEAr, that provides memory
mapping feature of SPI flash? I quickly searched, but failed.
I haven't seen it on any ARM chips, but OTOH I'm somewhat new to ARM.
On PC mainboards it is quite common, I'd even say the norm.
Are you aware of any target, other than SPEAr, that provides memory
mapping feature of SPI flash? I quickly searched, but failed.
ST STR750 i believe - it is also called SMI interface.
Cheers
Spen
___
Openocd-development mailing list
On 16/11/2010 21:14, Igor Skochinsky wrote:
Hello Antonio,
Tuesday, November 16, 2010, 3:53:53 PM, you wrote:
AB Are you aware of any target, other than SPEAr, that provides memory
AB mapping feature of SPI flash? I quickly searched, but failed.
NXP's LPC1800 (and, I think, LPC4000) have
The patch in attachment adds additional comments to previous commit.
Best Regards,
Antonio Borneo
0002-NOR-SPEARSMI-Add-comments-about-SPI.patch
Description: Binary data
___
Openocd-development mailing list
Openocd-development@lists.berlios.de
On Tue, Nov 16, 2010 at 4:18 PM, Øyvind Harboe oyvind.har...@zylin.com wrote:
...
It would be nice with some followup patches:
...
- a patch to get rid of the non-sensical .h files in drivers/nor/*.h and
just put the private data into the .c file.
Hi,
in attachment 3 patches to start
On Tue, Nov 16, 2010 at 11:23 PM, Spencer Oliver s...@spen-soft.co.uk wrote:
Are you aware of any target, other than SPEAr, that provides memory
mapping feature of SPI flash? I quickly searched, but failed.
ST STR750 i believe - it is also called SMI interface.
Yes, seems they share same
Merged.
Thanks!
--
Øyvind Harboe
Can Zylin Consulting help on your project?
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
___
Merged.
Thanks!
(Please post followup patches in new thread)
--
Øyvind Harboe
Can Zylin Consulting help on your project?
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
On Sat, Nov 13, 2010 at 1:09 PM, Peter Stuge pe...@stuge.se wrote:
Øyvind Harboe wrote:
So basically if OpenOCD had some generic support for
interfacing to SPI chips via the target CPU over JTAG,
then you could build on that for your flash driver?
That seems a completely separate feature to
Øyvind Harboe wrote:
there are parts of the patch which should be separated out,
because they are not really specific to spearsmi.
..
Refactor now, or then?
..
I haven't seen a proposed design and it seems some time off.
Fair enough. Could we add some commentary to remind future code
On Mon, Nov 15, 2010 at 4:07 PM, Peter Stuge pe...@stuge.se wrote:
Øyvind Harboe wrote:
there are parts of the patch which should be separated out,
because they are not really specific to spearsmi.
..
Refactor now, or then?
..
I haven't seen a proposed design and it seems some time off.
Antonio Borneo wrote:
I'm not thrilled about having this information local in the
spearsmi
Like generic SPI and SPI-flash layers. The flash
support shouldn't be SPEAr-specific in the least.
Understand your comments.
I agree that the table could be shared between SPEAr SMI and a
Øyvind Harboe wrote:
So basically if OpenOCD had some generic support for
interfacing to SPI chips via the target CPU over JTAG,
then you could build on that for your flash driver?
That seems a completely separate feature to your patch.
That's the point basically, there are parts of the
Any objections out there?
Looks OK to me on the surface of it. You're the expert here.
I'll let it cool off and if no feedback, I'll merge it.
The driver structure follows the prototype of the other drivers in
src/flash/nor/
The file .h that I'm adding is totally useless, since only provides
--- On Fri, 11/12/10, Peter Stuge pe...@stuge.se wrote:
I'm not thrilled about having this information local in the
spearsmi
Like generic SPI and SPI-flash layers. The flash
support shouldn't be SPEAr-specific in the least.
___
On Sat, Nov 13, 2010 at 1:27 AM, David Brownell davi...@pacbell.net wrote:
--- On Fri, 11/12/10, Peter Stuge pe...@stuge.se wrote:
I'm not thrilled about having this information local in the
spearsmi
Like generic SPI and SPI-flash layers. The flash
support shouldn't be SPEAr-specific in
Hi,
in attachment the driver for the serial NOR flash controller of ST
SPEAr devices.
The first patch adds driver support and documentation, the second one
adds the proper lines in the (current) only board file of this device
family.
The driver structure follows the prototype of the other drivers
18 matches
Mail list logo