Hello,
I am using the TMS320F28335 and I am trying to compute my application's CRC using a modified version (see code below) of crc_gen() function given in spru513g.pdf (appendix C). My modified version only works with CRC-32-PRIME: 4beab53b.
I created the crc table (shown below) using visual studio.
The modified crc_gen() function is not computing the same CRC as generated by the linker (Compiler version 6.2.5). My linker cmd file can be found here:
/* // TI File $Revision: /main/10 $ // Checkin $Date: July 9, 2008 13:43:56 $ //########################################################################### // // FILE: F28335.cmd // // TITLE: Linker Command File For F28335 Device // //########################################################################### // $TI Release:$ // $Release Date:$ //########################################################################### */ /* ====================================================== // For Code Composer Studio V2.2 and later // --------------------------------------- // In addition to this memory linker command file, // add the header linker command file directly to the project. // The header linker command file is required to link the // peripheral structures to the proper locations within // the memory map. // // The header linker files are found in <base>\DSP2833x_Headers\cmd // // For BIOS applications add: DSP2833x_Headers_BIOS.cmd // For nonBIOS applications add: DSP2833x_Headers_nonBIOS.cmd ========================================================= */ /* ====================================================== // For Code Composer Studio prior to V2.2 // -------------------------------------- // 1) Use one of the following -l statements to include the // header linker command file in the project. The header linker // file is required to link the peripheral structures to the proper // locations within the memory map */ /* Uncomment this line to include file only for non-BIOS applications */ /* -l DSP2833x_Headers_nonBIOS.cmd */ /* Uncomment this line to include file only for BIOS applications */ /* -l DSP2833x_Headers_BIOS.cmd */ /* 2) In your project add the path to <base>\DSP2833x_headers\cmd to the library search path under project->build options, linker tab, library search path (-i). /*========================================================= */ /* Define the memory block start/length for the F28335 PAGE 0 will be used to organize program sections PAGE 1 will be used to organize data sections Notes: Memory blocks on F28335 are uniform (ie same physical memory) in both PAGE 0 and PAGE 1. That is the same memory region should not be defined for both PAGE 0 and PAGE 1. Doing so will result in corruption of program and/or data. L0/L1/L2 and L3 memory blocks are mirrored - that is they can be accessed in high memory or low memory. For simplicity only one instance is used in this linker file. Contiguous SARAM memory blocks can be combined if required to create a larger memory block. */ MEMORY { PAGE 0: /* Program Memory */ /* Memory (RAM/FLASH/OTP) blocks can be moved to PAGE1 for data allocation */ ZONE0 : origin = 0x004000, length = 0x001000 /* XINTF zone 0 */ RAML0L1L2L3L4L5L6L7 : origin = 0x008000, length = 0x008000 // RAML0 : origin = 0x008000, length = 0x001000 /* on-chip RAM block L0 */ // RAML1 : origin = 0x009000, length = 0x001000 /* on-chip RAM block L1 */ // RAML2 : origin = 0x00A000, length = 0x001000 /* on-chip RAM block L2 */ // RAML3 : origin = 0x00B000, length = 0x001000 /* on-chip RAM block L3 */ ZONE6 : origin = 0x0100000, length = 0x100000 /* XINTF zone 6 */ ZONE7A : origin = 0x0200000, length = 0x00FC00 /* XINTF zone 7 - program space */ FLASHH : origin = 0x300000, length = 0x008000 /* on-chip FLASH */ FLASHG : origin = 0x308000, length = 0x008000 /* on-chip FLASH */ FLASHF : origin = 0x310000, length = 0x008000 /* on-chip FLASH */ FLASHE : origin = 0x318000, length = 0x008000 /* on-chip FLASH */ //FLASHD : origin = 0x320000, length = 0x008000 /* on-chip FLASH */ FLASHDC : origin = 0x320000, length = 0x010000 /* on-chip FLASH */ //FLASHC : origin = 0x328000, length = 0x008000 /* on-chip FLASH */ FLASHA : origin = 0x338000, length = 0x007F80 /* on-chip FLASH */ CSM_RSVD : origin = 0x33FF80, length = 0x000076 /* Part of FLASHA. Program with all 0x0000 when CSM is in use. */ BEGIN : origin = 0x33FFF6, length = 0x000002 /* Part of FLASHA. Used for "boot to Flash" bootloader mode. */ CSM_PWL : origin = 0x33FFF8, length = 0x000008 /* Part of FLASHA. CSM password locations in FLASHA */ OTP : origin = 0x380400, length = 0x000400 /* on-chip OTP */ ADC_CAL : origin = 0x380080, length = 0x000009 /* ADC_cal function in Reserved memory */ IQTABLES : origin = 0x3FE000, length = 0x000b50 /* IQ Math Tables in Boot ROM */ IQTABLES2 : origin = 0x3FEB50, length = 0x00008c /* IQ Math Tables in Boot ROM */ FPUTABLES : origin = 0x3FEBDC, length = 0x0006A0 /* FPU Tables in Boot ROM */ ROM : origin = 0x3FF27C, length = 0x000D44 /* Boot ROM */ RESET : origin = 0x3FFFC0, length = 0x000002 /* part of boot ROM */ VECTORS : origin = 0x3FFFC2, length = 0x00003E /* part of boot ROM */ PAGE 1 : /* Data Memory */ /* Memory (RAM/FLASH/OTP) blocks can be moved to PAGE0 for program allocation */ /* Registers remain on PAGE1 */ BOOT_RSVD : origin = 0x000000, length = 0x000050 /* Part of M0, BOOT rom will use this for stack */ RAMM01 : origin = 0x000050, length = 0x0007B0 //RAMM0 : origin = 0x000050, length = 0x0003B0 /* on-chip RAM block M0 */ //RAMM1 : origin = 0x000400, length = 0x000400 /* on-chip RAM block M1 */ RAML4 : origin = 0x00C000, length = 0x001000 /* on-chip RAM block L1 */ RAML5 : origin = 0x00D000, length = 0x001000 /* on-chip RAM block L1 */ RAML6 : origin = 0x00E000, length = 0x001000 /* on-chip RAM block L1 */ RAML7 : origin = 0x00F000, length = 0x001000 /* on-chip RAM block L1 */ ZONE7B : origin = 0x20FC00, length = 0x000400 /* XINTF zone 7 - data space */ FLASHB : origin = 0x330000, length = 0x008000 /* on-chip FLASH */ } /* Allocate sections to memory blocks. Note: codestart user defined section in DSP28_CodeStartBranch.asm used to redirect code execution when booting to flash ramfuncs user defined section to store functions that will be copied from Flash into RAM */ SECTIONS { /* Allocate program areas: */ .cinit : > FLASHB, PAGE=1, LOAD_SIZE(_cinit_loadsize), crc_table(_cinitCRC, algorithm=CRC32_PRIME) .cio : > RAML0L1L2L3L4L5L6L7, PAGE = 0 .sysmem : > RAML0L1L2L3L4L5L6L7, PAGE = 0 .text : > FLASHDC, PAGE = 0, LOAD_SIZE(_text_loadsize), crc_table(_textCRC, algorithm=CRC32_PRIME) codestart : > BEGIN, PAGE = 0, LOAD_SIZE(_codestart_loadsize), crc_table(_codestartCRC, algorithm=CRC32_PRIME) ramfuncs : LOAD = FLASHDC, PAGE = 0 RUN = RAML0L1L2L3L4L5L6L7, PAGE = 0 LOAD_START(_ramfuncs_loadstart), LOAD_SIZE(_ramfuncs_loadsize), RUN_START(_ramfuncs_runstart), crc_table(_ramfuncsCRC, algorithm=CRC32_PRIME) csmpasswds : > CSM_PWL PAGE = 0 csm_rsvd : > CSM_RSVD PAGE = 0 /* Allocate uninitalized data sections: */ .stack : > RAMM01 PAGE = 1 .ebss : > RAML0L1L2L3L4L5L6L7 PAGE = 0 .esysmem : > RAML0L1L2L3L4L5L6L7 PAGE = 0 /* Initalized sections go in Flash */ /* For SDFlash to program these, they must be allocated to page 0 */ .econst : > FLASHDC PAGE = 0, LOAD_SIZE(_econst_loadsize), crc_table(_econstCRC, algorithm=CRC32_PRIME) // .econst : LOAD = FLASHA, RUN = RAML0L1L2L3L4L5L6L7, PAGE = 0 .switch : > FLASHDC PAGE = 0 /* Allocate IQ math areas: */ IQmath : > FLASHDC PAGE = 0 /* Math Code */ IQmathTables : > IQTABLES, PAGE = 0, TYPE = NOLOAD /* Uncomment the section below if calling the IQNexp() or IQexp() functions from the IQMath.lib library in order to utilize the relevant IQ Math table in Boot ROM (This saves space and Boot ROM is 1 wait-state). If this section is not uncommented, IQmathTables2 will be loaded into other memory (SARAM, Flash, etc.) and will take up space, but 0 wait-state is possible. */ /* IQmathTables2 : > IQTABLES2, PAGE = 0, TYPE = NOLOAD { IQmath.lib<IQNexpTable.obj> (IQmathTablesRam) } */ FPUmathTables : > FPUTABLES, PAGE = 0, TYPE = NOLOAD /* Allocate DMA-accessible RAM sections: */ DMARAML4 : > RAML4, PAGE = 1 DMARAML5 : > RAML5, PAGE = 1 DMARAML6 : > RAML6, PAGE = 1 DMARAML7 : > RAML7, PAGE = 1 /* Allocate 0x400 of XINTF Zone 7 to storing data */ ZONE7DATA : > ZONE7B, PAGE = 1 /* .reset is a standard section used by the compiler. It contains the */ /* the address of the start of _c_int00 for C Code. /* /* When using the boot ROM this section and the CPU vector */ /* table is not needed. Thus the default type is set here to */ /* DSECT */ .reset : > RESET, PAGE = 0, TYPE = DSECT vectors : > VECTORS PAGE = 0, TYPE = DSECT /* Allocate ADC_cal function (pre-programmed by factory into TI reserved memory) */ .adc_cal : load = ADC_CAL, PAGE = 0, TYPE = NOLOAD .TI.crctab: > FLASHH } /* //=========================================================================== // End of file. //=========================================================================== */
I call gen_crc() at the beginning of main() as follows:
gen_crc(&ramfuncs_loadstart,(Uint32)&ramfuncs_loadsize);
Does anyone notice anything wrong with the table or code?
Stephen
Modified gen_crc function:
extern const unsigned long crcTable[]; /*****************************************************************************/ /* gen_crc - Return the CRC value for the data using the given CRC algorithm */ /* int id : identifies the CRC algorithm */ /* char *data : the data */ /* size_t len : the size of the data */ /*****************************************************************************/ Uint32 gen_crc(const Uint16 *data, Uint32 len) { /*-----------------------------------------------------------------------*/ /* Note: this is not a general-purpose CRC function. It does not handle */ /* parameterization by initial value, bit reflection, or final XOR */ /* value. This CRC function is specialized to the CRC algorithms in the */ /* linker used for C28x CRC tables. */ /*-----------------------------------------------------------------------*/ /*-----------------------------------------------------------------------*/ /* This CRC function is not intended to be optimal; it is written such */ /* that it works and generates the same result on all 8-bit and 16-bit */ /* targets, including C27x, C28x, other TI DSPs, and typical desktops. */ /*-----------------------------------------------------------------------*/ Uint32 crc = 0; Uint32 mask = 0x4BEAB53BUL; Uint32 i; for (i = 0; i < len; i++) { Uint16 datum = data[i]; /*--------------------------------------------------------------------*/ /* This loop handles 16-bit chars when we compile on 16-bit machines. */ /*--------------------------------------------------------------------*/ int n; for (n = 0; n < 2; n++) { /*----------------------------------------------------------------*/ /* For 16-bit machines, we need to feed the octets in an */ /* arbitrary order. For C2000, the arbitrary order we choose is */ /* to feed the LEAST significant octet of char 0 first. The */ /* first octet fed to the CRC is the LEAST-significant octet of */ /* char 0; the second octet is the MOST-significant octet of char */ /* 0. See the "Special Note regarding 16-bit char" in the */ /* Assembly Language Tools User's Guide. */ /*----------------------------------------------------------------*/ /*----------------------------------------------------------------*/ /* Using __byte is not necessary; we use it here to illustrate */ /* how it relates to octet order. */ /*----------------------------------------------------------------*/ Uint32 octet = __byte((int16*) &datum, n); Uint32 term1 = (crc << 8); int16 idx = ((crc >> 24) & 0xff) ^ octet; crc = term1 ^ crcTable[idx]; } } return crc & mask; }
CRC table:
const unsigned long crcTable[] = { 0x00000000,0x04C11DB7,0x09823B6E,0x0D4326D9, 0x130476DC,0x17C56B6B,0x1A864DB2,0x1E475005, 0x2608EDB8,0x22C9F00F,0x2F8AD6D6,0x2B4BCB61, 0x350C9B64,0x31CD86D3,0x3C8EA00A,0x384FBDBD, 0x4C11DB70,0x48D0C6C7,0x4593E01E,0x4152FDA9, 0x5F15ADAC,0x5BD4B01B,0x569796C2,0x52568B75, 0x6A1936C8,0x6ED82B7F,0x639B0DA6,0x675A1011, 0x791D4014,0x7DDC5DA3,0x709F7B7A,0x745E66CD, 0x9823B6E0,0x9CE2AB57,0x91A18D8E,0x95609039, 0x8B27C03C,0x8FE6DD8B,0x82A5FB52,0x8664E6E5, 0xBE2B5B58,0xBAEA46EF,0xB7A96036,0xB3687D81, 0xAD2F2D84,0xA9EE3033,0xA4AD16EA,0xA06C0B5D, 0xD4326D90,0xD0F37027,0xDDB056FE,0xD9714B49, 0xC7361B4C,0xC3F706FB,0xCEB42022,0xCA753D95, 0xF23A8028,0xF6FB9D9F,0xFBB8BB46,0xFF79A6F1, 0xE13EF6F4,0xE5FFEB43,0xE8BCCD9A,0xEC7DD02D, 0x34867077,0x30476DC0,0x3D044B19,0x39C556AE, 0x278206AB,0x23431B1C,0x2E003DC5,0x2AC12072, 0x128E9DCF,0x164F8078,0x1B0CA6A1,0x1FCDBB16, 0x018AEB13,0x054BF6A4,0x0808D07D,0x0CC9CDCA, 0x7897AB07,0x7C56B6B0,0x71159069,0x75D48DDE, 0x6B93DDDB,0x6F52C06C,0x6211E6B5,0x66D0FB02, 0x5E9F46BF,0x5A5E5B08,0x571D7DD1,0x53DC6066, 0x4D9B3063,0x495A2DD4,0x44190B0D,0x40D816BA, 0xACA5C697,0xA864DB20,0xA527FDF9,0xA1E6E04E, 0xBFA1B04B,0xBB60ADFC,0xB6238B25,0xB2E29692, 0x8AAD2B2F,0x8E6C3698,0x832F1041,0x87EE0DF6, 0x99A95DF3,0x9D684044,0x902B669D,0x94EA7B2A, 0xE0B41DE7,0xE4750050,0xE9362689,0xEDF73B3E, 0xF3B06B3B,0xF771768C,0xFA325055,0xFEF34DE2, 0xC6BCF05F,0xC27DEDE8,0xCF3ECB31,0xCBFFD686, 0xD5B88683,0xD1799B34,0xDC3ABDED,0xD8FBA05A, 0x690CE0EE,0x6DCDFD59,0x608EDB80,0x644FC637, 0x7A089632,0x7EC98B85,0x738AAD5C,0x774BB0EB, 0x4F040D56,0x4BC510E1,0x46863638,0x42472B8F, 0x5C007B8A,0x58C1663D,0x558240E4,0x51435D53, 0x251D3B9E,0x21DC2629,0x2C9F00F0,0x285E1D47, 0x36194D42,0x32D850F5,0x3F9B762C,0x3B5A6B9B, 0x0315D626,0x07D4CB91,0x0A97ED48,0x0E56F0FF, 0x1011A0FA,0x14D0BD4D,0x19939B94,0x1D528623, 0xF12F560E,0xF5EE4BB9,0xF8AD6D60,0xFC6C70D7, 0xE22B20D2,0xE6EA3D65,0xEBA91BBC,0xEF68060B, 0xD727BBB6,0xD3E6A601,0xDEA580D8,0xDA649D6F, 0xC423CD6A,0xC0E2D0DD,0xCDA1F604,0xC960EBB3, 0xBD3E8D7E,0xB9FF90C9,0xB4BCB610,0xB07DABA7, 0xAE3AFBA2,0xAAFBE615,0xA7B8C0CC,0xA379DD7B, 0x9B3660C6,0x9FF77D71,0x92B45BA8,0x9675461F, 0x8832161A,0x8CF30BAD,0x81B02D74,0x857130C3, 0x5D8A9099,0x594B8D2E,0x5408ABF7,0x50C9B640, 0x4E8EE645,0x4A4FFBF2,0x470CDD2B,0x43CDC09C, 0x7B827D21,0x7F436096,0x7200464F,0x76C15BF8, 0x68860BFD,0x6C47164A,0x61043093,0x65C52D24, 0x119B4BE9,0x155A565E,0x18197087,0x1CD86D30, 0x029F3D35,0x065E2082,0x0B1D065B,0x0FDC1BEC, 0x3793A651,0x3352BBE6,0x3E119D3F,0x3AD08088, 0x2497D08D,0x2056CD3A,0x2D15EBE3,0x29D4F654, 0xC5A92679,0xC1683BCE,0xCC2B1D17,0xC8EA00A0, 0xD6AD50A5,0xD26C4D12,0xDF2F6BCB,0xDBEE767C, 0xE3A1CBC1,0xE760D676,0xEA23F0AF,0xEEE2ED18, 0xF0A5BD1D,0xF464A0AA,0xF9278673,0xFDE69BC4, 0x89B8FD09,0x8D79E0BE,0x803AC667,0x84FBDBD0, 0x9ABC8BD5,0x9E7D9662,0x933EB0BB,0x97FFAD0C, 0xAFB010B1,0xAB710D06,0xA6322BDF,0xA2F33668, 0xBCB4666D,0xB8757BDA,0xB5365D03,0xB1F740B4 };