Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316402 - ti.catalog.c6000.OMAP4430 .cpuCore is too vague ("OMAP4430")
Summary: ti.catalog.c6000.OMAP4430 .cpuCore is too vague ("OMAP4430")
Status: CLOSED FIXED
Alias: None
Product: RTSC
Classification: Technology
Component: Platforms (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Sasha Slijepcevic CLA
QA Contact:
URL:
Whiteboard: target:3.20.01
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-09 21:07 EDT by Chris Ring CLA
Modified: 2011-06-20 14:43 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Ring CLA 2010-06-09 21:07:58 EDT
ti.catalog.c6000.OMAP4430's .cpuCore is set to "OMAP4430".  The documentation states that this field "uniquely identifies the instruction set that the CPU can decode and execute."

As an example, a common .cpuCore is "64x+".  I'd suggest changing this to "64T".

Config scripts that set memory types based on this .cpuCore are strange:

if (platform.match(/sdp4430/)) {
    if (Program.cpu.attrs.cpuCore == "OMAP4430") {
        /* strangely(!), C64T */
        internalHeapName = "L2SRAM";
    } else if (Program.cpu.attrs.cpuCore == "CM3") {
        /* M3 */
        internalHeapName = "L2_RAM";
    }
}

I've set this to major only b/c it would be good to fix this sooner than later so we have less of a migration issue.

Chris
Comment 1 Chris Ring CLA 2010-06-21 16:10:13 EDT
There's a similar issue with the ti.catalog.c6000.TMS320DM8168 platform's .cpuCore.  It's set to 'TMS320DM8168', but should probably be something like 'C674'.
Comment 2 Sasha Slijepcevic CLA 2010-06-22 13:21:04 EDT
- fixed in platform-l16

For OMAP4430, cpuCore is set to "64T". For DM730, DM740 and DM8168, cpuCore is set to "674".
Comment 3 Amit Mookerjee CLA 2010-07-15 18:11:24 EDT
Confirmed by inspection in xdcprod-m51
Comment 4 Dave Russo CLA 2011-06-20 14:43:43 EDT
Shipped in XDCtools 3.20.01