fixed some comments, re-worked pin labels in sch.

Fixed comments in System_Functions.ino which mapped Arduino pins to
display pins
Some labels on the schematic did not match the display pin they
connected to. Renamed those nets to make more sense.
Added a pdf of current schematic, not sure if this'll be sustainable.
Trying it out.
Defaulted display type to S7S in the firmware.

Author: Jim Lindblom

firmware/Serial 7-Segment Display/Serial_7_Segment_Display_Firmware/Serial_7_Segment_Display_Firmware.ino

@@ -35,7 +35,7 @@ SevSeg myDisplay; //Create an instance of the object
 //OpenSegment uses PNP and NPN transistors to drive larger displays
 #define S7S            1
 #define OPENSEGMENT    2
-#define DISPLAY_TYPE OPENSEGMENT
+#define DISPLAY_TYPE S7S
 
 //Global variables
 unsigned int analogValue6 = 0; //These are used in analog meter mode

firmware/Serial 7-Segment Display/Serial_7_Segment_Display_Firmware/System_Functions.ino

@@ -144,25 +144,25 @@ void setupDisplay()
   //This pinout is for the original Serial7Segment layout
   //-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 #if DISPLAY_TYPE == S7S
-  int digit1 = 16; //Analog 2 is 16
-  int digit2 = 17; //A3 is 17
-  int digit3 = 3;
-  int digit4 = 4;
+  int digit1 = 16; // DIG1 = A2/16 (PC2)
+  int digit2 = 17; // DIG2 = A3/17 (PC3)
+  int digit3 = 3;  // DIG3 = D3 (PD3)
+  int digit4 = 4;  // DIG4 = D4 (PD4)
 
   //Declare what pins are connected to the segments
-  int segA = 8; //Labeled B on schematic
-  int segB = 14; //Labeled C on schematic
-  int segC = 6; //Labeled A/COL on schematic
-  int segD = A1;
-  int segE = 23; //This is not a standard Arduino pin: Must add PB7 as digital pin 23 to pins_arduino.h
-  int segF = 7; 
-  int segG = 5; //Labeled G/APOS on schematic
-  int segDP= 22; //This is not a standard Arduino pin: Must add PB6 as digital pin 22 to pins_arduino.h
-
-  int digitColon = 2; //The digit is the top side of this LED
-  int segmentColon = 6; //The segment is the bottom side of this LED
-  int digitApostrophe = 9;
-  int segmentApostrophe = 7; //Labeled F on schematic (there is an incorrect label G/APOS)
+  int segA = 8;  // A = D8 (PB0)
+  int segB = 14; // B = A0 (PC0)
+  int segC = 6;  // C = D6 (PD6), shares a pin with colon cathode
+  int segD = A1; // D = A1 (PC1)
+  int segE = 23; // E = PB7 (not a standard Arduino pin: Must add PB7 as digital pin 23 to pins_arduino.h)
+  int segF = 7;  // F = D7 (PD6), shares a pin with apostrophe cathode
+  int segG = 5;  // G = D5 (PD5)
+  int segDP= 22; //DP = PB6 (not a standard Arduino pin: Must add PB6 as digital pin 22 to pins_arduino.h)
+
+  int digitColon = 2; // COL-A = D2 (PD2) (anode of colon)
+  int segmentColon = 6; // COL-C = D6 (PD6) (cathode of colon), shares a pin with C
+  int digitApostrophe = 9; // APOS-A = D9 (PB1) (anode of apostrophe)
+  int segmentApostrophe = 7; // APOS-C = D7 (PD7) (cathode of apostrophe), shares a pin with F
 
   int numberOfDigits = 4; //Do you have a 2 or 4 digit display?
 

hardware/Serial-7-Segment-Display.brd

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE eagle SYSTEM "eagle.dtd">
-<eagle version="6.1">
+<eagle version="6.4">
 <drawing>
 <settings>
 <setting alwaysvectorfont="no"/>
@@ -893,7 +893,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <class number="0" name="default" width="0" drill="0">
 </class>
 </classes>
-<designrules>
+<designrules name="default">
 <description language="en">&lt;b&gt;EAGLE Design Rules&lt;/b&gt;
 &lt;p&gt;
 The default Design Rules have been set to cover
@@ -1465,15 +1465,15 @@ design rules under a new name.</description>
 <wire x1="5.842" y1="4.826" x2="7.747" y2="4.826" width="0.254" layer="1"/>
 <wire x1="9.017" y1="3.556" x2="7.747" y2="4.826" width="0.254" layer="1"/>
 </signal>
