HelTecAutomation / CubeCell-Arduino

Heltec CubeCell Series (based on ASR6501, ASR6502 chip) Arduino support.
247 stars 138 forks source link

HTCC-AB01 pinout diagram can easily lead to errors and possible damage. #55

Closed lnlp closed 4 years ago

lnlp commented 4 years ago

In the HTCC-AB01 pinout diagram pin17/GPIO7 and pin16/GPIO6 look like they are standard pins on the side of the board (situated next to the GND pins). But in reality they are not.

This causes confusion and can easily lead to incorrect wiring which in worst case can damage the board and other components connected to it.

In the current version of the diagram it looks like the GPIO7 and GPIO6 pins are regular pins on the side of the board because that is how they are drawn in the diagram (while in reality they are not). When looking at a glance this gives incorrect information to the user. Only when looking very carefully one can see that GPIO7 and GPIO6 are actually not on the side of the board. This is therefore easily overlooked and error prone.

Why is this a problem: The pins are only labeled on the bottom side of the PCB but not on the top side. When using the HTCC-AB01 on a breadboard then the labels are not visible and one has to look at the pinout diagram and count the sequence number of a pin from the first or last pin in a row to know which pin is what because there are no pin labels. Looking at the pinout diagram one can easily overlook that GPIO7 and GPIO6 are NOT regular pins on the side because they are drawn like regular pins on the side. Therefore one can easily count bad sequence numbers and as a result wire the ATCC-AB01 board incorrectly. This is not only impractical but may also lead to damage. People are very visually oriented ('a picture can say more than a thousand words'). Even when someone already knows that GPIO7 and GPIO6 pins are not on the side, he/she may still be fooled by the diagram when counting pin sequence numbers.

Therefore please change the pinout diagram and draw the information for GPIO7 and GPIO6 at another area in the diagram, not next to and not in row with the side pins. It should be visibly clear, at a glance, that GPIO7 and GPIO6 are NOT regular pins on the side of the board.

wasn-eu commented 4 years ago

There are red lines connecting the label to the position of the pins. I dont think anyone would not see them

lnlp commented 4 years ago

Maybe you should read my explanation again. It clearly explains why the current layout easily leads to problems.

It should not be too much efforts to update the diagram (which should be simple if you have the source of the pdf document).

About 4.5% of the world population is color blind (men 16 times more than women). Many/most of them will not notice the red line at all. But even for people with perfect vision the red lines are not enough to correct the optical effect that pins GPIO7 and GPIO6 look like all the other pins on the side, because they are incorrectly drawn that way, which easily tricks people, even if they have noticed the red lines before.

mikenz commented 4 years ago

I agree with @lnlp on this one, the first time I went to use the board on a breadboard I started counting the pins off from the usb socket end and was off by one too.

I would suggest something like this: image

lnlp commented 4 years ago

I would suggest something like

That is exactly what I had in mind.

I usually print my pinout diagrams for easy use. When printing the original pinout diagram, the board and pin descriptions unfortunately are printed rather small (due to document layout). I have created a customized version that is easier to read when printed. It is sized for A4 paper and is attached below:

(In png format because the pdf's vector source is not available.)

Heltec CubeCell HTCC-AB01 Pinout Diagram

wasn-eu commented 4 years ago

fixed in https://github.com/HelTecAutomation/ASR650x-Arduino/commit/7e16adb74ae18b796e1c4a6151f23c55962b7b5f