You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/03 15:58:59 UTC

[GitHub] [incubator-nuttx] minabeoki opened a new pull request #5678: imx6: fix fpu compile option

minabeoki opened a new pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678


   ## Summary
   
   - imx6 fpu is vfpv3
   - imx6 fpu has 32 registers (CONFIG_ARM_HAVE_FPU_D32=y)
   - With this fix, "getprime" now succeeds by gcc-10.3.1.
   
   In the following page, there is "VFPv3" in the block diagram.
   
   https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-6-processors/i-mx-6quad-processors-high-performance-3d-graphics-hd-video-arm-cortex-a9-core:i.MX6Q
   
   The following statements can be found in the data sheet.
   
   > NEON register file with 32x64-bit general-purpose registers
   
   ## Impact
   
   - imx6
   
   ## Testing
   
   - getprime on sabre-6quad:knsh (QEMU)
   
   ```
   % arm-none-eabi-gcc -v
   gcc version 10.3.1 20210824 (release) (GNU Arm Embedded Toolchain 10.3-2021.10)
   % qemu-system-arm -semihosting -M sabrelite -m 1024 -smp 4 -nographic -kernel ./nuttx
   ABDGHIJKNOPQ
   
   NuttShell (NSH) NuttX-10.2.0
   nsh> /system/bin/hello
   Hello, World!!
   nsh> /system/bin/getprime
   Set thread priority to 10
   Set thread policy to SCHED_RR
   Start thread #0
   thread #0 started, looking for primes < 10000, doing 10 run(s)
   thread #0 finished, found 1230 primes, last one was 9973
   Done
   /system/bin/getprime took 1310 msec
   nsh>
   ```


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



[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678#issuecomment-1058246071


   Please participate in a review and make sure that all needed flags are selected compiler n Kconfig for imx6


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



[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678#issuecomment-1058245203


   Yes. I think we should follow-up with a more generic change.


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



[GitHub] [incubator-nuttx] minabeoki closed pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
minabeoki closed pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678


   


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



[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678#issuecomment-1058222933


   I think it goes in conflict with https://github.com/apache/incubator-nuttx/pull/5662


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



[GitHub] [incubator-nuttx] minabeoki commented on pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
minabeoki commented on pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678#issuecomment-1058238827


   @pkarashchenko 
   Hmm... Should I close this PR once?


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



[GitHub] [incubator-nuttx] minabeoki commented on pull request #5678: imx6: fix fpu compile option

Posted by GitBox <gi...@apache.org>.
minabeoki commented on pull request #5678:
URL: https://github.com/apache/incubator-nuttx/pull/5678#issuecomment-1058249955


   ok, I check #5662


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