[Bug 1001861] New: Patch for TWR-K60FN120 board

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] New: Patch for TWR-K60FN120 board

bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

            Bug ID: 1001861
           Summary: Patch for TWR-K60FN120 board
           Product: eCos
           Version: unknown
            Target: freescale_twr_k60n512 (Freescale Kinetis TWR-K60N512)
  Architecture/Host Cortex-M
                OS:
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: low
         Component: Patches and contributions
          Assignee: [hidden email]
          Reporter: [hidden email]
                CC: [hidden email]

Created attachment 2251
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2251&action=edit
HAL patch

This patch was tested with: Ethernet/LwIP, UART5/Terminal, and I2C0. It is very
similar to the K70 HAL but the DDRAM was removed, and IO pins were changed to
match the board. Compilation with and without Floating Point and Cache was
tested. It is simple to tree diff this against the K70 to see there are few
chances of mistakes, but someone that knows K70 well may want to diff it to
check.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

--- Comment #1 from Mike Jones <[hidden email]> ---
Created attachment 2252
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2252&action=edit
The ecos.db patch matching the HAL patch

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Ilija Kocho <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
           Assignee|[hidden email] |[hidden email]
                   |ware.org                    |

--- Comment #2 from Ilija Kocho <[hidden email]> ---
Hi Mike.
Thank you for your contribution.

Here are some remarks:

src/twr_k60f120m_misc.c contains DDRAM controller set up parameters and
possibly some code that has no function on this platform.

misc/redboot_K60_ROM_FPU.ecm has comment "Startup: ROM with external RAM" (the
rest of the file seems OK).

SGML doc: Needs some clean-up of K70 remnants (such as Board supports K70),
Memory Layouts snd Startup Types. Also in order to include this chapter in eCos
Reference Manual you need to put a reference in the
<ecos_repository>/doc/sgml/doclist

Ilija

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Mike Jones <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2252|0                           |1
        is obsolete|                            |

--- Comment #3 from Mike Jones <[hidden email]> ---
Created attachment 2257
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2257&action=edit
The ecos.db path matching the HAL Patch Rev 2

I could not find any references in the doclist pertaining to kinetis other than
the variant. So I did not make any doc changes other than for the k60f120.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Mike Jones <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2251|0                           |1
        is obsolete|                            |

--- Comment #4 from Mike Jones <[hidden email]> ---
Created attachment 2273
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2273&action=edit
Core Patch

A cleaned up patch.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Mike Jones <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2257|0                           |1
        is obsolete|                            |

--- Comment #5 from Mike Jones <[hidden email]> ---
Created attachment 2274
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2274&action=edit
Ecos DB Patch

A cleaned up patch.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

--- Comment #6 from Ilija Kocho <[hidden email]> ---
Checked in.

Mike, please check out and test if it works on your target.

Thanks
Ilija

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

--- Comment #7 from Mike Jones <[hidden email]> ---
Verified. Works on my platform.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Ilija Kocho <[hidden email]> changed:

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

--- Comment #8 from Ilija Kocho <[hidden email]> ---
(In reply to comment #7)
> Verified. Works on my platform.

Then it's resolved.
Mike, thank you for your contribution.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

--- Comment #9 from John Dallaway <[hidden email]> ---
Created attachment 2313
  --> http://bugs.ecos.sourceware.org/attachment.cgi?id=2313&action=edit
twr_k60f12m-doc-130702.patch

Trivial patch to fix missing </para>. Checked-in.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug 1001861] Patch for TWR-K60FN120 board

bugzilla-daemon (Bugzilla)-9
In reply to this post by bugzilla-daemon
Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001861

Ilija Kocho [Илија Кочо] <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--
You are receiving this mail because:
You are on the CC list for the bug.