UTILITY: XmlToIdc generates IDC from ecu and logger defs

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

Looks like you need to install version 3.5 of the .Net Framework.

CSXRT4 · **Joined:** Fri Apr 17, 2009 5:19 am **Posts:** 42

Ok, I wondered if that had something to do with it but I wasn't sure what .net I had or what was needed.

cooljac · **Joined:** Wed Aug 04, 2010 1:21 am **Posts:** 22

follow your thread I finally start to explore my 10 EDM Legacy 2.5 GT

and have done the "easy part" to find the ssm base

and I found the cal id (AE5F301C---AT)is similar with ADM 10 Liberty GT (AE5F301D---MT)

using xmltoidc utility to make some idc but don't know how to use it!!have checked for the thread

loading the idc in the Ida and then use command ??

need some help, please!!

dschultz · **Posted:** Thu Sep 02, 2010 9:18 pm

A question about the utility when creating references to tables...
For a 2D table XmlToIdc creates a command set like this:

Code:

MakeNameEx(0xC11E4, "FrontOxygenSensorScaling", SN_CHECK);
MakeNameEx(0xC11B0, "FrontOxygenSensorScaling_YAxis", SN_CHECK);
referenceAddress = DfirstB(0xC11E4);
if (referenceAddress > 0)
{
    referenceAddress = referenceAddress - 8;
    MakeNameEx(referenceAddress, "TableFrontOxygenSensorScaling", SN_CHECK);
}
else
{
    Message("No reference to FrontOxygenSensorScaling\n");
}

But for a 3D table the script only contains this:

Code:

MakeNameEx(0xC9B34, "PrimaryOpenLoopFuelingB", SN_CHECK);
MakeNameEx(0xC9AB8, "PrimaryOpenLoopFuelingB_XAxis", SN_CHECK);
MakeNameEx(0xC9AE4, "PrimaryOpenLoopFuelingB_YAxis", SN_CHECK);

Why is there no referenceAddress commands associated with 3D tables moving back 12 bytes and Name the table definition?

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

Cooljac: See the last three lines of the first post in this thread.

DSchultz: That was just an oversight on my part. I'll fix it.

dschultz · **Posted:** Tue Sep 07, 2010 3:25 am

I think I found another quirk...
In the ecu_defs.xml a table may be defined with a trailing space character in its name, like so:

Code:

  <table name="Primary Open Loop Fueling A " storageaddress="0xC99DC">
   <table type="X Axis" storageaddress="0xC9960" />
   <table type="Y Axis" storageaddress="0xC998C" />
  </table>
  <table name="Primary Open Loop Fueling B " storageaddress="0xC9B34">
   <table type="X Axis" storageaddress="0xC9AB8" />
   <table type="Y Axis" storageaddress="0xC9AE4" />
  </table>

When XmlToIdc builds the script to mark tables it drops the space from the name.

Code:

MakeNameEx(0xC99DC, "PrimaryOpenLoopFuelingA", SN_CHECK);
MakeNameEx(0xC9960, "PrimaryOpenLoopFuelingA_XAxis", SN_CHECK);
MakeNameEx(0xC998C, "PrimaryOpenLoopFuelingA_YAxis", SN_CHECK);

MakeNameEx(0xC9B34, "PrimaryOpenLoopFuelingB", SN_CHECK);
MakeNameEx(0xC9AB8, "PrimaryOpenLoopFuelingB_XAxis", SN_CHECK);
MakeNameEx(0xC9AE4, "PrimaryOpenLoopFuelingB_YAxis", SN_CHECK);

Is there something that can be done to retain the space (change to an alternate character) to identify that this is a different table def than the ones defined with no trailing space character?

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

I didn't know whitespace mattered when I wrote this, but now that you mention it, I remember reading something about that in another thread here recently. I can change trailing whitespace to underscores to make them visible.

Also, the reason for the missing Table names is that the code was expecting the X and Y axes sub-tables to always appear as X, then Y, and the reality is the always appear in the other order. Or vice versa.

I think I fixed that last night but haven't tested the fix yet.

dschultz · **Posted:** Thu Sep 23, 2010 9:42 pm

Hi:
Another enhancement please...
For the extparam option, could you provide an option to prefix the IDA name with the ID value from the logger.xml file?
They would list nicely in the IDA names window and also easier to associate with the logger file when updating it or trying to map new parameters of new ROMs.
For example: TargetThrottlePlatePositionExt could be E57_TargetThrottlePlatePositionExt

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

First post updated with changes provided by dschultz. Thanks!

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

Updated again, see first post for details.

