esp8266: Don't reset TX and RX lines. The pullup is required for

tannewt · tannewt · commit d1b15cf1a307 · 2017-04-13T14:15:06.000-07:00
the Adafruit Huzzah Breakout because it has a diode to protect
against 5v logic which effectively blocks 1s and relies on the
pull up to produce them.
diff --git a/esp8266/common-hal/microcontroller/Pin.c b/esp8266/common-hal/microcontroller/Pin.c
@@ -47,7 +47,8 @@ bool common_hal_mcu_pin_is_free(const mcu_pin_obj_t* pin) {
 
 void reset_pins(void) {
     for (int i = 0; i < 17; i++) {
-        if (i == 0 || (i > 6 && i < 13) || i == 12) {
+        // 5 is RXD, 6 is TXD
+        if (i == 0 || (i > 4 && i < 13) || i == 12) {
             continue;
         }
         uint32_t peripheral = PERIPHS_IO_MUX + i * 4;

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,8 @@ bool common_hal_mcu_pin_is_free(const mcu_pin_obj_t* pin) {`
`47`	`47`
`48`	`48`	`void reset_pins(void) {`
`49`	`49`	`for (int i = 0; i < 17; i++) {`
`50`		`- if (i == 0 \|\| (i > 6 && i < 13) \|\| i == 12) {`
	`50`	`+ // 5 is RXD, 6 is TXD`
	`51`	`+ if (i == 0 \|\| (i > 4 && i < 13) \|\| i == 12) {`
`51`	`52`	`continue;`
`52`	`53`	`}`
`53`	`54`	`uint32_t peripheral = PERIPHS_IO_MUX + i * 4;`