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

ev11l78a: configure DAC peripheral #62478

Closed
wants to merge 2 commits into from

Conversation

Flameeyes
Copy link
Contributor

@Flameeyes Flameeyes commented Sep 10, 2023

ev11l78a: configure DAC peripheral

This configures the DAC in the same way as it is configured for the
arduino_zero and other similar boards.

It also includes the board in the DAC tests.

Signed-off-by: Diego Elio Pettenò [email protected]


Stack created with Sapling. Best reviewed with ReviewStack.

Copy link
Member

@martinjaeger martinjaeger left a comment

Choose a reason for hiding this comment

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

The first two commits don't seem to be related or required for the DAC. If that's tje case, please remove them from this PR.

Otherwise looks good.

@nandojve
Copy link
Member

You need rebase : )

@nandojve
Copy link
Member

Ping

The ATSAMD20E16 used in this board only has 8KiB SRAM, so the defaults
are not really suitable for anything close to useful, particularly the
sink sample that most closely resembles the original demo for the board.

This should also allow more tests not to be skipped for this board as
otherwise they overflow RAM usage.

Signed-off-by: Diego Elio Pettenò <[email protected]>
This configures the DAC in the same way as it is configured for the
arduino_zero and other similar boards.

It also includes the board in the DAC tests.

Signed-off-by: Diego Elio Pettenò <[email protected]>
@Flameeyes
Copy link
Contributor Author

Rebased, This still include the commit from #61797 because of how Sapling handles the stack — there's a ReviewStack UI that wraps around the pull request, otherwise feel free to wait for the review until the dependent change is merged.

@nandojve
Copy link
Member

nandojve commented Jan 9, 2024

Hi @Flameeyes ,

The #61797 got in.

Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Mar 10, 2024
@github-actions github-actions bot closed this Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: DAC Digital-to-Analog Converter platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants