[Bug 1001449] New: AT25DFxxx SPI flash test application

[Bug 1001449] New: AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

           Summary: AT25DFxxx SPI flash test application
           Product: eCos
           Version: CVS
          Platform: All
        OS/Version: Other
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: low
         Component: Patches and contributions
        AssignedTo: unassigned@bugs.ecos.sourceware.org
        ReportedBy: ecos@hotmail.co.uk
                CC: ecos-patches@ecos.sourceware.org
             Class: Advice Request


Patch split from http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001291

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #2 from Christophe Coutand <ecos@hotmail.co.uk> 2012-01-08 08:38:38 GMT ---
Created an attachment (id=1493)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1493)
platform SPI flash example

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #1 from Christophe Coutand <ecos@hotmail.co.uk> 2012-01-08 08:37:48 GMT ---
Created an attachment (id=1492)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1492)
AT25DFxxx SPI flash

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #3 from Christophe Coutand <ecos@hotmail.co.uk> 2012-01-08 08:46:38 GMT ---
Hi Ilija,

I replaced the CDL with CYGNUM_DEVS_FLASH_SPI_AT25XXX_MAP_ADDR but maybe the
name shall still specify device 0. The platform could have several SPI flash.

In the attached solution CYGNUM_DEVS_FLASH_SPI_AT25XXX_MAP_ADDR is defined at
the platform level. The SPI flash test application accesses the definition by
including the platform header file: 

#include CYGDAT_DEVS_FLASH_SPI_AT25DFXXX_PLF_CDL

I agree, more could be defined in the platform CDL such as the speed. Bus and
chip select are usually fix.

Christophe

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ilijak@siva.com.mk

--- Comment #4 from Ilija Kocho <ilijak@siva.com.mk> 2012-01-09 20:38:52 GMT ---
(In reply to comment #3)
> Hi Ilija,
> 
> I replaced the CDL with CYGNUM_DEVS_FLASH_SPI_AT25XXX_MAP_ADDR but maybe the
> name shall still specify device 0. The platform could have several SPI flash.
> 

OK. Maybe with little abbreviation DEV0 instead of DEVICE0.


> In the attached solution CYGNUM_DEVS_FLASH_SPI_AT25XXX_MAP_ADDR is defined at
> the platform level. The SPI flash test application accesses the definition by
> including the platform header file: 
> 
> #include CYGDAT_DEVS_FLASH_SPI_AT25DFXXX_PLF_CDL
> 
> I agree, more could be defined in the platform CDL such as the speed. Bus and
> chip select are usually fix.

They are usually hardwired, but we could consider the CDL as variant wide. For
illustration I would direct you to Attachment #1503 and Attachment #1502.


> 
> Christophe

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

Christophe Coutand <ecos@hotmail.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #1492|0                           |1
        is obsolete|                            |
   Attachment #1493|0                           |1
        is obsolete|                            |

--- Comment #5 from Christophe Coutand <ecos@hotmail.co.uk> 2012-02-26 11:24:19 GMT ---
Created an attachment (id=1594)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1594)
Updated patch

Hi Ilija,

I have updated the patch according to your comments.

Regards,
Christophe

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #6 from Ilija Kocho <ilijak@siva.com.mk> 2012-02-26 20:23:19 GMT ---
(In reply to comment #5)
> Created an attachment (id=1594)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1594) [details]
> Updated patch
> 
> Hi Ilija,
> 
> I have updated the patch according to your comments.
> 
> Regards,
> Christophe

Thank you Christophe.
I have downloaded the patches, but following week I am at Embedded World.
Probably a week after.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #7 from Ilija Kocho <ilijak@siva.com.mk> 2012-03-12 21:57:55 GMT ---
Now checked in.

Christophe,
Thank you for contribution. Please update from CVS and check on your target.
Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

--- Comment #8 from Christophe Coutand <ecos@hotmail.co.uk> 2012-03-13 19:18:08 GMT ---
Thank you Ilija, re-tested OK. 
Christophe

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001449] AT25DFxxx SPI flash test application

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001449

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |NEXTRELEASE

--- Comment #9 from Ilija Kocho <ilijak@siva.com.mk> 2012-03-13 21:05:19 GMT ---
Checked in and tested.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.