Skip to content

Pr/1519 round2 #9

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

Merged
merged 19 commits into from
Oct 26, 2021
Merged

Pr/1519 round2 #9

merged 19 commits into from
Oct 26, 2021

Conversation

fpistm
Copy link

@fpistm fpistm commented Oct 26, 2021

Hi @beserge
I've fixed the pin definitions after useless pins removal as several indexes were misaligned.
PYn value is the index in the digitalPin array.

I've also add a reference in the README.md without link as currently it does not exists.

As a final step, please, think to squash all commits in one.

beserge and others added 19 commits October 14, 2021 16:46
Ive mostly filled out the .h and .cpp files
Still need to update some of the defines

The .ld file is copied verbatim from the seed
The .c file still needs to be modified
Simply use the name in boards.txt

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
As those pins are not defined in the analog array,
this save space.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@stephenhensley stephenhensley merged commit 63f46da into electro-smith:master Oct 26, 2021
@fpistm fpistm deleted the pr/1519-2 branch October 26, 2021 15:29
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