CJ4 · **Joined:** Tue Aug 24, 2010 2:39 pm **Posts:** 10

Great tool!
Can you put it into public source control (github, gitorious, ...) or is it already?
I think I had added a few lines in my private copy - inner loop to generate additional defs for 2+ byte params as I've got lots of them.

CSXRT4 · **Joined:** Fri Apr 17, 2009 5:19 am **Posts:** 42

Ok I finally got back to trying this out. I am using it on the Group N rom and I think there is a bug with the program.

CAL ID - A4TJ1X00
ECU ID - 3D04EA4605

I generated the .idc file for the tables but kept getting a syntax error about a ",". So I look in the tables.idc I created and found a bunch of doubles had been created and the second instance would not have an address. Like this

Code:

MakeNameEx(0x2E3ED, "P1518_STARTER_SWITCH_LOW_INPUT", SN_CHECK);
MakeNameEx(, "P1518_STARTER_SWITCH_LOW_INPUT_", SN_CHECK);

I removed all the instances and it worked after that

Also all the 3D tables had the message "No reference to ..." Is this normal???

Code:

No reference to Target_Boost_
No reference to Initial_Wastegate_Duty_
No reference to Max_Wastegate_Duty__
No reference to Turbo_Dynamics_Proportional
No reference to Turbo_Dynamics_Integral
No reference to Primary_Open_Loop_Fueling_Group_N_Conditions_Met
No reference to Primary_Open_Loop_Fueling_Group_N_Conditions_Not_Met
No reference to Minimum_Primary_Open_Loop_Enrichment_Throttle_
No reference to Front_Oxygen_Sensor_Scaling
No reference to Front_Oxygen_Sensor_Compensation_Atm_Pressure
No reference to CL_Fueling_Target_Compensation_Load
No reference to Throttle_Tipin_Enrichment
No reference to Speed_Density_Base_Load_Compensation_MAP_v_RPM
No reference to Speed_Density_Maximum_Load_Limit
No reference to MAF_Sensor_Scaling_Reference_Only
No reference to Base_Timing_A_
No reference to Base_Timing_B_
No reference to Base_Timing_Idle_Group_N_Conditions_Met
No reference to Base_Timing_Idle_Group_N_Conditions_Not_Met_Below_Speed_Threshold
No reference to Base_Timing_Idle_Group_N_Conditions_Not_Met_Above_Speed_Threshold
No reference to Knock_Correction_Advance_Max
No reference to Timing_Compensation_Per_Cylinder_A
No reference to Timing_Compensation_Per_Cylinder_B
No reference to Timing_Compensation_Per_Cylinder_C
No reference to Timing_Compensation_Per_Cylinder_D
No reference to Intake_Cam_Advance_Angle_AVCS_Group_N_Conditions_Met
No reference to Intake_Cam_Advance_Angle_AVCS_Group_N_Conditions_Not_Met
No reference to Idle_Speed_Target_A
No reference to Idle_Speed_Target_B

I attached the .idc file in a zip file

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

I've only tested it with 32-bit ROM images so far, and I suspect there are some differences with 16-bit ROMs like the Group-N.

nsfw · **Joined:** Thu Nov 23, 2006 2:23 am **Posts:** 2565

I just opened a 32-bit ROM and got a bunch of similar "No reference to..." messages. However all of the references it was supposed to create, actually did get created. Not sure what's up with that.

I might have run the script twice, so try that if you haven't already.

Easty · **Posted:** Sun Jul 22, 2012 2:11 am

Hi, I generated my idc file but when I try and execute it says I've got a syntax error near static?

Here is the first few lines of my idc file;

///////////////////////////////////////////////////////////////////////////////
// Table definitions for AZ1G400W
///////////////////////////////////////////////////////////////////////////////
#include <idc.idc>
static Tables_AZ1G400W()
{
auto referenceAddress;

MakeNameEx(43AC4, "Fuel_Pump_Duty", SN_CHECK);

MakeNameEx(0xC0ED4, "Target_Boost_", SN_CHECK);
MakeNameEx(0xC0E6C, "Target_Boost__X_Axis", SN_CHECK);
MakeNameEx(0xC0E98, "Target_Boost__Y_Axis", SN_CHECK);
referenceAddress = DfirstB(0xC0ED4);
if (referenceAddress > 0)
{
referenceAddress = referenceAddress - 12;
MakeNameEx(referenceAddress, "Table_Target_Boost_", SN_CHECK);
}
else
{

can some one please help?

RomRaider

UTILITY: XmlToIdc generates IDC from ecu and logger defs

Who is online