kywwilson11 commented on PR #15718: URL: https://github.com/apache/nuttx/pull/15718#issuecomment-2689004184
> Thank you @kywwilson11, amazing work! :-) > > * Please use signed commit (`git commit -s`) and we are ready to go :-) > > * This is initial commit, are there other things to do? > > * Do you plan to create / update documentation in future? > > * Do you know or can provide application that could verify cache mechanism works as expected (I have MB1814B NUCLEO-H503RB devkit)? In future we could add this feature to our `apps/ostest` or just as standalone application first :-) I will go ahead and sign the commit with git commit -s. Thanks for the suggestion. I wrote "Initial Commit" because this is the first commit for the icache. This peripheral is relatively new and as of now is only available on the STM32H5, STM32L5, STM32U3, and STM32U5. There was no development for the icache (or dcache) for any of the previously mentioned microcontroller architectures. Regarding documentation, I did add some documentation for the STM32H5 architecture recently as well as the Nucleo-H563ZI. Once this is pulled in I can update the documentation for the icache. Lastly, I don't know of an application nor will I be able to provide one that verifies the functionality of this ICACHE peripheral. As I mentioned above, my testing was limited. The image I included was from vscode which showed the status of the ICACHE peripheral's register set, which is defined by an svd file provided by the STM website. I included that just to show that the HMONR (hit monitor) and MMONR (miss monitor) registers were being updated by hardware. This alone shows that the icache is functioning, however this does not mean every feature is developed properly for Nuttx. In particular, there may be a better way to utilize the CRR registers for using cacheing external memories. Given that I have demonstrated basic functionality of the ICACHE for the STM32H5, and it did not take too much effort, I thought this would be a good start. I welcome others to take this initial pull request as a starting off point and make improvements where they see fit. -- 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
