Skip to content

modules: hal: mchp sam: Add SAMA5D2 and SAM9X7 header files#49

Open
TonyHan11 wants to merge 4 commits intozephyrproject-rtos:masterfrom
TonyHan11:sama5d2
Open

modules: hal: mchp sam: Add SAMA5D2 and SAM9X7 header files#49
TonyHan11 wants to merge 4 commits intozephyrproject-rtos:masterfrom
TonyHan11:sama5d2

Conversation

@TonyHan11
Copy link
Contributor

changes in this PR includes:

  • add header files and ATDF files for sama5d2 SoCs
  • update CMakeLists.txt accordingly

The files are from the "atdf" and "include" folders in
"Microchip SAMA5D2 Series Device Support (1.14.264)".
Download Link:
https://packs.download.microchip.com/Microchip.SAMA5D2_DFP.1.14.264.atpack

Signed-off-by: Tony Han <tony.han@microchip.com>
'hal-microchip' updates CMakeLists.txt for the Microchip SAMA5D2
Microprocessors (MPUs).

Signed-off-by: Tony Han <tony.han@microchip.com>
The files are from the "atdf" and "include" folders in
"Microchip SAM9X7 Series Device Support (1.9.170)".
Download Link:
https://packs.download.microchip.com/Microchip.SAM9X7_DFP.1.9.170.atpack

Signed-off-by: Tony Han <tony.han@microchip.com>
'hal-microchip' updates CMakeLists.txt for the Microchip SAM9X7
Microprocessors (MPUs).

Signed-off-by: Tony Han <tony.han@microchip.com>
@TonyHan11 TonyHan11 changed the title modules: hal: mchp sam: Add SAMA5D2 header files modules: hal: mchp sam: Add SAMA5D2 and SAM9X7 header files Dec 10, 2025
@NhMchp NhMchp self-assigned this Jan 15, 2026
@NhMchp NhMchp requested review from AzharMCHP and scottwcpg January 15, 2026 04:35
@TonyHan11
Copy link
Contributor Author

ping @nandojve

@TonyHan11
Copy link
Contributor Author

Hi @NhMchp ,

Would you like to help merging the commits in this PR for solving the CI failures related with pic32cx_sg61_cult/pic32cx1025sg61128 in zephyrproject-rtos/zephyr#103557? Thank you very much.

@NhMchp
Copy link
Collaborator

NhMchp commented Feb 12, 2026

Hi @NhMchp ,

Would you like to help merging the commits in this PR for solving the CI failures related with pic32cx_sg61_cult/pic32cx1025sg61128 in zephyrproject-rtos/zephyr#103557? Thank you very much.

@TonyHan11 , Could you please share details of failures related to pic32cx_sg61_cult/pic32cx1025sg61128 ?

@TonyHan11
Copy link
Contributor Author

Hi @NhMchp ,
Would you like to help merging the commits in this PR for solving the CI failures related with pic32cx_sg61_cult/pic32cx1025sg61128 in zephyrproject-rtos/zephyr#103557? Thank you very much.

@TonyHan11 , Could you please share details of failures related to pic32cx_sg61_cult/pic32cx1025sg61128 ?

https://github.com/zephyrproject-rtos/zephyr/actions/runs/21930226291/job/63332687272?pr=103557#step:13:1498

In zephyrproject-rtos/zephyr#103557, the hal_microchip reversion is pull/49/head. The failure is caused by some commits in the master branch of hal_microchip which are related to pic32 is not included.

It's better to update the reversion to the hash value after the merge. It might be solved by using pull/49/merge too.

@nandojve
Copy link
Member

Hi @NhMchp ,
Would you like to help merging the commits in this PR for solving the CI failures related with pic32cx_sg61_cult/pic32cx1025sg61128 in zephyrproject-rtos/zephyr#103557? Thank you very much.

@TonyHan11 , Could you please share details of failures related to pic32cx_sg61_cult/pic32cx1025sg61128 ?

https://github.com/zephyrproject-rtos/zephyr/actions/runs/21930226291/job/63332687272?pr=103557#step:13:1498

In zephyrproject-rtos/zephyr#103557, the hal_microchip reversion is pull/49/head. The failure is caused by some commits in the master branch of hal_microchip which are related to pic32 is not included.

It's better to update the reversion to the hash value after the merge. It might be solved by using pull/49/merge too.

To avoid this situation we should merge the HAL content only when the PR in zephyr side is ready. This is the only way to keep HAL and Zephyr content sane due to the number of activity in parallel.

@NhMchp
Copy link
Collaborator

NhMchp commented Feb 12, 2026

Hi @NhMchp ,
Would you like to help merging the commits in this PR for solving the CI failures related with pic32cx_sg61_cult/pic32cx1025sg61128 in zephyrproject-rtos/zephyr#103557? Thank you very much.

@TonyHan11 , Could you please share details of failures related to pic32cx_sg61_cult/pic32cx1025sg61128 ?

https://github.com/zephyrproject-rtos/zephyr/actions/runs/21930226291/job/63332687272?pr=103557#step:13:1498
In zephyrproject-rtos/zephyr#103557, the hal_microchip reversion is pull/49/head. The failure is caused by some commits in the master branch of hal_microchip which are related to pic32 is not included.
It's better to update the reversion to the hash value after the merge. It might be solved by using pull/49/merge too.

To avoid this situation we should merge the HAL content only when the PR in zephyr side is ready. This is the only way to keep HAL and Zephyr content sane due to the number of activity in parallel.

I see the issue related to pic32cx is fixed in the latest of master branch. I think, it will be resolved if you could rebase the branch to latest of master.

@TonyHan11
Copy link
Contributor Author

I see the issue related to pic32cx is fixed in the latest of master branch. I think, it will be resolved if you could rebase the branch to latest of master.

Hi @NhMchp ,

Temporarily solved by using pull/49/merge in the PR. Thanks.

Do you know who would handle the hal_microchip merge action?

@nandojve
Copy link
Member

Hi @TonyHan11 ,

What should the merge sequence in zephyr and when the HAL content will be need to build and pass in the tests?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments