Missing CO_new() in canopen_app_process ? about canopenstm32

[Pascal8749]

I think you might be right. With the current code I get problems with a "Reset Communication" command. I just tried your suggestion and replaced "canopen_app_resetCommunication()" with "canopen_app_init()". As a result the "Reset Communication" command works now. I also think that this is the most logical way. Personally, I would expect a "Reset Communication" command to reset the communication. Sounds silly, but deleting the instance and recreating it with the given parameters would meet that expectation exactly.

[HamedJafarzadeh]

I will check that and update you here . Reset Comm should reset the communication. Maybe I missed something...

[midiwidi]

Hi Hamed. Have you had a chance to verify the issue?

[HamedJafarzadeh]

Hi @midiwidi,

Yes, I had a look today and you are right. The way that CO_RESET_COMM handled is wrong. I will mark this as a bug and fix it in the next release.

[midiwidi]

Thanks for checking Hamed.

[a-leibold]

Compared with the main_black.c in the CANopenNode example it seems there is no CO_Delet() necessary