Change resistor comments for power LED

The resistor is not strictly necessary, although it does make a lot of sense on the G25 to match brightness between the two indicator LEDs.

Author: dmadison

examples/Shifter/LogitechShifterG25/LogitechShifterG25_Joystick/LogitechShifterG25_Joystick.ino

@@ -40,10 +40,17 @@ const int Pin_ShifterLatch  = 5;   // DE-9 pin 3
 const int Pin_ShifterClock  = 6;   // DE-9 pin 7
 const int Pin_ShifterData   = 7;   // DE-9 pin 2
 
-// These pins require extra resistors! If you have made the proper
-// connections, change the pin numbers to the ones you're using
-const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 1, requires pull-down resistor
-const int Pin_ShifterLED    = SimRacing::UnusedPin;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+// This pin is optional! You do not need to connect it in order
+// to read data from the shifter. Connecting it and changing the
+// pin number below will light the power LED. On the G25, I
+// recommend using a 100 Ohm resistor in series to match the
+// brightness of the sequential mode LED.
+const int Pin_ShifterLED = SimRacing::UnusedPin;  // DE-9 pin 5
+
+// This pin requies a pull-down resistor! If you have made the proper
+// connections, change the pin number to the one you're using. Setting
+// it will zero data when the shifter is disconnected.
+const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 1
 
 SimRacing::LogitechShifterG25 shifter(
 	Pin_ShifterX, Pin_ShifterY,

examples/Shifter/LogitechShifterG25/LogitechShifterG25_Print/LogitechShifterG25_Print.ino

@@ -36,10 +36,17 @@ const int Pin_ShifterLatch  = 5;   // DE-9 pin 3
 const int Pin_ShifterClock  = 6;   // DE-9 pin 7
 const int Pin_ShifterData   = 7;   // DE-9 pin 2
 
-// These pins require extra resistors! If you have made the proper
-// connections, change the pin numbers to the ones you're using
-const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 1, requires pull-down resistor
-const int Pin_ShifterLED    = SimRacing::UnusedPin;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+// This pin is optional! You do not need to connect it in order
+// to read data from the shifter. Connecting it and changing the
+// pin number below will light the power LED. On the G25, I
+// recommend using a 100 Ohm resistor in series to match the
+// brightness of the sequential mode LED.
+const int Pin_ShifterLED = SimRacing::UnusedPin;  // DE-9 pin 5
+
+// This pin requies a pull-down resistor! If you have made the proper
+// connections, change the pin number to the one you're using. Setting
+// it will zero data when the shifter is disconnected.
+const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 1
 
 SimRacing::LogitechShifterG25 shifter(
 	Pin_ShifterX, Pin_ShifterY,

examples/Shifter/LogitechShifterG27/LogitechShifterG27_Joystick/LogitechShifterG27_Joystick.ino

@@ -40,10 +40,15 @@ const int Pin_ShifterLatch  = 5;   // DE-9 pin 3
 const int Pin_ShifterClock  = 6;   // DE-9 pin 1
 const int Pin_ShifterData   = 7;   // DE-9 pin 2
 
-// These pins require extra resistors! If you have made the proper
-// connections, change the pin numbers to the ones you're using
-const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 7, requires pull-down resistor
-const int Pin_ShifterLED    = SimRacing::UnusedPin;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+// This pin is optional! You do not need to connect it in order
+// to read data from the shifter. Connecting it and changing the
+// pin number below will light the power LED.
+const int Pin_ShifterLED = SimRacing::UnusedPin;  // DE-9 pin 5
+
+// This pin requies a pull-down resistor! If you have made the proper
+// connections, change the pin number to the one you're using. Setting
+// it will zero data when the shifter is disconnected.
+const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 7
 
 SimRacing::LogitechShifterG27 shifter(
 	Pin_ShifterX, Pin_ShifterY,

examples/Shifter/LogitechShifterG27/LogitechShifterG27_Print/LogitechShifterG27_Print.ino

@@ -36,10 +36,15 @@ const int Pin_ShifterLatch  = 5;   // DE-9 pin 3
 const int Pin_ShifterClock  = 6;   // DE-9 pin 1
 const int Pin_ShifterData   = 7;   // DE-9 pin 2
 
-// These pins require extra resistors! If you have made the proper
-// connections, change the pin numbers to the ones you're using
-const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 7, requires pull-down resistor
-const int Pin_ShifterLED    = SimRacing::UnusedPin;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+// This pin is optional! You do not need to connect it in order
+// to read data from the shifter. Connecting it and changing the
+// pin number below will light the power LED.
+const int Pin_ShifterLED = SimRacing::UnusedPin;  // DE-9 pin 5
+
+// This pin requies a pull-down resistor! If you have made the proper
+// connections, change the pin number to the one you're using. Setting
+// it will zero data when the shifter is disconnected.
+const int Pin_ShifterDetect = SimRacing::UnusedPin;  // DE-9 pin 7
 
 SimRacing::LogitechShifterG27 shifter(
 	Pin_ShifterX, Pin_ShifterY,

src/SimRacing.cpp

@@ -85,7 +85,7 @@ LogitechShifterG27 CreateShieldObject<LogitechShifterG27, 2>() {
 	const PinNum Pin_Clock    = 15;  // DE-9 pin 1, should have 470 Ohm resistor to prevent shorts
 
 	const PinNum Pin_Detect   = A2;  // DE-9 pin 7, requires 10k Ohm pull-down
-	const PinNum Pin_LED      = 16;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+	const PinNum Pin_LED      = 16;  // DE-9 pin 5, has a 100-120 Ohm series resistor
 
 	return LogitechShifterG27(Pin_X_Wiper, Pin_Y_Wiper, Pin_Latch, Pin_Clock, Pin_DataOut, Pin_Detect, Pin_LED);
 }
@@ -103,7 +103,7 @@ LogitechShifterG25 CreateShieldObject<LogitechShifterG25, 2>() {
 	const PinNum Pin_Clock    = A2;  // DE-9 pin 7, should have 470 Ohm resistor to prevent shorts
 
 	const PinNum Pin_Detect   = 15;  // DE-9 pin 1, requires 10k Ohm pull-down
-	const PinNum Pin_LED      = 16;  // DE-9 pin 5, requires 100-120 Ohm series resistor
+	const PinNum Pin_LED      = 16;  // DE-9 pin 5, has a 100-120 Ohm series resistor
 
 	return LogitechShifterG25(Pin_X_Wiper, Pin_Y_Wiper, Pin_Latch, Pin_Clock, Pin_DataOut, Pin_Detect, Pin_LED);
 }

src/SimRacing.h

@@ -976,7 +976,7 @@ namespace SimRacing {
 		* @param pinDetect the digital input pin for device detection, DE-9 pin 7.
 		*                  Requires a pull-down resistor.
 		* @param pinLed    digital output pin to light the power LED on connection,
-		*                  DE-9 pin 5. Requires a 100 Ohm series resistor.
+		*                  DE-9 pin 5
 		*/
 		LogitechShifterG27(
 			PinNum pinX, PinNum pinY,
@@ -1109,7 +1109,7 @@ namespace SimRacing {
 		* @param pinDetect the digital input pin for device detection, DE-9 pin 1.
 		*                  Requires a pull-down resistor.
 		* @param pinLed    digital output pin to light the power LED on connection,
-		*                  DE-9 pin 5. Requires a 100 Ohm series resistor.
+		*                  DE-9 pin 5
 		*/
 		LogitechShifterG25(
 			PinNum pinX, PinNum pinY,