-<signal name="G/APOS">
+<signal name="G">
 <contactref element="R9" pad="2"/>
 <contactref element="ATMEGA328P-AU" pad="9"/>
 <wire x1="24.75" y1="14.605" x2="24.75" y2="14.209" width="0.254" layer="1"/>
 <wire x1="24.75" y1="14.209" x2="23.495" y2="12.954" width="0.254" layer="1"/>
 <wire x1="23.495" y1="12.954" x2="13.1038" y2="12.954" width="0.254" layer="1"/>
 <wire x1="12.635225" y1="12.485425" x2="13.1038" y2="12.954" width="0.254" layer="1"/>
 </signal>
-<signal name="F">
+<signal name="F/APOS">
 <contactref element="R8" pad="2"/>
 <contactref element="ATMEGA328P-AU" pad="11"/>
 <wire x1="14.59" y1="14.605" x2="12.49205625" y2="14.605" width="0.254" layer="1"/>
@@ -1503,7 +1503,7 @@ design rules under a new name.</description>
 <wire x1="7.62" y1="5.08" x2="1.905" y2="10.795" width="0.254" layer="16"/>
 <wire x1="1.905" y1="11.176" x2="1.905" y2="10.795" width="0.254" layer="16"/>
 </signal>
-<signal name="C">
+<signal name="B">
 <contactref element="R4" pad="2"/>
 <contactref element="ATMEGA328P-AU" pad="23"/>
 <via x="4.156571875" y="11.303" extent="1-16" drill="0.508"/>
@@ -1519,7 +1519,7 @@ design rules under a new name.</description>
 <wire x1="28.067" y1="14.605" x2="27.3304" y2="13.8684" width="0.254" layer="1"/>
 <wire x1="4.156571875" y1="12.065" x2="4.156571875" y2="11.303" width="0.254" layer="1"/>
 </signal>
-<signal name="B">
+<signal name="A">
 <contactref element="R3" pad="2"/>
 <contactref element="ATMEGA328P-AU" pad="12"/>
 <via x="20.828" y="13.843" extent="1-16" drill="0.508"/>
@@ -1530,7 +1530,7 @@ design rules under a new name.</description>
 <wire x1="9.9314" y1="13.1757125" x2="10.5986875" y2="13.843" width="0.254" layer="16"/>
 <wire x1="10.5986875" y1="13.843" x2="20.828" y2="13.843" width="0.254" layer="16"/>
 </signal>
-<signal name="A/COL">
+<signal name="C/COL">
 <contactref element="R2" pad="2"/>
 <contactref element="ATMEGA328P-AU" pad="10"/>
 <wire x1="18.4" y1="14.605" x2="17.399" y2="14.605" width="0.254" layer="1"/>

hardware/Serial-7-Segment-Display.pdf

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE eagle SYSTEM "eagle.dtd">
-<eagle version="6.1">
+<eagle version="6.4">
 <drawing>
 <settings>
 <setting alwaysvectorfont="no"/>
@@ -8528,7 +8528,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <label x="208.28" y="106.68" size="1.27" layer="95" xref="yes"/>
 </segment>
 </net>
-<net name="G/APOS" class="0">
+<net name="G" class="0">
 <segment>
 <pinref part="R9" gate="G$1" pin="2"/>
 <wire x1="99.06" y1="78.74" x2="101.6" y2="78.74" width="0.1524" layer="91"/>
@@ -8540,7 +8540,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <label x="208.28" y="76.2" size="1.27" layer="95" xref="yes"/>
 </segment>
 </net>
-<net name="F" class="0">
+<net name="F/APOS" class="0">
 <segment>
 <pinref part="R8" gate="G$1" pin="2"/>
 <wire x1="99.06" y1="83.82" x2="101.6" y2="83.82" width="0.1524" layer="91"/>
@@ -8576,7 +8576,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <label x="208.28" y="109.22" size="1.27" layer="95" xref="yes"/>
 </segment>
 </net>
-<net name="C" class="0">
+<net name="B" class="0">
 <segment>
 <pinref part="R4" gate="G$1" pin="2"/>
 <wire x1="99.06" y1="99.06" x2="101.6" y2="99.06" width="0.1524" layer="91"/>
@@ -8588,7 +8588,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <label x="208.28" y="111.76" size="1.27" layer="95" xref="yes"/>
 </segment>
 </net>
-<net name="B" class="0">
+<net name="A" class="0">
 <segment>
 <pinref part="R3" gate="G$1" pin="2"/>
 <wire x1="99.06" y1="104.14" x2="101.6" y2="104.14" width="0.1524" layer="91"/>
@@ -8600,7 +8600,7 @@ We've spent an enormous amount of time creating and checking these footprints an
 <label x="208.28" y="66.04" size="1.27" layer="95" xref="yes"/>
 </segment>
 </net>
-<net name="A/COL" class="0">
+<net name="C/COL" class="0">
 <segment>
 <pinref part="R2" gate="G$1" pin="2"/>
 <wire x1="99.06" y1="109.22" x2="101.6" y2="109.22" width="0.1524" layer="91"/>