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

Petal sm rev5 #42

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Petal sm rev5 #42

wants to merge 4 commits into from

Conversation

beserge
Copy link
Contributor

@beserge beserge commented Apr 27, 2022

Changes for petal_sm rev5.
Also: we currently have no examples for the petal SM. This should be rectified soon. Outside the scope of this PR probably though.

Test Procedure

Petal SM

Using this sketch:
petal_sm_test_sketch.zip

  • 6 pots and expression: add a bit of gain to the output whitenoise
  • footswitches: momentary bypass with either. (relay)
  • toggles: run through the colors on led 1, 3. Led 2 is off in the center.

Patch SM

Flashed the triplesaw example and confirmed it worked

Daisy Patch

Flashed the sequencer and vco and confirmed they both work.

Daisy Petal

Flashed the looper. Ring led1_r and fs_led4 blink for some reason, but the example works fine otherwise.
It does this on master so it's not a regression.
Flashed the multieffect. This worked fine.
Flashed the distortion. This worked fine.

Daisy Field

Flashed the stringvoice example and confirmed it worked,

Daisy Pod

Flashed the synthvoice and made sure it worked.

Audio is probably still broken, and may be newly broken on some of the boards
The petal sm pins are still on the rev4 ones from the other pr
I may have missed some changes / made some mistakes in going through these things
Flip the toggles around
Swap the led names to match the new ones from the schematic
Remove the extra relay
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.

1 participant