• State Resolved
  • Locked Locked
  • Replies 1 reply
  • Subscribers 98 subscribers
  • Views 257 views
  • Users 0 members are here
Support feedback
Options
Options
Related

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.

Bugs in PSP Diver ?

John Wilkes
Expert 1315 points
Other Parts Discussed in Thread: OMAP-L137

Hello,

 

I’m not sure if this is the right place to post this but I believe I have found a couple of bugs in the CSLR part of pspdrivers_01_20_00_07:

 

1. In the 'cslr_psc_OMAPL137.h' file, the definition of the 'CSL_Psc1Peripheral' enum does not agree with table 8.2 in "OMAP-L137 Applications Processor System Reference Guide" (SPRUG84b, page 85). The definitions of the lower entries in the enum are one lower. From experimentation, I think the document is correct and the driver code is wrong.

 

2. The file 'cslr_syscfg_OMAPL137.h' does not contain the closing bracket of the extern "C" statement that is conditional on the __cplusplus label. This causes errors if the file is used in a C++ project. The following code is required before the final '#endif':

 

#ifdef __cplusplus

}

#endif

 

Cheers,

 

John.

  • 0
    TI__Mastermind 24340 points

    Thanks for the feedback.

    John Wilkes said:
    1. In the 'cslr_psc_OMAPL137.h' file, the definition of the 'CSL_Psc1Peripheral' enum does not agree with table 8.2 in "OMAP-L137 Applications Processor System Reference Guide" (SPRUG84b, page 85). The definitions of the lower entries in the enum are one lower. From experimentation, I think the document is correct and the driver code is wrong.

    The document is correct. I just checked and this bug is corrected on version pspdrivers_01_30_00_05, that is available for download at:

    https://www-a.ti.com/downloads/sds_support/targetcontent/psp/bios_psp/index.html   (you will need a ti.com ID).

    John Wilkes said:
    2. The file 'cslr_syscfg_OMAPL137.h' does not contain the closing bracket of the extern "C" statement that is conditional on the __cplusplus label. This causes errors if the file is used in a C++ project. The following code is required before the final '#endif':
     
    #ifdef __cplusplus
    }
    #endif

     I will give feedback about this to the PSP team.