PPAGE handling in FreeRTOS port for HCS12X banked memory model about mcuoneclipse HOT 6 CLOSED

Hi Grzegorz,
thank you for your feedback. I have do admit that I'm not really using S12X any more, but I should still have a board somewhere. And I have used FreeRTOS with S12X only in a small project which was not banked (I believe), so it very well could be something I have missed. From your description it sounds that the Processor Expert code dealing with the tick interrupt should be different, and not sure what I have to change.
Could you provide the pice of source code you have changed so I can follow up on this? Maybe best with clear indication what you have changed where?

Thanks,
Erich

from mcuoneclipse.

Hi Erich,
I attach 2 modified and 2 original files, both generated by PE. Here's what happens in my understanding:

• TickCntr1_Interrupt occurs and program goes to ISR handler, PPAGE register stays intact
• ISR handler calls TickCntr1_OnInterrupt() with CALL instruction and PPAGE register may change if there are multiple flash pages used and this function is on different one than function when interrupt happened.
• portSAVE_CONTEXT macro is called and possibly incorrect PPAGE register is pushed on stack
• at the end of function portRESTORE_CONTEXT macro is called and possibly incorrect PPAGE value is pulled from stack and copied to register which results in jumping to random place in code

Solution would be to move portSAVE_CONTEXT macro to primary ISR handler - before function call(attached files), or move everything to ISR handler (and then remove code that fixes stack with LEAS instructions).

S12X_issue42.zip

BR
Grzegorz Sobczyk

from mcuoneclipse.

Hi Grzegorz,
thanks for the extra details. I wanted to investigate this last weekend, but did not find the time. I hope because you have found a solution it is not that urgent?

Thanks,
Erich

from mcuoneclipse.

from mcuoneclipse.

You can disable the overwrite if you disable code generation for a component, see https://mcuoneclipse.com/2012/03/23/disable-my-code-generation/

from mcuoneclipse.

closing issue: for now disabling code generation should do it.

from mcuoneclipse.