sinara-hw / EEM_FMC_Carrier

HPC FMC carrier in EEM format
Other
4 stars 2 forks source link

Replace CLK_SEL pull down to 1k #44

Closed marmeladapk closed 10 months ago

marmeladapk commented 2 years ago

SY58029U has unusually strong internal pullups (25k), so 10k is not enough for a pull down.

Affected resistors: R80 ed: See below.

marmeladapk commented 2 years ago

Also CLK_SEL0 should be pulled up and CLK_SEL1 should be pulled down to select internal oscillator by default.

ed: Rework: Remove R80 (there is internal pull up) and replace R79 to 1k and connect it to ground.

marmeladapk commented 1 year ago

https://github.com/m-labs/artiq/pull/2134#pullrequestreview-1532226430

Different setups may need different default clock source after boot, so adding DIP switches seems reasonable solution.