Bug 410545 - Wizard-generated platforms used with 3.25 do not work correctly with c64p.Cache in SYS/BIOS
Summary: Wizard-generated platforms used with 3.25 do not work correctly with c64p.Cac...
Status: CLOSED FIXED
Alias: None
Product: RTSC (ARCHIVED)
Classification: Technology
Component: Platforms (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Sasha Slijepcevic CLA
QA Contact:
URL:
Whiteboard: target:3.25.02
Keywords: readme
Depends on:
Blocks:
 
Reported: 2013-06-11 18:39 EDT by Sasha Slijepcevic CLA
Modified: 2013-10-08 17:35 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sasha Slijepcevic CLA 2013-06-11 18:39:10 EDT 
c64p.Cache calls platform.getCreateArgs() to fetch l1PMode, l1DMode and l2Mode from the platforms generated by the Platform Wizard. In 3.25, platforms do not use that function anymore because the original purpose of that function was to return $args, and we removed $args in 3.25. Now, getCreateArgs() always returns null.
Here are the forum threads related to this problem:
http://e2e.ti.com/support/embedded/bios/f/355/t/269783.aspx
http://e2e.ti.com/support/embedded/bios/f/355/t/263553.aspx
Comment 1 Sasha Slijepcevic CLA 2013-06-11 18:40:38 EDT 
Here is a related bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=396789
Comment 2 Sasha Slijepcevic CLA 2013-06-27 12:17:36 EDT 
- fixed in xdctools-f30
I replicated mode settings from the underlying generic platform instance to the main platform generated by the wizard. Now, when BIOS checks if the main platform has cache settings available it will find them and won't need to call getCreateArgs.
The change in getCreateArgs is not needed anymore for this use case, but I am leaving it anyway for possible other callers to getCreateArgs.
Comment 3 judah CLA 2013-07-11 12:50:37 EDT 
=======verifypass===========
This bug is fixed verified with xdcprod-r69.
Created a new platform and built a project with it to make sure cache settings from platform was used.
Comment 4 Sasha Slijepcevic CLA 2013-10-08 17:35:27 EDT 
shipped in 3.25.02.70.