sam3x: spi: missing macro

[everedero]

Add missing SPI_CSR_BITS macro for sam3x variants

[nandojve]

Hi @everedero ,

You need to follow the rules of the repo. See latest commits for guidance.
This should have a PR on zephyr side to pick latest changes.

[everedero]

@nandojve I tried opening an issue on Zephyr side, but I am not sure I have the right SHA.
https://github.com/zephyrproject-rtos/zephyr/pull/86299/commits
This whole process is still unclear to me, do you have a precise commit I could you point me to for reference? Thanks!

[nandojve]

@nandojve I tried opening an issue on Zephyr side, but I am not sure I have the right SHA. https://github.com/zephyrproject-rtos/zephyr/pull/86299/commits This whole process is still unclear to me, do you have a precise commit I could you point me to for reference? Thanks!

I added the links in the repo docs that point to correct place that mention how to do it, see https://docs.zephyrproject.org/latest/develop/modules.html#submitting-changes-to-modules

[everedero]

Yeah I did read that and it is still unclear to me, the doc mentions "the pull request number" to be copied in the revision field but I only found my own commit SHA1, I could not find a PR SHA1, and the CI got angry because it is a SHA1 outside of hal_atmel repository. So, I don’t know if I interpreted the doc correctly, but I opened a PR zephyrproject-rtos/zephyr#86299

[nandojve]

Yeah I did read that and it is still unclear to me, the doc mentions "the pull request number" to be copied in the revision field but I only found my own commit SHA1, I could not find a PR SHA1, and the CI got angry because it is a SHA1 outside of hal_atmel repository. So, I don’t know if I interpreted the doc correctly, but I opened a PR zephyrproject-rtos/zephyr#86299

No worries. pdgendt already gave the answer to you : )
You need to update the PR as he mentioned.

BTW, you still need to update

hal_atmel/asf/sam/include/sam3x/README

Lines 41 to 45 in da76744

	Patch Lst:
	* Fix TC registers macros
	* Fix RSTC registers macros
	* Fix PWM registers macros
	* Fix SSC registers macros

[nandojve]

Hi @everedero ,

I saw that the update on hal_atmel/asf/sam/include/sam3x/README still missing.

[everedero]

Thanks, I forgot to push it 😅