Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Ltt respin #97

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Ltt respin #97

wants to merge 22 commits into from

Conversation

ManavToor
Copy link
Contributor

Respinning LTT to increase range and remove rfd900x

Designed around cc1200 transciever. I basically copied the entire evaluation board cause rf is hard and their filters and matching networks are already made and tested. Did add an external PA which meant breaking away from cc1200 design (bad) so we instead implemented a switch for switching between receiving and transmitting to keep the cc1200 network untouched. Second switch is for switching between antenna 1 and 2. There is a second PA that we also want to test, hence all the DNM's.

@ManavToor ManavToor requested review from Rio6, BluCodeGH, Lucifersan, JasonBrave and AshTheEngiqueer and removed request for BluCodeGH December 4, 2024 14:09
@ManavToor ManavToor requested a review from Pdada1 December 4, 2024 14:30
@Rio6
Copy link
Contributor

Rio6 commented Dec 4, 2024

  1. looks like the resistor here is for 3v3, not 5v?
    圖片
  2. datasheet has two 22uF caps here
    圖片
    圖片
  3. would it make sense to run pic (and the LEDs) at 5V to isolate it from the transceiver more?
    圖片
  4. I think this should be 2.2p, and be part of CC1200 block instead of the balun block (see point 8)
    圖片
  5. where is this 200pF number from? Seems on the small side
    圖片
  6. can R8 and L1 share the same footprint?
    圖片
  7. I don't think R6 is doing anything here
    圖片
  8. We still want TRX_SW to be part of the network. Maybe something like this? (value of L12, L13 TBD, but should have high impedance at 900MHz)
    圖片

@Rio6
Copy link
Contributor

Rio6 commented Dec 10, 2024

  1. is not fixed
  2. I think L1 and R8 should share the same footprint
  3. one of these 2.2uF should be bigger, like 1nF since the original reference design only had 1 of them. Also 15nH is only 84ohms which is a bit low, we want it >> 50ohm
    圖片

@Rio6
Copy link
Contributor

Rio6 commented Dec 14, 2024

I just realized RF_IN on PA is missing M1 and M2 from datrasheet
圖片 圖片

Either remove C42 and add a DC blocking cap here
圖片

Or increase C54 (say to 1nF) so the total capacitance between C42 and C54 is still around 2.2pF

@ManavToor
Copy link
Contributor Author

Updates:

@Rio6
Copy link
Contributor

Rio6 commented Jan 1, 2025

  1. nit: if we're doing mounting hole with via may as well make it solid
    image
  2. nit: they can probably share the same via
    image
  3. I'd reduce the via clearance on the ground plane as well so it is not cut off here and some other places
    image
  4. nit: maybe remove the two 5V vias on top as well. Or better, move C44, C45 the the bottom-left of the strip image
  5. lastly you should fix DRC, check jlc for their size constraints as most errors seems to be from that. Update the net for footprints if the stock one isn't labelled right. And for the clerance ones you can probably just change the DRC setting.

@ManavToor
Copy link
Contributor Author

  1. Done
  2. nah
  3. Added a small fill zone connection between the holes
  4. replaced them with just one via directly in between the two caps
  5. lowered board constraints to slightly above jlc minimums

Also added a guard ring

@Rio6
Copy link
Contributor

Rio6 commented Jan 1, 2025

For the guard ring, I'd do it like this, excluding the sma connectors and including the transceiver, making it a rectangle. You might have to move the switch or SMA around a bit to make room. We want to put a shield over it, and the SMA wouldn't fit in there. Also rectangle shield is easier to maker. You should add 2 through holes on it for soldering the shield as well (or clips, if we can find one on digikey).

圖片

@AshTheEngiqueer
Copy link
Contributor

Sorry I'm late to the party

Schematic:
1.
image
is this 1A hold or trip current? Also do we want to update CAN connector from datamate to gecko?
2.
image
I don't think your footprint on the inductor is correct?

Layout:
1.
image
is there a reason this trace width changes? is it some RF thing? I'm pretty sure you can run the whole trace with the thicker thickness
2.
image
No via fence on the oscillator?
3. in general there's a lot of daisychaining, but especially with RF stuff routing is giga important? you two know more than I do but maybe worth a look to see if there's anything you can change.
4. Also in the email from JLC there's an extra charge because of the size of some of the vias/holes - either this is something we pay or you can look at increasing the size of vias/holes to minimum 0.2/0.45mm

@ManavToor
Copy link
Contributor Author

  1. Hold, trip @ 1.5A. Completely slipped my mind to switch to gecko's, nice catch

  2. It matches whats in the datasheet, not sure what you mean

  3. clearance errors, the footprint is super tiny and the pads are so close every time I run the trace to the pad DRC looses its mind

  4. Added some vias there

  5. You have to daisychain for high speed

  6. Updated the vias to 0.45mm diameter

@AshTheEngiqueer
Copy link
Contributor

This should be merged if it's been ordered

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants