Skip to content

modules: hal_infineon: fix build failure for Infineon CYT4BFC 320 BGA pin package #100524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nashif merged 1 commit into from
Dec 8, 2025
Merged

modules: hal_infineon: fix build failure for Infineon CYT4BFC 320 BGA pin package #100524

nashif merged 1 commit into from
Dec 8, 2025
+1 −1

Conversation

@shreyasifx
Copy link
Contributor

@shreyasifx shreyasifx commented Dec 4, 2025

Fixes #100523 which reports about error in Cmakelists.txt file within directory zephyr/modules/hal_infineon/mtb-hal-cat1/CMakeLists.txt.

Details

Removed the extra inclusion of 'c' file extension which considers the correct source file 320 BGA Pin Package and considers the correct source file cyhal_xmc7200_320_bga.c.

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Hello @Shreyas-G-Khanapur, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@zephyrbot zephyrbot added platform: Infineon Infineon Technologies AG size: XS A PR changing only a single line of code labels Dec 4, 2025
parthitce
parthitce requested changes Dec 4, 2025
View reviewed changes
Copy link
Member

@parthitce parthitce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your efforts. Adjust the commit message and Fixes tag in the commit body.

@shreyasifx shreyasifx force-pushed the fix_zephyr_issue_#100523 branch from 15e115c to 33750c2 Compare December 5, 2025 05:25
@shreyasifx shreyasifx changed the title fix for zephyr issue #100523: Build failure for Infineon CYT4BFC 320 BGA pin package modules: hal_infineon: fix build failure for Infineon CYT4BFC 320 BGA pin package Dec 5, 2025
@shreyasifx
Copy link
Contributor Author

shreyasifx commented Dec 5, 2025

Thanks for your efforts. Adjust the commit message and Fixes tag in the commit body.

@parthitce - I have updated the commit message and added the fix tag. Please review

@sreeramIfx
Copy link
Contributor

sreeramIfx commented Dec 5, 2025

Compliance Checks are failing still. Please address

@shreyasifx shreyasifx force-pushed the fix_zephyr_issue_#100523 branch from 33750c2 to d373fdb Compare December 5, 2025 06:21
@shreyasifx shreyasifx requested a review from parthitce December 5, 2025 06:55
@parthitce
Copy link
Member

parthitce commented Dec 5, 2025 

Signed-off-by: Shreyas-G-Khanapur <[email protected]>

this will not work. Probably

Signed-off-by: Shreyas G Khanapur <[email protected]>

or

Signed-off-by: ShreyasG Khanapur <[email protected]>

may work

@shreyasifx shreyasifx force-pushed the fix_zephyr_issue_#100523 branch from d373fdb to 49dab61 Compare December 5, 2025 08:27
@shreyasifx
modules: hal_infineon: Fix build failure for CYT4BFC 320 BGA pin package
0f8c970 
The CMakeLists.txt file in the MTB HAL CAT1 module incorrectly appended
a '.c' extension to the source file reference for the 320 BGA pin package,
causing a build error.

Remove the extraneous '.c' extension to
correctly reference cyhal_xmc7200_320_bga.c.

Fixes zephyrproject-rtos#100523

Signed-off-by: Shreyas G Khanapur <[email protected]>
@shreyasifx shreyasifx force-pushed the fix_zephyr_issue_#100523 branch from 49dab61 to 0f8c970 Compare December 8, 2025 05:59
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@nashif nashif merged commit b41ea17 into zephyrproject-rtos:main Dec 8, 2025
26 checks passed
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Hi @shreyasifx!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

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

Reviewers

@parthitce parthitce parthitce approved these changes

@sreeramIfx sreeramIfx sreeramIfx approved these changes

@billwatersiii billwatersiii Awaiting requested review from billwatersiii

@jsbatch jsbatch Awaiting requested review from jsbatch

@mcatee-infineon mcatee-infineon Awaiting requested review from mcatee-infineon

@talih0 talih0 Awaiting requested review from talih0

Assignees

@sreeramIfx sreeramIfx

Labels

platform: Infineon Infineon Technologies AG size: XS A PR changing only a single line of code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Build failure for Infineon CYT4BFC 320 BGA pin package

5 participants

@shreyasifx @sreeramIfx @parthitce @nashif @zephyrbot