TDA4VM: EventP_ID_23 macro definition maybe not correct in EventP.h

TS L

Intellectual 911 points

Part Number: TDA4VM

Hi Expert.

in SDK8.1,

you can see following statement “EventP_freertos.c”

EventP_wait --> DebugP_assert(eventMask <= EventP_ID_23);

you can see following macro definition in “EventP.h”

#define EventP_ID_23 0x800000

you can see following statement in “Event_groups.c ”

#define eventEVENT_BITS_CONTROL_BYTES 0xff000000UL

configASSERT( ( uxBitsToWaitFor & eventEVENT_BITS_CONTROL_BYTES ) == 0 );

Question:

is "EventP_wait --> DebugP_assert(eventMask <= EventP_ID_23); " correct?

i think it should be "DebugP_assert(eventMask <= 0xFFFFFF); "

over 3 years ago

0 Parth Nagpal over 3 years ago

TI__Mastermind 42135 points

Hi,

I am looking into this. Meanwhile, can you please share, if you are facing any issues due to this?

Regards,
Parth

0 TS L over 3 years ago in reply to Parth Nagpal

Intellectual 911 points

Hi Parth.

DebugP_assert occur. which block program running.

because the waiting event is 0x00FFFFFF.

Then,i check the code and found that:the waiting event value limited to <=0x800000.

0 Parth Nagpal over 3 years ago in reply to TS L

TI__Mastermind 42135 points

Hi,

Referring to EventP.h

These event IDs are defined to simplify the specification of mask argument. But that should not restrict you from using other events.
I believe changing DebugP_assert(eventMask <= EventP_ID_23) to DebugP_assert(eventMask <= 0xFFFFFF) should not create any issue. Have you tried this? Are you facing any issue if you make the above modification?

Regards,
Parth

0 TS L over 3 years ago in reply to Parth Nagpal

Intellectual 911 points

Hi Parth

No issue if i make above modification.

+1 Parth Nagpal over 3 years ago in reply to TS L

TI__Mastermind 42135 points

Hi,

TS L said:
No issue if i make above modification.

If you are not facing any issues, then you can go ahead and use this modification. I don't see any concerns with that.

Regards,
Parth

Processors

Processors forum

TDA4VM: EventP_ID_23 macro definition maybe not correct in EventP.h