zigastegu / LimeSDR-USB

USB 3.0 version of the LimeSDR board
30 stars 6 forks source link

LED improvements #21

Closed ynezz closed 4 years ago

ynezz commented 6 years ago

Hi,

thanks a lot for a great case and a this project! Just a small nitpicks:

  1. I had to drill bigger holes(+1mm in diameter) for the 4 LEDs, otherwise I wouldn't be able to put it all together with the screws. Bottom of the LEDs are wider by about 1mm in diameter so the 3mm hole isn't big enough so the front panel doesn't align with the aluminium case.

  2. I miss the description of the LEDs on the panel. Maybe you can add the some info about the LEDs in the free space above USB and PWR connectors? Or just move the clock IN/OUT connectors more to the left (above USB/PWR connectors) and add description of the LEDs above the LEDs?

zigastegu commented 6 years ago

Thank you for taking the time to comment here!

  1. Leds don't fit in plastic holders without modification. Please check instructions for assembly? Maybe something is not well described. I could not source "original" ones. No need to drill holes on PCB/rear panel.

  2. Hm... Yes I guess something could be done for predefined ones

Any visual explanation for both comments would be very helpful to me.

ynezz commented 6 years ago

Please check instructions for assembly?

I've found this GitHub repository after I've finished the assembly, in the guide found in this reposiotry it's pretty clear, that you've modified LEDs so they fit in the holes. I've simply completed the assembly from the photos on your eBay listing as I couldn't find any mention about this assembly instructions, maybe I've just missed it :-)

Maybe something is not well described

It's OK, maybe just put small paper with URL to the assembly instructions to the package?

Yes I guess something could be done for predefined ones

Just some description of the LEDs, so it's clear which one is LED1, LED4, FX3/USB and Power. Currently there's no text describing the purpose of the LEDs and one has to open image with description to find out which one is LED1 for example.

I understand, that LED1 and LED4 are user configurable LEDs, but this could be fixed with either custom front panel or custom sticker once needed.

zigastegu commented 6 years ago

Assembly instructions: Instructions.md Link to Github on listing for panel not directly as in listing for KIT.

zigastegu commented 6 years ago

Rear panel view: R top:

R bottom:

L top:

L bottom:

Observations: When streaming R bottom is red and L bottom is flashing just green