Skip to content

PSOC6 port setup improvements#205

Merged
jaenrig-ifx merged 4 commits intoports-psoc6-mainfrom
updated-psoc6-readme
Apr 7, 2025
Merged

PSOC6 port setup improvements#205
jaenrig-ifx merged 4 commits intoports-psoc6-mainfrom
updated-psoc6-readme

Conversation

@jaenrig-ifx
Copy link
Copy Markdown
Member

  • Added notes for installation on macOS.
  • Fixed dev-setup.sh script for non-default MTB installation path.

@jaenrig-ifx jaenrig-ifx requested a review from NikhitaR-IFX April 1, 2025 12:28
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2025

Code size report:


Copy link
Copy Markdown
Member

@NikhitaR-IFX NikhitaR-IFX left a comment

Choose a reason for hiding this comment

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

I guess this addresses the user discussion in discord no?

@jaenrig-ifx
Copy link
Copy Markdown
Member Author

I guess this addresses the user discussion in discord no?

Correct!

@NikhitaR-IFX
Copy link
Copy Markdown
Member

I guess this addresses the user discussion in discord no?

Correct!

Perfect!

@jkorte-dev
Copy link
Copy Markdown

I tested the build on MacOS with branch updated-psoc6-readme following the instructions in the Readme. (softlink ~/ModusToolbox is not needed any more)

The PATh was adjusted with export PATH="/opt/homebrew/opt/make/libexec/gnubin:$PATH"
and everything compiled just fine, but I haven’t tried the make „program“ target yet.

In my opinion the remarks for building on MacOS are comprehensive but one sentence leaded me into a trap when I tried my first build:

„Install ModusToolbox. Minimum version required is 3.0.“

When you install the ModusToolbox with ModusToolboxSetup you can only install version 3.4 and I thought it would be OK because it is higher than 3.0. But 3.4 has a different compiler version.

Therefore I would leave out the word „Minimum“ and add a link to the download of ModusToolbox 3.4 for MacOS. (https://softwaretools.infineon.com/tools/com.ifx.tb.tool.modustoolbox)

When I downgraded from ModusToolbox 3.4 to ModusToolbox 3.4
This line in tools/psoc6/dev-setup.sh grabbed both versions (3.0 and 3.4)
echo ${mtb_path}/tools_*

I am not sure how to handle this (grab fist, last or restrict to tools_3.0 for now)
As soon as possible I will try target „program“ (I used ModusToolbox Programmer)

@jaenrig-ifx
Copy link
Copy Markdown
Member Author

I tested the build on MacOS with branch updated-psoc6-readme following the instructions in the Readme. (softlink ~/ModusToolbox is not needed any more)

The PATh was adjusted with export PATH="/opt/homebrew/opt/make/libexec/gnubin:$PATH" and everything compiled just fine, but I haven’t tried the make „program“ target yet.

In my opinion the remarks for building on MacOS are comprehensive but one sentence leaded me into a trap when I tried my first build:

„Install ModusToolbox. Minimum version required is 3.0.“

When you install the ModusToolbox with ModusToolboxSetup you can only install version 3.4 and I thought it would be OK because it is higher than 3.0. But 3.4 has a different compiler version.

Therefore I would leave out the word „Minimum“ and add a link to the download of ModusToolbox 3.4 for MacOS. (https://softwaretools.infineon.com/tools/com.ifx.tb.tool.modustoolbox)

When I downgraded from ModusToolbox 3.4 to ModusToolbox 3.4 This line in tools/psoc6/dev-setup.sh grabbed both versions (3.0 and 3.4) echo ${mtb_path}/tools_*

I am not sure how to handle this (grab fist, last or restrict to tools_3.0 for now) As soon as possible I will try target „program“ (I used ModusToolbox Programmer)

Thanks for checking again @jkorte-dev 😺
You are right. The specification of the the only validated version being 3.0 is missing. Sorry. I will add that.
In the future, we have to evaluate and enable the compatibility with new 3.x versions.

Regarding ${mtb_path}/tool_* ... I see that can lead to problems if you have different versions of the MTB installed. So maybe we should fix it to 3.0 until we can support other versions and filter the compatible ones. Only once we have checked the later versions.

@jkorte-dev
Copy link
Copy Markdown

I tested target "program" now and it worked like a charm :-)

Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
@jaenrig-ifx jaenrig-ifx force-pushed the updated-psoc6-readme branch from 9fa4c3f to ae3d0b3 Compare April 7, 2025 10:06
@jaenrig-ifx
Copy link
Copy Markdown
Member Author

I tested target "program" now and it worked like a charm :-)

Perfect. Glad to hear 👏

Thanks a lot for reviewing!

The README.md is now updated the script fixing the tools to 3.0. If you have multiple versions of the ModusToolbox it shouldn´t be a problem now.

@jaenrig-ifx jaenrig-ifx merged commit e99508b into ports-psoc6-main Apr 7, 2025
23 of 29 checks passed
@jaenrig-ifx jaenrig-ifx deleted the updated-psoc6-readme branch April 7, 2025 10:36
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.

3 participants