[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5541: arch:xtensa: modify svcall to swint

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

xiaoxiang781216 commented on pull request #5541:
URL: https://github.com/apache/incubator-nuttx/pull/5541#issuecomment-1046514473


   > @zhuyanlinzyl we still have a file at `xtensa/src/common/svcall.h` should this one be renamed as well?
   > 
   > BTW, other architectures (like RISC-V and ceva) also have an `svcall.h` file. I think that was a copy/paste mistake from ARM. Not sure about ceva but for RISC-V I think it should be renamed to `ecall.h`. cc. @xiaoxiang781216
   
   Here is the full list:
   ···
   ./arch/risc-v/src/common/svcall.h
   ./arch/ceva/src/common/svcall.h
   ./arch/xtensa/src/common/svcall.h
   ./arch/arm/src/armv7-m/svcall.h
   ./arch/arm/src/armv7-a/svcall.h
   ./arch/arm/src/armv7-r/svcall.h
   ./arch/arm/src/armv8-m/svcall.h
   ./arch/arm/src/armv6-m/svcall.h
   ···
   syscall is a general concept, arch can map to different instruction(.e.g. trap, syscall, ecall, swint...).


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

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