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

Scripted pin gen #87

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Scripted pin gen #87

merged 1 commit into from
Sep 13, 2023

Conversation

NikhitaR-IFX
Copy link
Member

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description
At this point, the script make-pins.py is triggered and running successfully as part of "make" command. There are couple of things that require refactoring especially in Makefile but this can be addressed as I finish with DESMAKERS-3536. Consider this as first-level PR to be informed on changes. In nutshell, the script will automatically:

  • Extract pins for cpu and make intermediate files.
  • Generate pins.h, pins_<BOARD_NAME>.c and pins_qstr.h which are all required micropython objects for pins.

@github-actions
Copy link

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40

@@ -42,7 +42,7 @@ MTB_TYPE=COMBINED
# configurations. If TARGET is manually edited, ensure TARGET_<BSP>.mtb with a
# valid URL exists in the application, run 'make getlibs' to fetch BSP contents
# and update or regenerate launch configurations for your IDE.
TARGET=APP_CY8CPROTO-062-4343W
TARGET=APP_CY8CPROTO-063-BLE
Copy link
Member

Choose a reason for hiding this comment

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

Don´t change this so far, as for the ci multi-board-support this is not yet set active when programming etc. So it will break all on-target-test of the previous board.
Soon, it won´t matter much this value :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep! I last tested it with new board and the changes remained I guess while pushing. I will revert.

@NikhitaR-IFX NikhitaR-IFX changed the base branch from make_pins_feature to multi-board-support September 12, 2023 07:54
@NikhitaR-IFX NikhitaR-IFX changed the base branch from multi-board-support to make_pins_feature September 12, 2023 07:56
Signed-off-by: NikhitaR-IFX <Nikhita.Rajasekhar@infineon.com>
@NikhitaR-IFX NikhitaR-IFX merged commit c639e32 into make_pins_feature Sep 13, 2023
15 of 17 checks passed
@NikhitaR-IFX NikhitaR-IFX deleted the scripted-pin-gen branch September 14, 2023 05:51
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.

2 participants