search

tinyvision-ai-inc / UPduino-v3.0

UPduino 3.0: new 4 layer layout, various other improvements
MIT License
324 stars 39 forks source link

No current limiting resistors for the three LEDS? #28

Open zapta opened 7 months ago

zapta commented 7 months ago

The three LEDs don't have current limiting resistors. When a LED is on, do the FPGA outputs that drive them operate within spec?

image
vr2045 commented 7 months ago

The current source is inside the FPGA and can be adjusted and is capable of driving the LED's directly. Please see the RTL blinky to see how you can do this.

On Thu, Feb 22, 2024 at 9:50 AM Zapta @.***> wrote:

The three LEDs don't have current limiting resistors. When a LED is on, do the FPGA outputs that drive them operate within spec? image.png (view on web) https://github.com/tinyvision-ai-inc/UPduino-v3.0/assets/2184769/2c6878c6-3643-4fe1-a6ac-cf4d38a82855

— Reply to this email directly, view it on GitHub https://github.com/tinyvision-ai-inc/UPduino-v3.0/issues/28, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHDV6C4TRV5PPYDJBC55ULLYU6AOPAVCNFSM6AAAAABDVMQL56VHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DSNJZHE2TQNI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

zapta commented 7 months ago

Thanks. Found it. I guess I can abstract it as a module with three binary inputs to turn the leds on/off. Will look into it.

I suggest to mention this special led requirement in the PCF file and in the schematic since it's not obvious.