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.

Typo/Bug Found in CAN Bootloader Code

Hello TI/QJ,

I noticed this in the bl_config.h file for the CAN bootloader:

//*****************************************************************************
//
// Selects the baud rate to be used for the UART.
//
// Depends on: UART_ENABLE_UPDATE
// Requires: None
// UART Baud Rates can be: 9600, 19200, 38400, 57600, 115200
//*****************************************************************************
#define UART_BAUDRATE 118200
#define UART sciREG1 /* Use UART port 1 for UART boot */


Should that be 115200? Guess it's a typo?

Is there a better place to post when this type of issue is found?

Thanks,
JohnW.

  • Hello John,

    On first glance it would appear that your are correct in that this #define should be 115200 instead of 118200; however, I don't have access to the code to investigate and I'm not familiar with how this parameter is used in the code so I can't definitively say if it is a bug or not. I have sent your post to QJ so that he can have a look at it and respond. If you don't hear back from him in a few days starting Monday, let me know.
  • Hello Chuck,

    In the other two files is the following:

    boot_spi:

    bl_config.h:

    //*****************************************************************************

    //

    // Selects the baud rate to be used for the UART.

    //

    // Depends on: UART_ENABLE_UPDATE

    // Requires: None

    // UART Baud Rates can be: 9600, 19200, 38400, 57600, 115200

    //*****************************************************************************

    #define UART_BAUDRATE     115200

    #define UART              sciREG1   /* Use UART port 1 for UART boot */

    boot_uart:

    bl_config.h:

    //*****************************************************************************

    //

    // Selects the baud rate to be used for the UART.

    //

    // Depends on: UART_ENABLE_UPDATE

    // Requires: None

    // UART Baud Rates can be: 9600, 19200, 38400, 57600, 115200

    //*****************************************************************************

    #define UART_BAUDRATE     115200

    #define UART              sciREG1   /* Use UART port 1 for UART boot */

    and in boot_can:

    bl_config.h:

    //*****************************************************************************

    //

    // Selects the baud rate to be used for the UART.

    //

    // Depends on: UART_ENABLE_UPDATE

    // Requires: None

    // UART Baud Rates can be: 9600, 19200, 38400, 57600, 115200

    //*****************************************************************************

    #define UART_BAUDRATE     118200

    #define UART              sciREG1   /* Use UART port 1 for UART boot */

    That's clearly a typo and needs to be fixed.  This is from the archive on the following page:

    Regards,
    johnw

  • Hi John,

    Understood. The software developer is on copy and receives an email each time a post is made to this thread.