Skip to content
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.

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

Scripts pin #182

Merged
merged 2 commits into from
Aug 27, 2024
Merged

Scripts pin #182

merged 2 commits into from
Aug 27, 2024

Conversation

IFX-Anusha
Copy link

  • Combined make-pins.py and make-pins-csv.py into a single file for generation of the pin details from pin packages

  • Replaced CSV file generation with table

  • Evaluated the possibility of defining the pin package file path in the mpconfigboard.mk but then as there are releases in the file path, updating that often will be a problem. So better that is part of the script itself.

Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

Nice 😃 Simpler now.
Only the thing we found out. It would be nice to find out the out of index objects there.
If that gets difficult, we can create a new bug ticket for that...

ports/psoc6/Makefile Outdated Show resolved Hide resolved
Copy link
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.

Looks all good!

Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
Copy link

@ramya-subramanyam ramya-subramanyam left a comment

Choose a reason for hiding this comment

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

Looks good!

@IFX-Anusha IFX-Anusha merged commit d55388d into ports-psoc6-main Aug 27, 2024
27 of 30 checks passed
@IFX-Anusha IFX-Anusha deleted the scripts_pin branch August 27, 2024 06:39
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