Because of the holidays, TI E2E™ design support forum responses will be delayed from Dec. 25 through Jan. 2. Thank you for your patience.

  • State Resolved
  • Locked Locked
  • Replies 2 replies
  • Subscribers 83 subscribers
  • Views 413 views
  • Users 0 members are here
Support feedback
Options
Options
Similar topics

This thread has been locked.

If you have a related question, please click the "Ask a related question" button in the top right corner. The newly created question will be automatically linked to this question.

TMS320F28P659DK-Q1: Package C2000 5.0.0.0 with support for 28P65: issues in some files

Davide Dentoni
Prodigy 175 points
Part Number: TMS320F28P659DK-Q1
Other Parts Discussed in Thread: C2000WARE

Hello support,
we have integrated new C2000 Ware 5.0.0.0 package into our project based on DSP 28P65.


We think we have found some issues that we submit to you:
1. file \libraries\flash_api\f28p65x\include\FlashAPI\Types.h: enum Fapi_FlashBankType contains only Fapi_FlashBank0, while in our opinion also Fapi_FlashBank1 and Fapi_FlashBank2 should be defined.

Could you confirm ?


2. file \driverlib\f28p65x\driverlib\inc\hw_emif.h: we think that definitions related to register offset are not aligned to the official TRM. Could you confirm ?


3. file \device_support\f28p65x\common\include\f28p65x_ipc_defines.h: we have found inconsistency between the field PUMPREQUEST used in macros SeizeFlashPump() and ReleaseFlashPump()
    and the definition of structure CPU1TOCPU2_IPC_REGS_CPU1VIEW (file f28p65x_ipc.h) that contain field FLASHCTLSEM instead of PUMPREQUEST.

Could you confirm ?

  • 0
    TI__Mastermind 33885 points

    Hi, 

    Thanks for the post. 

    1. Yes, I think the enum Fapi_FlashBankType should have additional entries for other banks.

    2. Regarding the hw_emif.h, the register offsets do match with the ones specified in the TRM .  Pls see the snapshots below of the TRM and the hw_emif.h files.

    Pls clarify where are you seeing the mismatch.

    3.  PUMPREQUEST should have been FLASHCTLSEM

    Will raise these as issues and get them fixed .

    Best Regards

    Siddharth

  • 0 in reply to Siddharth Deshpande
    Prodigy 175 points

    Hello,

    thank you for your feedback.

    About point 2, I was referring to the #define EMIF_O_TOTAL_SDRAM_AR_INC_ (file hw_emif.h) that does not match the register name in the TRM, while it matches the name of a register contained in the preliminary TRM that had a different offset (see image here below).

    I agree with you: offset definitions in file hw_emif.h of C2000WARE 5.0.0.0 are coherent with the official TRM (except for the suffix _INC_)