# @Seve https://tscircuit.com/

Channel: #designreview
Source: https://discord.com/channels/1233487248129921135/1555623225436938241
Started: 2026-10-02T16:52:19.686000+00:00
Last activity: 2026-10-17T07:42:59.517000+00:00

## Seve — 2026-10-02T18:59:04.744000+00:00

[attachment]
Attachment: CleanShot_2026-10-02_at_11.59.002x.png — https://community.tscircuit.com/media/1555655377461313576

## Seve — 2026-10-02T18:59:17.392000+00:00

non-standard USB-C footprint, also issues with it

## Seve — 2026-10-02T18:59:59.414000+00:00

each motor connection is a separate component (a single plated hole = single component?)
Attachment: CleanShot_2026-10-02_at_11.59.372x.png — https://community.tscircuit.com/media/1555655606357065788

## Seve — 2026-10-02T19:00:21.887000+00:00

all said that page is maybe the cleanest

## Seve — 2026-10-02T19:00:36.887000+00:00

but you can use schematicbox for that rather than 3 separate components

## Seve — 2026-10-02T19:01:19.624000+00:00

3S cell protection- i thought you were using single cell? Am I confused? What battery does this go with? (I shouldn't need to ask!!!)
Attachment: CleanShot_2026-10-02_at_12.00.522x.png — https://community.tscircuit.com/media/1555655942941708388

## Seve — 2026-10-02T19:02:56.374000+00:00

annoyed that this isn't near the battery connectors, also why does the battery connect in two places?
Attachment: CleanShot_2026-10-02_at_12.02.202x.png — https://community.tscircuit.com/media/1555656348467994664

## Mustafa7 — 2026-10-05T17:41:10.281000+00:00

https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#files
<@757706909351411845> review please!
The schematic-analysis reports a lot of false positives or wrong suggestions, am working on fixing them. The battery is "one 3S, 2200 mAh LiPo battery pack"  
```ONE 3S LiPo battery
   ├── Thick red/black wires → XT60 plug → J13
   └── Four thin wires → small JST-XH plug → J15```
I have improved the schematic to follow the references.

## Seve — 2026-10-05T17:42:57.441000+00:00

i have to keep telling people this, but generally you _never mix vertical and side connectors on the same board_, you have both vertical and side connectors, just go with the side connectors (those are usually more stable and available), triple check that everything plugs in!!!

## Seve — 2026-10-05T17:44:01.779000+00:00

i'm looking at your banana plug connector and i'm not sure it's right, triple check these things, if they don't match it'll be a waste of money!

## Seve — 2026-10-05T17:44:09.546000+00:00

i think your motor connector is wrong as well

## Seve — 2026-10-05T17:48:05.451000+00:00

yea the schematic still needs a lot of work to be readable but it's a good one to chase to get all the issues. One of the big issues is just really long "schematic subcircuits" like this one. Ideally the "path to ground" or "path to power" is all visible from the pin
Attachment: CleanShot_2026-10-05_at_10.47.282x.png — https://community.tscircuit.com/media/1556724676322004992

## Mustafa7 — 2026-10-06T14:44:23.002000+00:00

<@757706909351411845> does this sound good?
Attachment: Screenshot_2026-10-06_at_8.14.06_PM.png — https://community.tscircuit.com/media/1557040832631013416

## Seve — 2026-10-06T16:07:25.714000+00:00

no absolutely not

## Seve — 2026-10-06T16:07:32.226000+00:00

the entire thing i'm avoiding is adapter wires

## Seve — 2026-10-06T16:07:55.615000+00:00

getting annoyed about this, you should now do a full assembly

## Seve — 2026-10-06T16:09:16.211000+00:00

i.e. you need to create the entire system in 3d in tscircuit assembly components with <assembly.cable /> etc. because it seems like you cannot visualize the system

## Seve — 2026-10-06T16:09:45.311000+00:00

you may need to add support for the particular cables you're using. For the model, you will want to probably create this via e.g. Opus and build123d

## Seve — 2026-10-06T16:25:57.520000+00:00

https://cleanshot.com/share/3tDGd23y

## Mustafa7 — 2026-10-07T17:38:25.951000+00:00

https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#files
Attachment: Screenshot_2026-10-07_at_10.59.31_PM.png — https://community.tscircuit.com/media/1557447019491106937
Attachment: Screenshot_2026-10-07_at_10.59.39_PM.png — https://community.tscircuit.com/media/1557447020208328865
Attachment: Screenshot_2026-10-07_at_10.59.48_PM.png — https://community.tscircuit.com/media/1557447020694995006
Attachment: Screenshot_2026-10-07_at_10.59.59_PM.png — https://community.tscircuit.com/media/1557447021043130439
Attachment: Screenshot_2026-10-07_at_11.00.15_PM.png — https://community.tscircuit.com/media/1557447021458235422

## Mustafa7 — 2026-10-07T17:38:32.161000+00:00

<@757706909351411845>

## Mustafa7 — 2026-10-07T17:39:24.161000+00:00

cannot see assembly on 3d viewer, this is from local server. Do we have to create a new package integrating the assembly?

## Seve — 2026-10-07T17:45:59.333000+00:00

is tscircuit up to date

## Seve — 2026-10-07T17:47:01.324000+00:00

btw let me know when you're asking for review, i know you sent a link there but based on the pin headers and empty space on the board i'm guessing you're not ready for review yet

## Mustafa7 — 2026-10-07T17:49:10.060000+00:00

yes

## Mustafa7 — 2026-10-07T17:49:48.054000+00:00

it was 70x65 before but after adding sockets it became 80x95 thats why there is some empty space left

## Mustafa7 — 2026-10-07T17:51:15.819000+00:00

those sockets are big

## Mustafa7 — 2026-10-07T18:03:02.925000+00:00

Ill see until where i can decrease the size

## Seve — 2026-10-07T18:04:31.594000+00:00

i would investigate that bug

## Seve — 2026-10-07T18:04:41.403000+00:00

don't worry about decreasing the size yet

## Seve — 2026-10-07T18:04:49.967000+00:00

you need to get rid of all these pin headers and get your board ready

## Seve — 2026-10-07T18:05:12.795000+00:00

why do you have a million pin headers

## Mustafa7 — 2026-10-07T18:23:54.195000+00:00

<@757706909351411845> some of the headers were for these, we have to put these on pcb itself right?
Attachment: Screenshot_2026-10-07_at_11.53.19_PM.png — https://community.tscircuit.com/media/1557458464383434854

## Seve — 2026-10-07T18:30:02.972000+00:00

oof

## Seve — 2026-10-07T18:30:15.360000+00:00

yea you need to include all that in your assembly and source it all

## Seve — 2026-10-07T18:30:43.087000+00:00

if you don't need it don't put it on

## Mustafa7 — 2026-10-07T19:28:49.699000+00:00

Removed pin headers and shinked it down by 5mm 
https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#files

## Mustafa7 — 2026-10-07T19:55:08.201000+00:00

Let me know if any issues, headed to bed. I'll fix in morning

## Seve — 2026-10-07T19:55:52.569000+00:00

your banana plugs are facing the wrong way i think

## Seve — 2026-10-07T19:56:33.789000+00:00

you need a power switch

## Seve — 2026-10-07T19:56:43.929000+00:00

yea try to complete a full assembly

## Mustafa7 — 2026-10-08T04:51:05.913000+00:00

Wdym? You only said to do motor assembly right?

## Seve — 2026-10-08T04:51:36.023000+00:00

just make sure everything needed to run the motor is connected, if there's something needed to run the motor, make sure it's added

## Mustafa7 — 2026-10-08T17:04:58.238000+00:00

<@757706909351411845> The board includes USB-PD charging and battery protection, each with a separate battery-temperature input. Should we keep two external temperature probes, or would you prefer a simpler temperature-sensing arrangement?

## Seve — 2026-10-08T17:21:25.468000+00:00

i'm not sure

## Seve — 2026-10-08T17:21:30.621000+00:00

what's the reference design do?

## Mustafa7 — 2026-10-08T17:21:43.495000+00:00

reference design doesnt have battery

## Seve — 2026-10-08T17:21:51.757000+00:00

why do we need a battery?

## Seve — 2026-10-08T17:22:08.518000+00:00

i really just want to control a motor with USB PD and USB DATA

## Mustafa7 — 2026-10-08T17:22:12.856000+00:00

You recommended me to add battery right

## Seve — 2026-10-08T17:22:14.331000+00:00

i think there is a lot of stuff added

## Mustafa7 — 2026-10-08T17:22:26.224000+00:00

usb pd charges the battery

## Seve — 2026-10-08T17:22:26.676000+00:00

i'm not designing the product

## Seve — 2026-10-08T17:22:34.988000+00:00

i said that a battery can help with current draw

## Seve — 2026-10-08T17:22:44.293000+00:00

you're the product designer but i think we probably don't want the battery

## Seve — 2026-10-08T17:22:48.431000+00:00

if usb-pd can power it

## Seve — 2026-10-08T17:23:00.268000+00:00

especially because the board is huge right now

## Mustafa7 — 2026-10-08T17:23:03.624000+00:00

ok ill remove battery

## Seve — 2026-10-08T17:23:28.790000+00:00

ask ai, understand the product, redesign as needed, you have to make this product SELL which means it's gotta be good

## Seve — 2026-10-08T17:23:46.920000+00:00

i don't think people are going to want to buy a battery and we wouldn't package a battery i think

## Seve — 2026-10-08T17:24:07.169000+00:00

don't just do random stuff that i'm saying, i'm here to help not design the product

## Seve — 2026-10-08T17:24:15.235000+00:00

you should be able to answer WHY for every design decision

## Mustafa7 — 2026-10-08T20:20:17.401000+00:00

https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#files
Please review when you get time, am probably asking for too many reviews!
Build is running still not completed yet. THis includes the motor adapter etc assembly.

## Rishabh — 2026-10-08T20:29:42.663000+00:00

The power traces should be on the outer layers
Attachment: image.png — https://community.tscircuit.com/media/1557852513020153856

## Rishabh — 2026-10-08T20:31:05.606000+00:00

I see via stitching happening on your board, some of them look uncessary
Attachment: image.png — https://community.tscircuit.com/media/1557852860845391922

## Rishabh — 2026-10-08T20:37:39.775000+00:00

Add knockout silkscreen to these, confusing which one is data and which is pwr
Attachment: image.png — https://community.tscircuit.com/media/1557854514432704562

## Rishabh — 2026-10-08T20:38:22.237000+00:00

I think get a review from AI first, I made ai review this and it gives a lot of placement issue and the width of traces

## Mustafa7 — 2026-10-13T06:58:24.615000+00:00

<@757706909351411845> <@809856037376491570> https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#pcb
review please

## Rishabh — 2026-10-13T10:37:15.634000+00:00

Your schematic is messed up, can't read the labels on any of the components. Everything is overlapping
Attachment: image.png — https://community.tscircuit.com/media/1559515357100642334

## Mustafa7 — 2026-10-13T13:17:39.224000+00:00

The one you shared is showing the ports right?
Attachment: image0.jpg — https://community.tscircuit.com/media/1559555721257943110

## Mustafa7 — 2026-10-13T13:19:12.652000+00:00

I think that is a circuit to svg ports drawing issue

## Mustafa7 — 2026-10-14T06:27:23.117000+00:00

https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#pcb
<@757706909351411845> can you take a look

## Seve — 2026-10-14T16:44:32.176000+00:00

this is looking really good, can you make sure the PD is configurable

## Mustafa7 — 2026-10-15T05:36:01.782000+00:00

<@757706909351411845> this is configured for 9v

## Seve — 2026-10-15T16:14:17.930000+00:00

yes, that's why I commented to say that you need to make it configurable

## Seve — 2026-10-15T16:14:50.596000+00:00

(is there any reason not to?)

## Mustafa7 — 2026-10-15T17:53:01.492000+00:00

STUSB4500 the one am using supports upto 20v, so 20v should be the last range right? i think configurable till 20v is good right?

## Mustafa7 — 2026-10-15T18:03:02.789000+00:00

you want it to be configurable between 5, 9, 15, 20v?

## Mustafa7 — 2026-10-16T17:06:51.496000+00:00

https://tscircuit.com/MustafaMulla29/rp2040-bldc-motor-controller-new#pcb
<@757706909351411845> For 5v codex says this: 
```Some of the USB’s 5 V is lost in the cable and board, so the motor-driver chip may receive less than the 4.5 V it needs.
If that happens, the motor may fail to start or stop when working harder.```

## Seve — 2026-10-16T17:21:53.188000+00:00

the motor driver should be receiving usb pd voltage no?

## Seve — 2026-10-16T17:22:08.614000+00:00

msybe join discord voice so we can talk about it

## Seve — 2026-10-16T17:22:26.646000+00:00

a motor driver might not even work at 5v

## Seve — 2026-10-16T17:22:44.814000+00:00

do you have a multi-color LED on this board? I don't see it

## Seve — 2026-10-16T17:22:47.796000+00:00

those are super important

## Mustafa7 — 2026-10-17T06:04:20.891000+00:00

Yes thats correct

## Mustafa7 — 2026-10-17T06:04:33.245000+00:00

Ill add those now

## Seve — 2026-10-17T06:06:06.735000+00:00

> Yes thats correct

fwiw this is another case of people wasting my time with "AI said this". Don't tell me what AI says, ask me a question or tell me I'm wrong etc. but idgaf what AI says. I have no idea why you posted the following quote, what was I supposed to say? If you're requesting PD voltage then this quote makes no sense, so i have no idea why you posted it or why you responded that you were getting pd voltage. 

> Some of the USB’s 5 V is lost in the cable and board, so the motor-driver chip may receive less than the 4.5 V it needs.
> If that happens, the motor may fail to start or stop when working harder.

## Seve — 2026-10-17T06:06:18.998000+00:00

"AI said this" = "Fake engineering"!!!

## Seve — 2026-10-17T06:06:58.879000+00:00

This is to say I have no idea why you posted that quote, now i'm sitting here trying to think of why you posted that quote if you know that it's requesting pd voltage

## Mustafa7 — 2026-10-17T06:07:58.196000+00:00

I meant that is that true? that 5v wont work reliably?

## Seve — 2026-10-17T06:08:28.566000+00:00

of course you can't power the bldc on the 5v line

## Seve — 2026-10-17T06:08:33.714000+00:00

that's the entire point of the pd line

## Seve — 2026-10-17T06:08:46.994000+00:00

again i have no idea what you're talking about, why would the motor ever receive 5v

## Mustafa7 — 2026-10-17T06:09:26.740000+00:00

ok i never knew these, i dont have much knowledge thats why i asked

## Seve — 2026-10-17T06:09:48.806000+00:00

yes but mustafa you literally can't not know that, that is so critical

## Seve — 2026-10-17T06:10:02.940000+00:00

do research etc. there are mistakes but you really have to understand that stuff really deeply

## Seve — 2026-10-17T06:10:48.750000+00:00

you should watch a couple hours of videos on the design of BLDC controllers for example

## Seve — 2026-10-17T06:21:35.952000+00:00

your USB PD port has this VBUS_RAW, which is the negotiated, higher voltage. It's negotiated by your USB PD chip (the CH*) one
Attachment: CleanShot_2026-10-16_at_23.20.522x.png — https://community.tscircuit.com/media/1560900569353297930

## Seve — 2026-10-17T06:39:24.430000+00:00

https://www.youtube.com/shorts/71tAOaD3kfI

## Mustafa7 — 2026-10-17T07:42:59.517000+00:00

Yes watched some videos of usb pd