arduino / docs-content

Arduino documentation (docs.arduino.cc)
https://docs.arduino.cc/
Creative Commons Attribution Share Alike 4.0 International
170 stars 386 forks source link

BUS bugs inside the Arduino Nano Every Fritzing part #356

Open BastianFie opened 2 years ago

BastianFie commented 2 years ago

I just downloaded the fritzing part for the Arduino Nano Every from here (it said version 6 from mar 26 2020) build by @kengdahl.

This version / file has some BUS bugs inside it (at least from my understanding from the Arduino module)

Steps to check that: 1.) Download the fritzing part, rename the part from fzpz to zip and expand that 2.) Edit the *.fzp inside the expanded ZIP file with an text editor (e.g. notepad++) 3.) Scroll to the bottom to the buses section until you see the following:

  <bus id="+5v">
   <nodeMember connectorId="connector34"/>
   <nodeMember connectorId="connector56"/>
  </bus>

This means, there should be a +5V BUS for connector34 and connector56, however if you check the rest from the file, there is no connector 34 and no 56. If you search inside the *.fzp for "+5V" you will find the following section:

  <connector name="+5V" id="connector58" type="male">
   <description>+5V</description>
   <views>
    <breadboardView>
     <p svgId="connector58pin" layer="breadboard"/>
    </breadboardView>
    <schematicView>
     <p svgId="connector58pin" layer="schematic"/>
    </schematicView>
    <pcbView>
     <p svgId="connector58pin" layer="copper0"/>
     <p svgId="connector58pin" layer="copper1"/>
    </pcbView>
   </views>
  </connector>

So +5V pin is connector 58 and not 34 or 56. However if you now check again the BUS section and search there for connector58 you can find the following:

  <bus id="d11/mosi">
   <nodeMember connectorId="connector29"/>
   <nodeMember connectorId="connector58"/>
  </bus>

So the D11/MOSI and +5V share the same connection?!? I wouldn´t expect that is that really true? Based on the reason that the +5V BUS definition inside the file is wrong, I think the other section is also wrong. I for example would expect a BUS for GND which didn´t exist. The reset BUS by the way also didn't seamed to be correct (connector59 is not a Reset one its A7)

Could somebody please pick up the file and fix it? Looks like I couldn´t build a merge request here as that's mostly some documentation part.

I would expect the following BUS by the way:

 <buses>
  <bus id="reset">
   <nodeMember connectorId="connector18"/>
   <nodeMember connectorId="connector57"/>
  </bus>
  <bus id="GND">
   <nodeMember connectorId="connector19"/>
   <nodeMember connectorId="connector32"/>
  </bus>
 </buses>

Note: Its also reported in the fritzing forum here.

vanepp commented 2 years ago

There is a corrected version of this part as well as a new part for the SMD version available here:

https://forum.fritzing.org/t/bugs-in-arduino-nano-every-fritzing-part-v6/16589/2

feel free to use them if you like.

marqdevx commented 2 years ago

@kengdahl Could you please have a look? 😄

kengdahl commented 1 year ago

@marqdevx I'm afraid that I don't have the knowledge on how to fix this.

vanepp commented 1 year ago

I posted a fixed version in the Fritzing forum entry 2 posts above (along with a SMD version of the part.) You are welcome to use them both if you like. You should be able to just copy the .fzpz files from the Fritzing post in to your web site after checking they do what you want.

marqdevx commented 1 year ago

@vanepp I'll check locally if everything is alright, and commit the changes giving you the authorship.

Thanks for the contribution 😄 I'll keep you updated.