TMC2130Stepper icon indicating copy to clipboard operation
TMC2130Stepper copied to clipboard

TMC2130: SDI and SDO swapped in Fritzing PCB view

Open eyer1951 opened this issue 5 years ago • 10 comments

In the PCB definition, the Fritzing part for TMC2130 has SDO and SDI swapped. I had some boards manufactured, unfortunately.

In this example i just connected DIR to SDO to illustrate the issue. bad2130pcb.zip

eyer1951 avatar Nov 05 '18 18:11 eyer1951

@soswow

teemuatlut avatar Nov 05 '18 18:11 teemuatlut

Oy! CS and SCK are also swapped.

eyer1951 avatar Nov 05 '18 21:11 eyer1951

I'm not that familiar with Fritzing but everything looks good to me. Something I'm missing?

teemuatlut avatar Nov 05 '18 22:11 teemuatlut

Thanks for checking. Can you open the sketch file I provided? If so, you will quickly see that the PCB design does not match the TMS2130 icon shown in the breadboard (part layout). Pin locations for PCB wiring don't match the part. The location of the SDO connection (should be in the middle of the row) in the PCB layout is shown (wrongly) as 2nd from the end of the part (location of SDI). There is a simplified parts editor in Fritzing, but i could not fix this. Also note, CS and SCK are swapped. It appears @soswow was the author of the Fritzing parts file.

eyer1951 avatar Nov 05 '18 22:11 eyer1951

Now I see the issue! I think the order of named pins need to be changed in the Connecters tab.

EDIT: After reordering image

teemuatlut avatar Nov 05 '18 22:11 teemuatlut

I tried renaming them in the Connectors tab, but that didn't work. I couldn't get Schematic, Breadboard, and PCB to synchronize correctly. '

eyer1951 avatar Nov 05 '18 22:11 eyer1951

How did you accomplish the reordering? Exactly. Will you publish the corrected Fritzing part? Thanks.

eyer1951 avatar Nov 06 '18 03:11 eyer1951

I did figure out how to associate each connections with the "icon" (holes on the part). I had not seen how to enable the menu with the checkmarks and "select graphic." So, for my own use, i now have a part that works. It would be nice if the "official" one were to be corrected and re-published.

eyer1951 avatar Nov 06 '18 06:11 eyer1951

I'll look into that today. Sorry for swapping (

On Tue, Nov 6, 2018, 17:49 eyer1951 [email protected] wrote:

I did figure out how to associate each connections with the "icon" (holes on the part). I had not seen how to enable the menu with the checkmarks and "select graphic." So, for my own use, i now have a part that works. It would be nice if the "official" one were to be corrected and re-published.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/teemuatlut/TMC2130Stepper/issues/56#issuecomment-436148654, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGmIW4xmKiQUuBpNEJIi_c0pGlNaZqNks5usTDmgaJpZM4YO_RH .

soswow avatar Nov 06 '18 06:11 soswow

raised a PR

soswow avatar Nov 06 '18 08:11 soswow