Skip to content

fix: sim7080g module - #29

Open
rodneyosodo wants to merge 1 commit into
mainfrom
sim7080g
Open

rodneyosodo wants to merge 1 commit into
mainfrom
sim7080g

Conversation

@rodneyosodo

Copy link
Copy Markdown
Collaborator

What type of PR is this?

What does this do?

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

Did you document any new/modified features?

Notes

2026-09-11_13-04_1 2026-09-11_13-04

Signed-off-by: Rodney Osodo <socials@rodneyosodo.com>

@JeffMboya JeffMboya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JeffMboya

Copy link
Copy Markdown
Contributor

Run Update PCB from Schematic; C18 and L3 exist in the schematic but are missing from the layout.

@JeffMboya

Copy link
Copy Markdown
Contributor

Populate C18 with 0R instead of DNP, because a depopulated series part breaks the GNSS feed entirely.

@JeffMboya

Copy link
Copy Markdown
Contributor

Move SIM7080G_DCD off J2 pin 5, which lands on the base board's shared SCL line.

@JeffMboya

Copy link
Copy Markdown
Contributor

Set the DNP attribute and clear the JLCPCB part number on C7, C9 and C10 before fabrication.

@JeffMboya

Copy link
Copy Markdown
Contributor

Restore the ground stitching vias, since this PR drops GND vias from sixty-one to seven.

@JeffMboya

Copy link
Copy Markdown
Contributor

Pour ground on an inner layer, so the RF feeds reference ground rather than the In1 +3V3 plane.

@JeffMboya

Copy link
Copy Markdown
Contributor

Keep RF_ANT and GNSS_ANT on the top layer, since both currently drop to In2.Cu through vias.

@JeffMboya

Copy link
Copy Markdown
Contributor

Recalculate the antenna trace width for 50R, because 0.5mm over this 0.1mm prepreg is roughly 26R.

@JeffMboya

Copy link
Copy Markdown
Contributor

Move C15, C16 and C17 beside U2's VBAT pins; they currently sit ten to fifteen millimetres away.

@JeffMboya

Copy link
Copy Markdown
Contributor

Add netclass patterns for RF_50R and VBAT_HIGH, which are defined but currently match no nets.

@JeffMboya

Copy link
Copy Markdown
Contributor

Restore the silk_over_copper and starved_thermal DRC severities rather than setting both to ignore.

@JeffMboya

Copy link
Copy Markdown
Contributor

Give U2's USB_BOOT a defined level, because JP1 ships open and leaves the pin floating.

@JeffMboya

Copy link
Copy Markdown
Contributor

Bias or close JP2 and JP3, since U2's UART1_RTS input is otherwise left undefined.

@JeffMboya

Copy link
Copy Markdown
Contributor

Add a fuse or current limit to the GNSS bias, since 0805 R3 cannot survive one watt.

@JeffMboya

Copy link
Copy Markdown
Contributor

Place D8 beside U3, because ESD clamping degrades over the present fifteen millimetre run to the connector.

@JeffMboya

Copy link
Copy Markdown
Contributor

Add a bulk capacitor on SIM_VDD next to U3, where C6's 0.1uF is the only decoupling.

@JeffMboya

Copy link
Copy Markdown
Contributor

Add test points on GND, +3V3 and VBAT, as TP1 through TP5 cover only +1V8 and USB.

@JeffMboya

Copy link
Copy Markdown
Contributor

Regenerate the jlcpcb gerbers, BOM and CPL, which still describe the July board rather than this revision.

@JeffMboya

Copy link
Copy Markdown
Contributor

Re-export schematic.pdf, which still shows C7, C9 and C10 as 22pF and D3 instead of D8.

@JeffMboya

Copy link
Copy Markdown
Contributor

Bump the title block revision and date, since v0.5.0 no longer describes this substantially changed schematic.

@JeffMboya

Copy link
Copy Markdown
Contributor

Strip C48677.kicad_sym to ESDA6V1W5, since it also ships ESP32-C6, E07-400M10S and four unrelated connector symbols.

@JeffMboya

Copy link
Copy Markdown
Contributor

Extend the C48677 courtyard to enclose its pads, which sit entirely outside the current 0.65mm outline.

@JeffMboya

Copy link
Copy Markdown
Contributor

Repoint the C48677 3D model at the committed C48677.step, because the EASYEDA2KICAD variable will not resolve.

@JeffMboya

Copy link
Copy Markdown
Contributor

Confirm the KiCad 10 format bump is intended, since every other board here still targets 9.0.

@JeffMboya

Copy link
Copy Markdown
Contributor

Give U1 a JLCPCB field and align D8 and U3, which use 'LCSC Part' instead.

@JeffMboya

Copy link
Copy Markdown
Contributor

Fix the two hole clearance errors on the +1V8 via at 112.0, 91.7, currently 0.00mm against 0.15mm.

@JeffMboya

Copy link
Copy Markdown
Contributor

Move the GND teardrop out of J4's keepout area, which DRC reports as an items-not-allowed error.

@JeffMboya

Copy link
Copy Markdown
Contributor

Restoring silk_over_copper and starved_thermal reveals seven and eleven hidden violations, so fix those rather than ignoring.

@JeffMboya

Copy link
Copy Markdown
Contributor

Commit an fp-lib-table so the modules library resolves, because DRC cannot currently check U2, U3 or D8.

@JeffMboya

Copy link
Copy Markdown
Contributor

Assign real pin types in the C48677 symbol, since its Unspecified pins add five ERC pin_to_pin warnings.

@JeffMboya

Copy link
Copy Markdown
Contributor

Add PWR_FLAGs or correct the VDD_EXT and SIM_VDD pin types, which ERC errors as undriven rails.

@JeffMboya JeffMboya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants