xiaoxiang781216 commented on PR #16463:
URL: https://github.com/apache/nuttx/pull/16463#issuecomment-2926863480

   > _Note: Please adhere to [Contributing 
Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md)._
   > ## Summary
   > 
   >     1. Added support for STM32H723.
   > 
   >     2. ARM: Added SP register initialization during startup.
   > 
   >     3. Added FDCAN3 support.
   > 
   >     4. Fixed UART compilation issue (removed macro definition restrictions 
related to the "up_restoreusartint undefined" error).
   > 
   >     5. Adjusted interrupt initialization timing: interrupts should be 
configured as soon as the chip starts up to prevent abnormal initialization 
after the OS starts, which could otherwise result in the BootLoader’s interrupt 
handler being triggered.
   > 
   >     6. Updated CMake to add ELF file suffix and fixed a bug in CMake (to 
be investigated).
   > 
   >     7. Added DMA align manager component.
   > 
   >     8. Optimized mmcsd code logic.
   > 
   >     9. Removed FAT layer DMA configuration; DMA logic should reside in the 
device driver layer.
   > 
   > 
   
   Don't add the unrelated change in one patch, if you want your patch get 
merged finally. For DMA change, I agree the comment from @PetervdPerk-NXP and 
please split DMA change into the small patch like:
   
   1. Support the unaligned address in sdmmc common driver
   2. stm32 the specific change in sdmmc driver
   3. Remove the unaligned address support from fatfs layer


-- 
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-unsubscr...@nuttx.apache.org

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

Reply via email to