TonBERRY/tonberry-pico
3
0
Fork 0
Code Issues 12 Pull Requests 3 Actions Packages Projects 1 Releases Wiki Activity

schematic: Fix I2S_SD pullup
All checks were successful
Build RPi Pico firmware image / Build-Firmware (push) Successful in 3m28s
Details
Check code formatting / Check-C-Format (push) Successful in 8s
Details
Check code formatting / Check-Python-Flake8 (push) Successful in 9s
Details
Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s
Details
Run unit tests on host / Run-Unit-Tests (push) Successful in 8s
Details

Browse Source 
Sparkfun I2S audio board already has MAX98357A SD pin pullup for mono
mode, so external pullup is redundant and can actually cause issues.
Remove it.
This commit is contained in:
Matthias Blankertz
2025-07-22 22:07:44 +02:00
parent 2796dbcf16
commit 34f9a44cdb
1 changed files with 24 additions and 175 deletions
Download Patch File Download Diff File Expand all files Collapse all files

199
hardware/tonberry-pico/tonberry-pico.kicad_sch
View File

@@ -2347,15 +2347,15 @@
)
(uuid "1a911333-582d-4aef-a546-acfaa15e92ce")
)
(text "I2S_SD:\nLeft/2 + Right/2\nmode"
(text "Sparkfun board has pullup\nfor mono mode by default"
(exclude_from_sim no)
(at 59.182 68.58 0)
(at 65.024 91.186 0)
(effects
(font
(size 1.27 1.27)
)
)
(uuid "442b0a7f-27a4-45d5-ba3f-392a624afa5c")
(uuid "a299ef28-05a8-4b90-8b36-1e4fa5063a43")
)
(junction
(at 193.04 101.6)
@@ -2375,6 +2375,10 @@
(color 0 0 0 0)
(uuid "e452439a-8818-4be8-bcc4-c26f3276ba13")
)
(no_connect
(at 50.8 92.71)
(uuid "10706d1d-1bc6-44cc-835d-b8fa3a3b6bd0")
)
(no_connect
(at 170.18 139.7)
(uuid "772cb13b-a39d-4540-a627-d741ae566dfc")
@@ -2633,16 +2637,6 @@
)
(uuid "61d55cac-25ff-4b94-850d-39c3ad6f0e65")
)
(wire
(pts
(xy 31.75 92.71) (xy 49.53 92.71)
)
(stroke
(width 0)
(type default)
)
(uuid "638a190f-f331-40ae-8ee9-d088d216e176")
)
(wire
(pts
(xy 157.48 59.69) (xy 162.56 59.69)
@@ -2943,6 +2937,16 @@
)
(uuid "b893aba8-4828-4d16-b439-4c8bc498044d")
)
(wire
(pts
(xy 31.75 92.71) (xy 50.8 92.71)
)
(stroke
(width 0)
(type default)
)
(uuid "bbffc780-5b57-45bf-b3fa-3b8cba94a267")
)
(wire
(pts
(xy 121.92 85.09) (xy 104.14 85.09)
@@ -3073,16 +3077,6 @@
)
(uuid "e5e39417-bae8-449a-bdc3-ccde84eb8a5d")
)
(wire
(pts
(xy 49.53 76.2) (xy 49.53 92.71)
)
(stroke
(width 0)
(type default)
)
(uuid "e6a50daf-f91a-47f2-b4b7-a9f8864cafea")
)
(wire
(pts
(xy 99.06 62.23) (xy 99.06 72.39)
@@ -3115,7 +3109,7 @@
)
(wire
(pts
(xy 43.18 100.33) (xy 43.18 95.25)
(xy 43.18 95.25) (xy 43.18 101.6)
)
(stroke
(width 0)
@@ -3173,16 +3167,6 @@
)
(uuid "fc2056d1-3684-4dd8-b246-53d83b7d7512")
)
(wire
(pts
(xy 49.53 64.77) (xy 49.53 68.58)
)
(stroke
(width 0)
(type default)
)
(uuid "ff79dc83-d0c0-4e54-a4a0-d73c49bb4af7")
)
(label "SD_SCK"
(at 83.82 67.31 0)
(effects
@@ -3403,141 +3387,6 @@
)
(uuid "ef716abf-d5c9-485a-8d9b-cba5d9b64623")
)
(symbol
(lib_id "Device:R")
(at 49.53 72.39 0)
(unit 1)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(dnp no)
(uuid "0dba1186-6524-4282-90a1-e43af2371113")
(property "Reference" "R1"
(at 52.07 71.1199 0)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Value" "634k"
(at 52.07 73.6599 0)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Footprint" ""
(at 47.752 72.39 90)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(property "Datasheet" "~"
(at 49.53 72.39 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(property "Description" "Resistor"
(at 49.53 72.39 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(pin "2"
(uuid "08fd4621-268b-4331-aea1-885698dd7098")
)
(pin "1"
(uuid "dfa35725-41ed-43ce-a906-abde0d386cdb")
)
(instances
(project "tonberry-pico"
(path "/7226ca0f-138a-46b4-89f3-dc32dfb1f9f7"
(reference "R1")
(unit 1)
)
)
)
)
(symbol
(lib_id "power:+3V3")
(at 49.53 64.77 0)
(unit 1)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(dnp no)
(fields_autoplaced yes)
(uuid "155dd1f6-f675-468b-b443-9581eb4514a9")
(property "Reference" "#PWR07"
(at 49.53 68.58 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(property "Value" "+3V3"
(at 49.53 59.69 0)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Footprint" ""
(at 49.53 64.77 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(property "Datasheet" ""
(at 49.53 64.77 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(property "Description" "Power symbol creates a global label with name \"+3V3\""
(at 49.53 64.77 0)
(effects
(font
(size 1.27 1.27)
)
(hide yes)
)
)
(pin "1"
(uuid "aa7566f9-5551-4f75-a9a2-e6ecee097c88")
)
(instances
(project "tonberry-pico"
(path "/7226ca0f-138a-46b4-89f3-dc32dfb1f9f7"
(reference "#PWR07")
(unit 1)
)
)
)
)
(symbol
(lib_id "power:+5V")
(at 45.72 64.77 0)
@@ -4486,7 +4335,7 @@
)
(symbol
(lib_id "power:GND")
(at 43.18 100.33 0)
(at 43.18 101.6 0)
(unit 1)
(exclude_from_sim no)
(in_bom yes)
@@ -4495,7 +4344,7 @@
(fields_autoplaced yes)
(uuid "90f7f757-dce2-492f-8d74-03b43473ca02")
(property "Reference" "#PWR05"
(at 43.18 106.68 0)
(at 43.18 107.95 0)
(effects
(font
(size 1.27 1.27)
@@ -4504,7 +4353,7 @@
)
)
(property "Value" "GND"
(at 43.18 105.41 0)
(at 43.18 106.68 0)
(effects
(font
(size 1.27 1.27)
@@ -4512,7 +4361,7 @@
)
)
(property "Footprint" ""
(at 43.18 100.33 0)
(at 43.18 101.6 0)
(effects
(font
(size 1.27 1.27)
@@ -4521,7 +4370,7 @@
)
)
(property "Datasheet" ""
(at 43.18 100.33 0)
(at 43.18 101.6 0)
(effects
(font
(size 1.27 1.27)
@@ -4530,7 +4379,7 @@
)
)
(property "Description" "Power symbol creates a global label with name \"GND\" , ground"
(at 43.18 100.33 0)
(at 43.18 101.6 0)
(effects
(font
(size 1.27 1.27)