[GitHub] [incubator-nuttx] Ouss4 edited a comment on pull request #2014: Add the optional interrupt stack to the Xtensa architecture

[GitBox <gi...@apache.org>]

Ouss4 edited a comment on pull request #2014:
URL: https://github.com/apache/incubator-nuttx/pull/2014#issuecomment-711139881


   > However, I also noticed that arm_intstack_base() for imx6 and cxd56xx were incorrect.
   Currently they return "bottom" of the interrupt stack for the current CPU.
   So I will need to fix them later.
   
   Yes, this is what I was reffering to.  arm_intstack_base is returning the bottom of the stack while it was used assuming it returns the top of the stack.
   
   This function is used for debugging only in arm_assert (arch_dumpstate for some architectures) and arm_checkstack.  Its usage in xxx_checkstack is correct though.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org