search

profdc9 / Apple2Card

Apple II Peripheral Card that Interfaces to a ATMEGA328P for SD card storage
67 stars 11 forks source link

PCB vs schematics: pull-up resistors #5

Open ThorstenBr opened 1 year ago

ThorstenBr commented 1 year ago

The PCB wasn't updated when the pull-up resistors were changed from 4k7 to 10k, and from 470 to 4k7 (see commit a848aa25b4dfe19334df68c5f3d26fe0f7ec0768). The current PCB still has the 4k7 / 470 Ohm values. Not a big deal, but a bit confusing. I noticed this since I changed the silk screen of my PCBs to include the component values.

You should update the PCB in KiCad with the latest schematics. Log:

Change R3 value from 4k7 to 10k.
Change R18 value from 4k7 to 10k.
Change R2 value from 4k7 to 10k.
Change R21 value from 4k7 to 10k.
Change R22 value from 4k7 to 10k.
Change R28 value from 470R to 4k7.
Change R7 value from 4k7 to 10k.
Change R27 value from 470R to 4k7.
Change R23 value from 4k7 to 10k.
profdc9 commented 1 year ago

I looked in the Kicad schematic file (.sch) and it looks up to date to the printed PDF file (.pdf). Are you look at the latest commit for both?

Thanks,

Dan

On Wed, Oct 5, 2022 at 1:58 PM ThorstenB @.***> wrote:

The PCB wasn't updated when the pull-up resistors were changed from 4k7 to 10k, and from 470 to 4k7 (see commit a848aa2 https://github.com/profdc9/Apple2Card/commit/a848aa25b4dfe19334df68c5f3d26fe0f7ec0768). The current PCB still has the 4k7 / 470 Ohm values. Not a big deal, but a bit confusing. I noticed this since I changed the silk screen of my PCBs to include the component values.

You should update the PCB in KiCad with the latest schematics. Log:

Change R3 value from 4k7 to 10k. Change R18 value from 4k7 to 10k. Change R2 value from 4k7 to 10k. Change R21 value from 4k7 to 10k. Change R22 value from 4k7 to 10k. Change R28 value from 470R to 4k7. Change R7 value from 4k7 to 10k. Change R27 value from 470R to 4k7. Change R23 value from 4k7 to 10k.

— Reply to this email directly, view it on GitHub https://github.com/profdc9/Apple2Card/issues/5, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFIJZBFC77UV7CMM3N3L23WBXFWRANCNFSM6AAAAAAQ52SMBM . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ThorstenBr commented 1 year ago

I created a merge request. You could simply merge this (just the resistor values in the PCB design are updated/changed). Or you can do the update manually (and drop my request). It's in the KiCad PCB design itself (not the PDF): "Apple2Card.kicad_pcb". If you open the PCB editor to look at the PCB, it was still showing the old resistor values there.