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.

TDA4VM: MCU2-1 TimerP_create , will cause appLogWaitMsecs timer error .

Part Number: TDA4VM

We find MCU2-1 call TimerP_create id=0, will cause c71 appLogWaitMsecs  stuck.

Is there have any update?

  • Our sdk version is 08.04.00.02

  • Hi ,

    This is very strange, timer id is hear DM Timer12 on the main domain, where as, API appLogWaitMsecs uses GTC timer, so there should not be any issue in using these two timers.. 

    Can you please share the testcase on EVM, if possible? 

    Regards,

    Brijesh

  • Hi Brijesh,

    I test on evm board , the resule is the same .

    You can check as follow code.

    If disable timerP , cx7_1 will run ok.

    /platform/j721s2/rtos/mcu2_1

    main.c 

    /*
     *
     * Copyright (c) 2018 Texas Instruments Incorporated
     *
     * All rights reserved not granted herein.
     *
     * Limited License.
     *
     * Texas Instruments Incorporated grants a world-wide, royalty-free, non-exclusive
     * license under copyrights and patents it now or hereafter owns or controls to make,
     * have made, use, import, offer to sell and sell ("Utilize") this software subject to the
     * terms herein.  With respect to the foregoing patent license, such license is granted
     * solely to the extent that any such patent is necessary to Utilize the software alone.
     * The patent license shall not apply to any combinations which include this software,
     * other than combinations with devices manufactured by or for TI ("TI Devices").
     * No hardware patent is licensed hereunder.
     *
     * Redistributions must preserve existing copyright notices and reproduce this license
     * (including the above copyright notice and the disclaimer and (if applicable) source
     * code license limitations below) in the documentation and/or other materials provided
     * with the distribution
     *
     * Redistribution and use in binary form, without modification, are permitted provided
     * that the following conditions are met:
     *
     * *       No reverse engineering, decompilation, or disassembly of this software is
     * permitted with respect to any software provided in binary form.
     *
     * *       any redistribution and use are licensed by TI for use only with TI Devices.
     *
     * *       Nothing shall obligate TI to provide you with source code for the software
     * licensed and provided to you in object code.
     *
     * If software source code is provided to you, modification and redistribution of the
     * source code are permitted provided that the following conditions are met:
     *
     * *       any redistribution and use of the source code, including any resulting derivative
     * works, are licensed by TI for use only with TI Devices.
     *
     * *       any redistribution and use of any object code compiled from the source code
     * and any resulting derivative works, are licensed by TI for use only with TI Devices.
     *
     * Neither the name of Texas Instruments Incorporated nor the names of its suppliers
     *
     * may be used to endorse or promote products derived from this software without
     * specific prior written permission.
     *
     * DISCLAIMER.
     *
     * THIS SOFTWARE IS PROVIDED BY TI AND TI'S LICENSORS "AS IS" AND ANY EXPRESS
     * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
     * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
     * IN NO EVENT SHALL TI AND TI'S LICENSORS BE LIABLE FOR ANY DIRECT, INDIRECT,
     * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
     * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
     * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
     * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
     * OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
     * OF THE POSSIBILITY OF SUCH DAMAGE.
     *
     */
    
    #include <app.h>
    #include <utils/console_io/include/app_log.h>
    #include <stdio.h>
    #include <string.h>
    #include <ti/osal/osal.h>
    #include <ti/osal/TaskP.h>
    #include <ti/osal/TimerP.h>
    #include <app_ipc_rsctable.h>
    
    
    
    
    #ifndef NULL_PTR
    #define NULL_PTR ((void *)0)
    #endif
    #define CLOCK_INTERVAL_TIME			(1000)		//10ms
    #define CLOCK_INTERVAL_UNIT_US		(CLOCK_INTERVAL_TIME*1000)
    
    static TimerP_Params Ch1_CanMsg_TimerParams;
    static TimerP_Handle Ch1_CanMsg_Timer = NULL;
    static TimerP_Params Ch2_CanMsg_TimerParams;
    static TimerP_Handle Ch2_CanMsg_Timer = NULL;
    
    static void ch1_CanMsg_ClockHandler()
    {
        appLogPrintf("ch1_CanMsg_ClockHandler: action!!! ... !!!\n");
    }
    
    
    static void ch2_CanMsg_ClockHandler()
    {
        appLogPrintf("ch2_CanMsg_ClockHandler: action!!! ... !!!\n");
    }
    
    static void timer_handle()
    {
        TimerP_Handle handle;
        TimerP_Params_init(&Ch1_CanMsg_TimerParams);
    	Ch1_CanMsg_TimerParams.runMode     = TimerP_RunMode_CONTINUOUS;
    	Ch1_CanMsg_TimerParams.startMode   = TimerP_StartMode_AUTO ;
    	Ch1_CanMsg_TimerParams.periodType  = TimerP_PeriodType_MICROSECS ;
    	Ch1_CanMsg_TimerParams.period      = CLOCK_INTERVAL_UNIT_US;
    	Ch1_CanMsg_TimerParams.arg         = NULL_PTR;
    	Ch1_CanMsg_Timer = TimerP_create(0, (TimerP_Fxn)&ch1_CanMsg_ClockHandler, &Ch1_CanMsg_TimerParams);
        TimerP_start((TimerP_Handle)Ch1_CanMsg_Timer);
    
    #if 0
        TimerP_Params_init(&Ch2_CanMsg_TimerParams);
    	Ch2_CanMsg_TimerParams.runMode     = TimerP_RunMode_CONTINUOUS;
    	Ch2_CanMsg_TimerParams.startMode   = TimerP_StartMode_AUTO ;
    	Ch2_CanMsg_TimerParams.periodType  = TimerP_PeriodType_MICROSECS ;
    	Ch2_CanMsg_TimerParams.period      = CLOCK_INTERVAL_UNIT_US;
    	Ch2_CanMsg_TimerParams.arg         = NULL_PTR;
    	Ch2_CanMsg_Timer = TimerP_create(2, (TimerP_Fxn)&ch2_CanMsg_ClockHandler, &Ch2_CanMsg_TimerParams);
    
        TimerP_start((TimerP_Handle)Ch2_CanMsg_Timer);
    #endif
    }
    
    static void appMain(void* arg0, void* arg1)
    {
        int count = 0;
        appInit();
        appRun();
        appLogPrintf("========= timer_handle =========\n");
        timer_handle();
        #if 1
        while(1)
        {
            appLogWaitMsecs(100u);
            if(count%10 == 0)  appLogPrintf("mcu2_1_run[%d]\n",count);
            count++;
            
        }
        #else
        appDeInit();
        #endif
    }
    
    void StartupEmulatorWaitFxn (void)
    {
        volatile uint32_t enableDebug = 0;
        do
        {
        }while (enableDebug);
    }
    
    static uint8_t gTskStackMain[8*1024]
    __attribute__ ((section(".bss:taskStackSection")))
    __attribute__ ((aligned(8192)))
        ;
    
    int main(void)
    {
        TaskP_Params tskParams;
        TaskP_Handle task;
    
        /* This is for debug purpose - see the description of function header */
        StartupEmulatorWaitFxn();
    
        OS_init();
    
        TaskP_Params_init(&tskParams);
        tskParams.priority = 8u;
        tskParams.stack = gTskStackMain;
        tskParams.stacksize = sizeof (gTskStackMain);
        task = TaskP_create(appMain, &tskParams);
        if(NULL == task)
        {
            OS_stop();
        }
        OS_start();
    
        return 0;
    }
    
    uint32_t appGetDdrSharedHeapSize()
    {
        return DDR_SHARED_MEM_SIZE;
    }
    

    /platform/j721s2/rtos/c7x_1

    main.c 

    /*
     *
     * Copyright (c) 2018 Texas Instruments Incorporated
     *
     * All rights reserved not granted herein.
     *
     * Limited License.
     *
     * Texas Instruments Incorporated grants a world-wide, royalty-free, non-exclusive
     * license under copyrights and patents it now or hereafter owns or controls to make,
     * have made, use, import, offer to sell and sell ("Utilize") this software subject to the
     * terms herein.  With respect to the foregoing patent license, such license is granted
     * solely to the extent that any such patent is necessary to Utilize the software alone.
     * The patent license shall not apply to any combinations which include this software,
     * other than combinations with devices manufactured by or for TI ("TI Devices").
     * No hardware patent is licensed hereunder.
     *
     * Redistributions must preserve existing copyright notices and reproduce this license
     * (including the above copyright notice and the disclaimer and (if applicable) source
     * code license limitations below) in the documentation and/or other materials provided
     * with the distribution
     *
     * Redistribution and use in binary form, without modification, are permitted provided
     * that the following conditions are met:
     *
     * *       No reverse engineering, decompilation, or disassembly of this software is
     * permitted with respect to any software provided in binary form.
     *
     * *       any redistribution and use are licensed by TI for use only with TI Devices.
     *
     * *       Nothing shall obligate TI to provide you with source code for the software
     * licensed and provided to you in object code.
     *
     * If software source code is provided to you, modification and redistribution of the
     * source code are permitted provided that the following conditions are met:
     *
     * *       any redistribution and use of the source code, including any resulting derivative
     * works, are licensed by TI for use only with TI Devices.
     *
     * *       any redistribution and use of any object code compiled from the source code
     * and any resulting derivative works, are licensed by TI for use only with TI Devices.
     *
     * Neither the name of Texas Instruments Incorporated nor the names of its suppliers
     *
     * may be used to endorse or promote products derived from this software without
     * specific prior written permission.
     *
     * DISCLAIMER.
     *
     * THIS SOFTWARE IS PROVIDED BY TI AND TI'S LICENSORS "AS IS" AND ANY EXPRESS
     * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
     * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
     * IN NO EVENT SHALL TI AND TI'S LICENSORS BE LIABLE FOR ANY DIRECT, INDIRECT,
     * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
     * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
     * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
     * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
     * OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
     * OF THE POSSIBILITY OF SUCH DAMAGE.
     *
     */
    
    #include <app.h>
    #include <utils/console_io/include/app_log.h>
    #include <utils/misc/include/app_misc.h>
    #include <stdio.h>
    #include <string.h>
    #include <ti/osal/osal.h>
    #include <ti/osal/TaskP.h>
    #include <ti/osal/HwiP.h>
    #include <ti/osal/CacheP.h>
    #include <app_mem_map.h>
    #include <app_ipc_rsctable.h>
    #include <ti/csl/soc.h>
    #include <ti/csl/csl_clec.h>
    
    #if (defined (FREERTOS))
    #include <ti/kernel/freertos/portable/TI_CGT/c7x/Cache.h>
    #include <ti/kernel/freertos/portable/TI_CGT/c7x/Hwi.h>
    #include <ti/kernel/freertos/portable/TI_CGT/c7x/Mmu.h>
    #else
    #include <ti/sysbios/family/c7x/Cache.h>
    #include <ti/sysbios/family/c7x/Hwi.h>
    #include <ti/sysbios/family/c7x/Mmu.h>
    #endif
    
    /* For J7ES/J721E/TDA4VM the upper 2GB DDR starts from 0x0008_8000_0000 */
    /* This address is mapped to a virtual address of 0x0001_0000_0000 */
    #define DDR_C7X_1_LOCAL_HEAP_VADDR (DDR_C7X_1_LOCAL_HEAP_ADDR)
    #define DDR_C7X_1_LOCAL_HEAP_PADDR (DDR_64BIT_BASE_PADDR + (DDR_C7X_1_LOCAL_HEAP_ADDR - DDR_64BIT_BASE_VADDR))
    
    #define DDR_C7X_1_SCRATCH_VADDR    (DDR_C7X_1_SCRATCH_ADDR)
    #define DDR_C7X_1_SCRATCH_PADDR    (DDR_64BIT_BASE_PADDR + (DDR_C7X_1_SCRATCH_ADDR - DDR_64BIT_BASE_VADDR))
    static void appMain(void* arg0, void* arg1)
    {
        int count = 0;
        appInit();
        appRun();
        #if 1
        while(1)
        {
            appLogWaitMsecs(100u);
            if(count%10 == 0)  appLogPrintf("c71_run[%d]\n",count);
            count++;
        }
        #else
        appDeInit();
        #endif
    }
    
    void StartupEmulatorWaitFxn (void)
    {
        volatile uint32_t enableDebug = 0;
        do
        {
        }while (enableDebug);
    }
    
    /* IMPORTANT NOTE: For C7x,
     * - stack size and stack ptr MUST be 8KB aligned
     * - AND min stack size MUST be 16KB
     * - AND stack assigned for task context is "size - 8KB"
     *       - 8KB chunk for the stack area is used for interrupt handling in this task context
     */
    static uint8_t gTskStackMain[64*1024]
    __attribute__ ((section(".bss:taskStackSection")))
    __attribute__ ((aligned(8192)))
        ;
    
    /* DRU configuration */
    #define J721E_DDR_QOS_EXP_DRU_QUEUE_PRIORITY   (2)
    #define J721E_DDR_QOS_EXP_DRU_QUEUE_ORDER_ID   (4)
    #define J7ES_DRU_NUM_CH (5)
    #define J7ES_DRU_CFG_y(i) (0x6D008000 + ((i) * 8))
    #define writel(x,y) (*((uint32_t *)(y))=(x))
    
    void setup_dru_qos(void)
    {
        #if 0
        unsigned int channel;
    
        for (channel = 0; channel < J7ES_DRU_NUM_CH; ++channel)
        {
            writel((J721E_DDR_QOS_EXP_DRU_QUEUE_ORDER_ID << 4) | J721E_DDR_QOS_EXP_DRU_QUEUE_PRIORITY, J7ES_DRU_CFG_y(channel));
        }
        #endif
    }
    
    /* A copy of this function is in both C7 main files, except the cfgClec.rtMap value */
    static void appC7xClecInitDru(void)
    {
        CSL_ClecEventConfig   cfgClec;
        CSL_CLEC_EVTRegs   *clecBaseAddr = (CSL_CLEC_EVTRegs*) CSL_COMPUTE_CLUSTER0_CLEC_REGS_BASE;
    
        uint32_t i;
        uint32_t dru_input_start = 192;
        uint32_t dru_input_num   = 16;
        /* program CLEC events from DRU used for polling by TIDL
         * to map to required events in C7x
         */
        for(i=dru_input_start; i<(dru_input_start+dru_input_num); i++)
        {
            /* Configure CLEC */
            cfgClec.secureClaimEnable = FALSE;
            cfgClec.evtSendEnable     = TRUE;
    
            /* cfgClec.rtMap value is different for each C7x */
            cfgClec.rtMap             = CSL_CLEC_RTMAP_CPU_4;
    
            cfgClec.extEvtNum         = 0;
            cfgClec.c7xEvtNum         = (i-dru_input_start)+32;
            CSL_clecConfigEvent(clecBaseAddr, i, &cfgClec);
        }
    }
    
    int main(void)
    {
        TaskP_Params tskParams;
        TaskP_Handle task;
    
        OS_init();
    
        appC7xClecInitDru();
    
        setup_dru_qos();
    
        TaskP_Params_init(&tskParams);
        tskParams.priority = 8u;
        tskParams.stack = gTskStackMain;
        tskParams.stacksize = sizeof (gTskStackMain);
        task = TaskP_create(appMain, &tskParams);
        if(NULL == task)
        {
            OS_stop();
        }
        OS_start();
    
        return 0;
    }
    
    uint32_t g_app_rtos_c7x_mmu_map_error = 0;
    
    void appMmuMap(Bool is_secure)
    {
        Bool            retVal;
        Mmu_MapAttrs    attrs;
    
        uint32_t ns = 1;
    
        if(is_secure)
            ns = 0;
        else
            ns = 1;
    
        Mmu_initMapAttrs(&attrs);
    
        attrs.attrIndx = Mmu_AttrIndx_MAIR0;
        attrs.ns = ns;
    
        retVal = Mmu_map(0x00000000U, 0x00000000U, 0x20000000U, &attrs, is_secure);
        if(retVal==FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x20000000U, 0x20000000U, 0x20000000U, &attrs, is_secure);
        if(retVal==FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x40000000U, 0x40000000U, 0x20000000U, &attrs, is_secure);
        if(retVal==FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x60000000U, 0x60000000U, 0x10000000U, &attrs, is_secure);
        if(retVal==FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x78000000U, 0x78000000U, 0x08000000U, &attrs, is_secure); /* CLEC */
        if(retVal==FALSE)
        {
            goto mmu_exit;
        }
    
        attrs.attrIndx = Mmu_AttrIndx_MAIR7;
    
        retVal = Mmu_map(0x80000000U, 0x80000000U, 0x20000000U, &attrs, is_secure); /* OCMC - 1MB */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0xA0000000U, 0xA0000000U, 0x20000000U, &attrs, is_secure); /* OCMC - 1MB */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x70000000U, 0x70000000U, 0x00400000U, &attrs, is_secure); /* MSMC - 4MB */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(0x41C00000U, 0x41C00000U, 0x00100000U, &attrs, is_secure); /* OCMC - 1MB */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        /*The region mapped by the MMU is intentionally set to 2MB for L2 SRAM since
          page sizes are a function of the region size and having smaller page sizes
          negatively affects the performance of L2 SRAM as the table walks with the 
          translation table in DDR are expensive, especially in context of high- 
          throughput, low-latency memory like L2 SRAM*/
        retVal = Mmu_map(L2RAM_C7x_1_ADDR, L2RAM_C7x_1_ADDR, 0x00200000, &attrs, is_secure); /* L2 sram 448KB   */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(DDR_C7x_1_DTS_ADDR, DDR_C7x_1_DTS_ADDR, DDR_C7x_1_DTS_SIZE, &attrs, is_secure); /* ddr            */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(DDR_C7X_1_LOCAL_HEAP_VADDR, DDR_C7X_1_LOCAL_HEAP_PADDR, DDR_C7X_1_LOCAL_HEAP_SIZE, &attrs, is_secure); /* ddr            */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(DDR_C7X_1_SCRATCH_VADDR, DDR_C7X_1_SCRATCH_PADDR, DDR_C7X_1_SCRATCH_SIZE, &attrs, is_secure); /* ddr            */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(DDR_SHARED_MEM_ADDR, DDR_SHARED_MEM_ADDR, DDR_SHARED_MEM_SIZE, &attrs, is_secure); /* ddr            */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        attrs.attrIndx = Mmu_AttrIndx_MAIR4;
    
        retVal = Mmu_map(APP_LOG_MEM_ADDR, APP_LOG_MEM_ADDR, APP_LOG_MEM_SIZE, &attrs, is_secure);
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(TIOVX_OBJ_DESC_MEM_ADDR, TIOVX_OBJ_DESC_MEM_ADDR, TIOVX_OBJ_DESC_MEM_SIZE, &attrs, is_secure);
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(IPC_VRING_MEM_ADDR, IPC_VRING_MEM_ADDR, IPC_VRING_MEM_SIZE, &attrs, is_secure);
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(DDR_C7x_1_IPC_ADDR, DDR_C7x_1_IPC_ADDR, DDR_C7x_1_IPC_SIZE, &attrs, is_secure); /* ddr            */
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
        retVal = Mmu_map(TIOVX_LOG_RT_MEM_ADDR, TIOVX_LOG_RT_MEM_ADDR, TIOVX_LOG_RT_MEM_SIZE, &attrs, is_secure);
        if(retVal == FALSE)
        {
            goto mmu_exit;
        }
    
    mmu_exit:
        if(retVal == FALSE)
        {
            g_app_rtos_c7x_mmu_map_error++;
        }
    
    
    
        return;
    }
    
    void appCacheInit()
    {
        Cache_Size  cacheSize;
    
        /* init cache size here, since this needs to be done in secure mode */
        cacheSize.l1pSize = Cache_L1Size_32K;
        cacheSize.l1dSize = Cache_L1Size_32K;
        cacheSize.l2Size  = Cache_L2Size_64K;
    
        Cache_setSize(&cacheSize);
    }
    
    void InitMmu(void)
    {
        /* This is for debug purpose - see the description of function header */
        StartupEmulatorWaitFxn();
    
        g_app_rtos_c7x_mmu_map_error = 0;
    
        appC7xClecInitForNonSecAccess();
    
        appMmuMap(FALSE);
        appMmuMap(TRUE);
    
        appCacheInit();
    }
    

  • Hi Pierre Hsieh,

    Are you still facing this issue? Let us know if we can otherwise close this ticket.

    Regards,

    Brijesh