From f24ea7aa76cb39e0c957e93894a55f1de345302e Mon Sep 17 00:00:00 2001
From: wolfv6 <wolfv6@users.noreply.github.com>
Date: Fri, 23 Sep 2016 14:52:09 -0600
Subject: [PATCH] add refPortWrite.begin(strobeOn) to Scanner_IOE::begin(),
 update unit tests

---
 src/PortMCP23S17.cpp                          |  14 +++-----------
 src/Scanner_IOE.cpp                           |   4 +---
 tutorials/breadboard_keyboard_supplies.ods    | Bin 17707 -> 17707 bytes
 .../PortMCP23S17_bb.JPG}                      | Bin
 .../PortMCP23S17_read.ino}                    |  13 ++++++-------
 .../PortMCP23S17_write.ino}                   |  18 ++++++++----------
 6 files changed, 18 insertions(+), 31 deletions(-)
 rename unit_tests/{PortRead_MCP23S17/PortRead_MCP23S17_bb.JPG => PortMCP23S17_read/PortMCP23S17_bb.JPG} (100%)
 rename unit_tests/{PortRead_MCP23S17/PortRead_MCP23S17.ino => PortMCP23S17_read/PortMCP23S17_read.ino} (68%)
 rename unit_tests/{PortWrite_MCP23S17/PortWrite_MCP23S17.ino => PortMCP23S17_write/PortMCP23S17_write.ino} (60%)

diff --git a/src/PortMCP23S17.cpp b/src/PortMCP23S17.cpp
index 99141c5..312e013 100644
--- a/src/PortMCP23S17.cpp
+++ b/src/PortMCP23S17.cpp
@@ -22,6 +22,9 @@ strobeOn is logic level of strobe on, HIGH or LOW
 MCP23S17 SPI interface is 10 MHz max.
 The electrical limitation to bus speed is bus capacitance and the length of the wires involved.
 Longer wires require lower clock speeds. 
+
+begin() is called from Scanner_IOE::begin() twice, once each for refPortWrite and refPortRead.
+The first 4 lines only need to be called once, but seem to work OK if called a second time.
 */
 void PortMCP23S17::begin(const uint8_t strobeOn)
 {
@@ -41,10 +44,6 @@ void PortMCP23S17::begin(const uint8_t strobeOn)
     {
         pullUp = 0;
     }
-//todo
-Keyboard.print(" strobeOn="); Keyboard.print(strobeOn);
-Keyboard.print(" readPins="); Keyboard.print(readPins, BIN);
-Keyboard.print(" pullUp="); Keyboard.println(pullUp, BIN);
 
     transfer(port.DEVICE_ADDR << 1, port.num, readPins); //configure IODIR
     transfer(port.DEVICE_ADDR << 1, port.num + 0x0C, pullUp); //configure GPPU
@@ -65,13 +64,6 @@ void PortMCP23S17::write(const uint8_t pin, const bool logicLevel)
     {
         port.outputVal |= pin;                  //set pin output to high
     }
-//todo
-//Keyboard.print(" readPins="); Keyboard.print(readPins, BIN);
-Keyboard.print(" pin="); Keyboard.print(pin, BIN);
-Keyboard.print(" logicLevel="); Keyboard.print(logicLevel);
-Keyboard.print(" outputVal="); Keyboard.println(port.outputVal, BIN);
-//Keyboard.print(" ="); Keyboard.print();
-//delay(200);
 
     transfer(port.DEVICE_ADDR << 1, port.num + 0x12, port.outputVal); //set GPIO port to outputVal
 }
diff --git a/src/Scanner_IOE.cpp b/src/Scanner_IOE.cpp
index 07425b1..6951553 100644
--- a/src/Scanner_IOE.cpp
+++ b/src/Scanner_IOE.cpp
@@ -13,6 +13,7 @@ Initiates communication protocal and configs ports.
 void Scanner_IOE::begin()
 {
     refPortRead.begin(strobeOn);
+    refPortWrite.begin(strobeOn);
 }
 
 /* scan() is called on every iteration of sketch loop().
@@ -23,13 +24,10 @@ read_pins_t Scanner_IOE::scan(const uint8_t strobePin)
 {
     uint8_t readState;                          //bits, 1 means key is pressed, 0 means released
 
-delay(2000);//todo
     //strobe on
     refPortWrite.write(strobePin, strobeOn);
     delayMicroseconds(3);                       //time to stabilize voltage
-    //delayMicroseconds(300);                     //todo
 
-delay(2000);
     //read the port pins
     readState = refPortRead.read();
 
diff --git a/tutorials/breadboard_keyboard_supplies.ods b/tutorials/breadboard_keyboard_supplies.ods
index fdb7d1581254effa59e2f4da60b7dce80d196253..d7b3dec754e3cbd2cb13543e083f93008a5131e2 100644
GIT binary patch
delta 4167
zcmY*ccQ{<z9v!`t1kr}kdkF@EQ3p{%^iK3%BFcn_BRWAaqC^+Hm*~AEdI^FdBzg<c
zLI_^m-23i(=byEHXP>pd{oDKd_F1bm3Zpa%gIH4y3!4H2!Uus$GUJK0F-tP5Eu0{~
z0W}O^1eQE=B`5+q0LAbb$#Ug}&wa&vjC#9n^R9A^J0^ieb73JWK@QjFbPA{S6!mv)
z3z6&RxpAwK92^`=KLs<Nn}wWNkkZ^weD``iz6310JPC?-G@&862!E7Vnb9%2X%pl>
zOIQR6kBdN%hQ{ffy)e~y%0uVIBgn!>2wUdE1mycoy4@%66Gm!29nF=$uG$^JWI3GG
zx06>WS7K2~c)NNQQRDcwc*oCLq&an4M@VkcRnfLK>t$}@)tyBC{zM~!N!=42mS_5;
zpH!uVV+y=PeP&%cJnQeqJ*Q{J=$R75+nFhLa#f6S8M*x+z(}3T`1DE4OW5H(C^a^M
zmJ3LetBzh}WR~A5vX7Zz+{bJWA=e^1q$lhx!*BLIRJ?pA{0o%3Ri=5eo;*2bvH}TX
zlu%#|B%)rm?aDh#)OXGUs@FtmdsdnIiwAH}{MlaG;STL;lN8Xxja$kuTk|)E*hY!v
zCl<P~Mp8Tm-8k>dn+IOVI~zz2B=?gk_5&X{rnB)9b2wD7J2gV$8*?#+=K-ZTQfbmu
z*#NUm@o!>x(Hr#7D*%$SFoAJ!#ki|%0o(dS$qV=6&%*Am7T<)l=WZiAeznPc(`Iro
z#4a<yV$h)_Wg9e?lECrCh&?Kzd0wRF3M#nAHnWtQrH_rYgH}@y9@XFH3Jc_<?*gn0
zqfNgb`ZjXXnID|j8#wo!u`#i|{gF}p!JfjJtm=@lVyp1_Vzxp$iEwk!X8j<n3rfRz
z)IiYL+WWQsws2T<#qq{Ia?|yDWpNQvGn&tzNL*2btAH0T#?deQ4SK7(AVc?fpK8N6
zSPwT)ZF(hc7D^*~tA9D~E)x%Xye^=Fb&hj=o5*Nf&Ys4P!V_=Zde1zh%rt<JMMC3k
zbKVCAmDJj)ET2YOss$!ST5Y<=7)R!HWEeD63p)B3)9hVgrV<9e^_9;nl##w?-HCv~
zQ;r{)(EI*leS<vTt=3ALHE`W|Zy}R<vjkPgb()g*q)rVDI{L_A6!T)R8Y(~#YYU^J
zaFXeiud~}|?Ve~lMImlhDf-m^Guacn!09I!0$lhmDt(DxY)QW3z8A_<o{w;!62>@S
zT9h%1%?07l+6##XTZXLdOK|>_wCVkVAD5PL#*B-Tzah>0iAak2*p}ObwpChs=Gq{&
zac|1<k;_kZj_LFd)5)Jdr6gPf@x$dm5^#od(oT+nTjV<C_a*e~Beejb#wgJ^v!U<=
zrC=Nan|>L?v4ys!#lm5P94L~j2l`#ncxD&<yaU^YHh6}w|Ac|Dc(%2`gS!LUdsTOL
z2$LPl`bxT7i_X1rznhCW`Or;YX*2#y!wypY-V&T&fuR<j*IFL%3u6ZemeEd_F%F@w
z&SvinFnze(z)nEJx9ksDzyfCmY%^o7hHk@YgFEP)0^7#Tf?7(q9KQ1%M#NkeA27+V
zh-{L1%y=wi`q@((gq<NFR7wCmVd>Rkxg{(o)}^#k8uq@M)vW21+}j_>nSVp~YQIB~
zY-q0>vJ9(Q$fwzUvH-m9xZrM_T%l`_JdJGgZ|o_dF&}$QWPPn-mSX9-9xk^U`h)n^
zDOc5-S(b+L*Qu<@8@L@pxmlU@DHl2K_B#v2dWiG}0}go&=$6=;!4KbmI`i<CiDh^f
zf8`%=YdLEr=S@oEGw0KajV+q4OevzW#nbp{4(n5ips$-=4oyx#N$o^!T6<mRclh~N
zM-Icic|pG!F+{_>_l4h>cZ)&2WE$~(zp?iGu+K>MaH-R;=F@!7&~66D+nn7)x5IbA
z_*FpV6=l%_S|#GcW9{~qS~FuWG4so@Y^@(;(W5e&YIyiU3iV$Ma6q6QCJ^Y)WsHxH
z|NAl)Qvrc$E!iN%*lx8u9jH)2S?rlZ^U~S@K?nx*Yw0;mM-mXo&Gr993Bq7tBzE#1
zMd&G9(Z+7)jKdLa=%{4%!jKp<lDWE6`$9vo(Ps^YI{X1s(GeQQ83F?zHcaP>83Z^`
zt%2@!wI;pVQFp37a=E@TIrctfa38n6EO;_DW*nrc;lojdk;MpAd56}2pk^}V%VRpZ
ztv|btU5>BaC_FpIob--5n0*bfdUx5RxP*Wa$)C4Byx{e|QZ_V7vN6rjVsl?;I<m`4
zWVNtRb|6rrNuA62dYwc&Ys<E=z~|6e_|Y0%2<4bOeE@0GUS9;ok0`s$hTwKMY%b-l
zN?a{>ikECm(mPE}8!GuZ$g@6%B1X7)?u`gO+JDMUO`cxpl2Kk0l%xnK?!ma(`nrv+
zGQSzJ>ra>!*p42aG*g!ei{=M?WHN{AXFyI%WcH~qIz=I#S8KFKWQS-jKn2Uyi~{&-
z@I2_11ygfIW!7<5PoaljjjB<z>dzij{b$>2SDoFLa3jz~EmB4nw|ldvMT0K%r^xD9
zz~ohf0Xl59n#7p>ErZOPU<_U}dj#AU8hIzW$uZGf4@sSCm%*%UusY2b{|r#Lq`NtH
zT_glE7Xt)(i1q(Rh?<2m0#mSCR{<T&st@t=hS=v^b6UI_B<LkP?&ubEs+>-!r<aj&
z#Jbowtj#RVM%MbeZxXG6uzLg6`^KU_tmN5m%n;Q1q!4v~tot?wpYL|;luvl?d~sAL
zlc96re>6n%lKIXTz5XD0SFu+}YmT8IV|tXHU{k$kW(D;fg);3#3!ploE6NroB3~-u
zm#<FAVmznoK~kZn=FeA1$m4>zYo^{%*5$nw_x4iPG89K+XZdy-w?AJIxgdvyN*Jr7
zuZOzIxvi^v{;+;U%hJC6MPKy6>`C%aeyKfAcran|TDp@ibz_Q^?SsavVI+xY*B$kf
zhn741`Mys=f7&?3DF7Koc3`5w&L?{28alekj;k~LndxRNPXpI2UY@POC%@R0@OEz%
z*WmBe5G2gMILmC_x75Bqx%$jh`BZ2+sav#&5H`RrP%AsLMq~ZDR5g2c7yY?ugTAHD
zlcWt2x-TA|GV`U26D|uaqPACoI0hb>$-<r%YA<Ft86Rh_lLN^s(H=(Pchm)ZI!9a4
zB_^rkU&~AENTBPMvqU(^yZkODv(VAlFPB<xlVXlKyhMu3JCtzz$MjO1h(K-ORzoY>
zun5D<u)cbp{>~Yibp>^%l0JPBl9TA1T~h8L;k<$zEkl9=y9PbAZM~ip6n=PKoDHwZ
zbRM1LqlnO5BA~BO&0s`)TlYn?5xSVZ!1I~4#%M%WF-;Cgl8LlvHl32PlD0m|er*+f
zg-uGhgSlU>mXtkJpUq3-Aip_+4hR!sH*a{{m+dN|%hO7}LZMHR7M^@g8WG(O8XO|^
z$o1@@M}9*FPB4TIpH(?lbI31zIF`f=kJ2F6VNva41bFxsu=V%8q3M1JoJ&HGrD3Vd
zA7kQRO;lzsYI!HP`<hW!aWw0tIKTK8+wMCqq$yALzBT5+un$uFc#hZB*E99*@lt5&
zE8dNJ$9E~WoG(2r&6Z(>5oFEcyoVMHEeP*2umC@{oh?(WrpAzS(1dAo*(*kl&O|hB
zCR40cEKpO9lB0ibdWX}wk0M)+Xed1nKT_L!#ISXE^@W0-e~T4*<VDTxYRXX7AuZxs
z*fbUfSS%qJnI(gEr6f&1<aBj^5+mqGyt?ygIFWFq7eZxLS2C7&=4a}GgHEX5cG|8a
zqR(O27PD96G51-Gc}lTs`qYQW=Ji7Ym6HMLV8B$_iWvv&h}L{Hs(L@pla#7IO<JcN
z`KwE~bXJg1POCv+z|-ZFvr(mjT4WK*VpmmH<Qh;8b}ptigscjv-0~J8oFjG)<}cr5
zo$7wZe&12BR?yzJV1Lw4Po(}$?}N1=2360D+Qcrl9nL8DS-#6_2}++8-K3M1F;(|2
z4Irq-$up0Mi$6wE!SmOzzFNa}a|6fln&>1O$(VPo_C@P2A+S0s>ji;K*1C<CSsZnc
z$i;AF*Nmi7lIa1mbo2-71&EBgQ`Rqd@ul*yw6arHbfFK^9LacEu3U|~yPCTrpSrsk
zx(a!|r@grt)1n2PYES!F5((LpdFB>H50p}##UkGa2g!x=D6A}M5v@FaAW%(BR;*_R
zEvcl^5T;kEFZ)sbOIeOJm-XH`6!s7?%%xJ@g|DOn1I?i{+eQ?dY_K9~WF6j~U;C`}
zFrc3D%!uwTeK#3MU5d<LuRhPWZb3@yZd7bFP!|iRkdddT7HuRIafWpE7PA~D0hm|)
zby}kyVe!o5+`5FviV;!9G}~>@G1u_&oh3WB>A!w2uN}N|Na=F19VU?|eoACFPy}$O
zs>2F22aOJx`d$btHFX>}Yc!V+0Y8fpBr(2rI<`~xPlYDs(j^V8&!_Mnd}Gj@;X085
z-TlJKi!DA#@9P~ga91DM_)7FWGw`0#zdW!Q;cS9yxi_}X?m)0C_&^KX6*2=}#c37s
z>&w&4qB9?9wLNg_ruQj6UL#mY<lQ-_-g-43*&VVu0u*c(r}Df<oM3oLONKG2QF-n3
zPj)~V5tav$3uzf=O87XBbW2fT1z`}J%yYA99fT|k<%&kGm;Sj-e;xz{RL<9;>C5!j
z6oM4;<-vaXQbo`h@7E{K$8}Z-PsQxdH*VJ)KC13(lyx5+8yj2rVIiJSr|h6;t~ddn
zQIIVSm=ym`=*GPRnKB;4Tl1NC=<aV_Df-EytH>!%dOK+T>w&0~z>s?-%kZe=@Eeqj
z{{56En{)&9nN%<p)^bD?ut7!7Oq)cD@WZPsL$HM6J>%Y-;*NV;iKwcEU+1sz(Vw7u
zk4!H=9H|OWJU&XZ1#7q&l*=mBbr@z08yx-IJwATjVb~5g=O*X5EHw6dbS^S;Y9sF0
z<elAq>6SDkQAPbMZ^8SbG<5!&Tqo`NxADQl@2klJHgG{84>l0V-%jXHEJaO7iQ)cD
zEv1-IEm9OW`SgaW{X>Ot|F|a9N4Vb2ATJ#H2bfW^@`Qx{X58N+%&1{`LR2gq|K^Jn
zIP5pnl1fWq+@QLQ%?+YzW&hG|$tvLASg{+Eh7yqzBDg93GGD02a`zxN)&Ks-f2ZjG
zd5%yWa#VjQ{%Q@iBu5Dzf`UMvUcN52p8QW;T{P7&FeyO)#BY?50vN?BPl1Y5p!%oZ
z7YHyW2OK4!z<IN(o9Yo9m4*0kl?+FDNHGxqnda{;;6;Ej#!xB<(HqGvc`zos6bcRp
zqvGTV|9N97h2ocI|EEfj`aQ{A)VTaD#y`IE-zLES(%-CKDgnWLqbg9i*)`M`1-1VG
DSsAf=

delta 4198
zcmZ8kc{tQv`=1$O8+*x+vBpE$r4VBsvc(`%_PsKUHDoMbOO_<tj4hL~WXYDLku8)W
zBKwxLvTKk~h<@pL-|P20@A>C*-S@ff<(&Jv&iQ=G;=yI{U}m%-4U`iEVgP|MB~qBp
zAej=?4p@QTK<E(=1g!l^FL8#cx8}F6WnrFW*pgQHU4qxvPZ8@{XyYE@^fE=_cE`wq
zABN1vDdNHS8;LS7{T511%~r*`J6}UTevVRhL6{DrOD~uDg6<h2QGc<xDlH>&BxbV#
z+y1ZI4OQ!=irYdQu?Uz@@g_|JX%jC$h1Swt<@A2N2K|u?j73`)gGI7kcgAYhBpkc4
zX?0yj(M4L&9Rtz${DS2laW(pdrEXTP7aFrx&E$0nI33Yi`}_IC7KV7ZNO$?&k*hn8
zQ&lDC-9_!Qii#8z1I9gCgQ~fbL->TjhvOIN*T$-`EIRS!eH;mP<wY(P!A%{y2xab7
z=14FLu{RgEPUp+ck*6?nzdJbD1Z@AZTAY3l9OU$n))tApW)cvyp{bB{qtp(meo%Me
zD32{t8>LzwZfC`nP-Ego(cOnzdltwg?AsY!k<PpgF62=`?U+C8$UTds*RYyDqcbaA
zUBn*0%Ef={Zi0EhW#toF5Y-!&O>_D!I$_;fYDXA=x$!a&z6b^Kpyj^jzWVanVKI?9
z_c@tyK0P{TFw8kGbY@^|KO_8U6QuVeY#Qp4>YQUGxA~wYc(H<|JnQl~l~3y#t%@a5
z=b$^a;S{H~GX{qdsEM-6ow*m=98X^ma;KEAv_U)O_pP^UBj-kmy0$Tc%~W)}kf4Ei
z75fd~%+2NyKIyHPYt1dx5i7YcV?F(n(3`71Ja5LqlIn3|&uV{#dR!B@Py5V8{(O_O
ztTElzPaLyspHDsRb%J1l#Npl&AD;Io@44<s=>R5mtwd9n__KzZ&ZlSdu+%U07!|4I
zS+DmLXxTO>qcm1c&TTlVjqpbQtQN*d_(>5I*j8(0iI!Z>rEK%XH_#5r1<Gh)q-O~n
ze4VDYgU;End$QI{RHkawo-;7e4MT0(o?WAAYd|Ew6R&C%>dOjcyid!fUUQ#?4ceYo
zJHIe{Y6ix~*2~dn0hr89bi@50LGA1XJ*DocxDs=hLS&|c*Q+bTYQ?<8>%6>0WVGmj
zPc%nH#oFuE;n9?CM<b>8#w@V(sL&(lX||f8@HF3pr{bGN`frGHu8f<PI^-YgO~x6n
za);NmZ;+*9Bj}-DWV1+`>TO$rJj9<T<@5cZZ*BE#I3g1U{RFt&4_LP^$*1+Q*x8fU
zew7*71guuc)cs5*7`*@7{m#?SsOy&=u!zBTK^FUs@%ztTw-GZ^jMFTgh-Bu<d=?kQ
zEurFp{%ENDJb(Gk^%<`X^rB#2)azu&)AhoY?g7*md$2s?%LA@?JX<oShux5DIyvgn
zGuW;xm=GZsN**mqOB#DV2ThR(Qe-;w?B4guou<Ds1^D5@wIXa?US$Hn%l$bZi8x=#
zC4?6*0ax*+6Mj_aYw6yXou&^2#oiQ?LWMV@d$VQ^n)31EU#X&Z>T{TbGa|H&z8G)I
z?D3S^-3+%*Km0mOZ`(ThruATX?a=@_tdA6I`#gjtYdUe?WrZo~+5C*j$bhr`dG`P%
zd^2-SdPVS9XWkekBj#zBkM$9-UNWc2wDDzhN_25+1zZ*boDXc`Zt7cowOM99_8rox
zSR6jyXZJjw&-bY<uj%a_?WHmN=DR0Si<L6e$@44Ixlauy`i#daAMgubNxWRJe(NG0
zL2E+E+r_smeYP)%IL%Y<?^Wlu_?72no$^j2m3*TwQKX}b`sMce5AD!#fSQ~VdQt<p
zb8-0twwb=c%g}3fiscLD-TE!uq^E86?rl|WEbd#*>W$78foIg>%NA=73AzQj*XY4^
ztjaFx;h=->s~H-uXmy9=&Wx?0N7SkOrmb~=F0C1GQ1}H#8`3l6oYq#kMhgP93xPm?
zKgJ9U48I>^MST#c)=^Y|nHF2?zVU)^S55;uzGe5M_N|-%n4411<BDYgfjn{ljw1Y$
zM*xONL$aLF-+#ux?HT=g=E~r8H(jTQ>&%dQ!qsJF2gY*s0Siimt&Of)mt3ZF<)g&D
zhB|XKyS~5h;MrR@1LNLs>Npgdf5-_~UOPh0<ae%KS;-+&pm`*ZJ_i~#?UI5SEU%mt
z6}?vB?O(v_xOQGaSns2>1N{E=At&aC#4=!DcZoHW?V~WmN~838>iFJCM;+5-AAilX
zW}l4F>=d)q@CcziA;}vqj<GSntg#MbX(YerxnQzEk`HN+C94I#Kc6Q5oo;#J?l#LB
z`ZJtm+M2WX^HWw*0akfBPuC*R^>OT_ZRmP0+NWFINHKFMIESGFx+M3GpwM+IB@LKQ
zEVyb`UQW9hQngj#5M<*qana<Pq)b=jl_$Bkmu2|NMw5aq$lnt}9rik}?7JrJ(Fak9
zt&by$x(8b~Ey;5yuP;x(zh<(({BcMLsqf5HEC<{ftb4)Vf~(c8yN7A}_-UFgIYM@S
zvvNe&Q7!i8gx7~(y9U|)fgX|R10}KQ`?o-bV;}m-st-|&9a9PhH%ItlOLU6ku8Icv
z8_v%Nm>$v{Prh`jdNCgi0-4bK&kV;D1cAwm>_;v3LgLy35Q6B0zEx9ol;fLzFb@5g
z8XoE*Vz6j%527=n4;&^E_12JByN8wf{krphG9l}CRIQX%T8&F6?W$suwv$IawNfNH
zNfEc^I(1-wD9DxU6{hTLih?x5EYTuLr;MdN#otS@8-KG<Zm46*spRg~Z!s_hBEO|E
zU?kh0a9NEENVt|FOuLKXCA@8|=G^5ht~>G8EMps8X)6t55!7Osp`^DJJZb#u$2HO=
z4nIh`i(Ya%e2n<mesmWT=dsXua2Qr+-@~0`3pCg?Vn>w_+%_2RQncfoQ+#3d)XvS8
zvT8=ITcyAW>%g;8sS-V6=1~B26?NgW5<P?DwZ#Pm9sexdFzTnS3Cex?TdZ|)ChyPF
zdKPsHlXA}8uaeFdtmcc`kob-?Dg|a*Rpw8b6n#h?r?UWyP0YKdFE^?>_tc-G?e49d
zdK5fUZLK!ee_;9}=VU|>3xC2Z8Wc^tlHtA{4QT`N>ZMB7*6H4beEJIjxgK#df$6iJ
z!i#aRieIh8bK$&pYm4{TTRrVnaM-$}&3Ijv8;S4bL4mFm?-H<b4*eqR0qjSPgxn1#
z(2H27cg}tYRJmTe?_`?G+#0i=a51}uQ&~(5z5^XVpEkl{Hq<X(4mE!<*^`PNkzEvN
z&^FVPPp`lZK6pf4Vj2M8@g^CSB)|D55dYd5a}}edU0pJXBg*`|7C1l)(I4W6!Kj}1
zNb4Kyq8u_JdMcQjpG#XtnoUO5JkDdxQ*=Mz>Arr{7%f4ME6cO`^oYB-3OlaaJQ%3q
zf`X#$0^&}l@vhgqx6&l+Dn0Iuy|NUXy7XP{FtIH!-|W?X+#nE=oW|t|k@y}!WjTT$
z+9otvv>W-X`~_EPfZKW)-rb?wK*u{J^DLIp{*<11aoKGuTkpmv(+Wa(ElKCqKK+X~
z?ZeN!XQQ){%I9RaxzbkL4}BbMKh@{Pu{A17ZaMHbY5SL-mXU$tU4;_Snq}^hqc#oY
ziGpHn#1E%)gc1R##F`3%CSQjQySV!TXRapGJ5usIq;TtDwNmfv2$W?=lT&wZL=8vv
zsb~=iG;^(z1q~RX7#S5vR{wxI#Y)-|$9efi58xH%*D!roVLPt{;I@>SzQ;fCHq~I9
zzBMIle@#oLBzC{oaw7g3U(gwTQ5AU3G{kbmT+7T~9_SE+r)I_>5*1uaE*B4QaFW_-
zw~G3R8+^d#_PKxrE%ZFqa1%0#gn6b)=u#AW<J>%?u-WzxoTX0DtVR74Wz09$70PzR
z^%tf+x@&W%Z(dvqe{r$=?eMf^MQI0ykohEJLffs<H(kEog-<7i-hL2MXzrw#SY6H!
zVDN@OzzJejZ%WMLW`%Wjsm*WjKBupE+#stfPsQ7S({4e1=w7(GH&&*j-d<H7+Xq)S
z!K$Bl!ItvjoYV$yDM3JqlGItR9`<lpZ7;~|AZ0l;d036dglbm9o2%#=<uW^yJ~L>w
zI^+4wabtTrt^WmYV8yMH(6Mht_XO6}gFWK`zB0OyPL<D^FLT@J2I>QcUnwDm3`t3`
z!}4&JZ@aJDirU|%Tc}}V&c<mFp;lxF+$~O7^z3LR$@B!P#;ypFyqK?51x8s25>0oS
za3SN)C%oQ#h&kj9lTiMqQavPs30)pvf5dvw;Jz=VyLDx-9ooULGj&2Vnmtp=-<JXu
zhO7Hlr1_B?MY`_7(jcK6`9T%TJ2Pq#TTz1)K8HP?kDC=kE{uDyy3inPzHZV1ZLQOA
z>&e1P2y^j;@{p8IZ9HeD?eVr^)1wQmcw*P1aPK(<^;2)>qto+wF>#B<a+aSak-3%T
zTcIGyHvQ`izC@KztLy}+@^x<kTmqnzp!vfJOo$z=gT0y}_xksrc<r2-uN2AnRQavp
z9rx{cMj?B}W>V0*ty;#w@xty5F>mv#^3Kj8bx$$a$$Z6MS@~i&E4kSgn0ofUaK}hm
z^jDlQ2|cS3^J?o<gQCh;3OlUjEk=yRpG6&}x`!KoB9xmF?;~b<N#anzWiSco8lzfL
zSutf;Va*296ahTug7}38{%Jd*{pRxq-xY7JZgAAB{tCJ8b8m)3CX*YeFR>!-DgE)y
z3HfT3`m7cESC-l&8stXR-%_r!4N_~!Ge17q;mSWpVnzf*hiRX;f5r{lVK(e8XFdwd
z91#N~lVn%=6;p9Y8J|c7t5|^8Cvyp<AbBBXm=$Xos|#b>z1shTC%K?fyQ*6C*YMZ$
z_Xx~tQ04yD=58j&sh>C8VhNKLhRLQ#`$s>@_Ex?ue0{P(?m3KBs5Vo-F%Y^kW;07z
zc)TzgvAWx2DZSR_`z6J9jQW(z^OwC5?uh32KV*M;DR7Yv1o9CDf&QgS2{}mNR1^&M
z|4Q}PLuk~75$uo*$67T9sc;O#NTp+#RJAw;ch!r3gD{~D2_<x?GSmI7bARs$6CP+W
z|1;sPiXz?L#H*H74GI1oSbIg?<roRI8h?_nXrLI5Y1uJhBV5pwXFN9k@Gye6rmVoR
z_0J#ux1#>X+9tGW!v944IVE9M^Aw^-0R-~99pvHaCw&L!fi?s~I6?oF;)H%}1VK`Z
zGZn>o+`?}QBB)9z(f;ed7Dkm&i9`_4DA=*b28Dnms1op~bAo>}M-T||pXy@&=mG@l
zP$jIXo+X56GarZ9Y9SzeNJ6kGKl8teANwtkzYB12dOLc$ddU1wbI1D+e#dEvA7^3z
XZM{Jfq_xie=LQKys6TU<LmB=bTIs2e

diff --git a/unit_tests/PortRead_MCP23S17/PortRead_MCP23S17_bb.JPG b/unit_tests/PortMCP23S17_read/PortMCP23S17_bb.JPG
similarity index 100%
rename from unit_tests/PortRead_MCP23S17/PortRead_MCP23S17_bb.JPG
rename to unit_tests/PortMCP23S17_read/PortMCP23S17_bb.JPG
diff --git a/unit_tests/PortRead_MCP23S17/PortRead_MCP23S17.ino b/unit_tests/PortMCP23S17_read/PortMCP23S17_read.ino
similarity index 68%
rename from unit_tests/PortRead_MCP23S17/PortRead_MCP23S17.ino
rename to unit_tests/PortMCP23S17_read/PortMCP23S17_read.ino
index 8be8881..65ef54d 100644
--- a/unit_tests/PortRead_MCP23S17/PortRead_MCP23S17.ino
+++ b/unit_tests/PortMCP23S17_read/PortMCP23S17_read.ino
@@ -1,5 +1,5 @@
-/* unit test for PortRead_MCP23S17
-Picture of hardware is in unit_tests/PortRead_MCP23S17/PortRead_MCP23S17_bb.JPG
+/* unit test for PortMCP23S17
+Picture of hardware is in unit_tests/PortMCP23S17_read/PortMCP23S17_bb.JPG
 The setup is an MCP23S17 I/O expander on a Teensy LC controller.
 MCP23S17 port-B pins are alternately grounded and energized.
 portBState is a bitwise reading of port B.
@@ -7,20 +7,19 @@ output is: 10101010
 */
 
 #include "PortIOE.h"
-#include "PortRead_MCP23S17.h"
-#include "Scanner_IOE.h"
+#include "PortMCP23S17.h"
 
 const uint8_t PortIOE::DEVICE_ADDR = 0x20;      //MCP23S17 address, all 3 ADDR pins are grounded
-PortIOE portB(1, 0);
+PortIOE portB(1);
 
-PortRead_MCP23S17 portBRead(portB, ~0);
+PortMCP23S17 portBRead(portB, ~0);
 
 void setup()
 {
     uint8_t portBState;                         //bit pattern
 
     delay(6000);
-    portBRead.begin(LOW);
+    portBRead.begin(HIGH);                      //HIGH or LOW, not matter
 
     portBState = portBRead.read();
     Keyboard.print("portBState = ");
diff --git a/unit_tests/PortWrite_MCP23S17/PortWrite_MCP23S17.ino b/unit_tests/PortMCP23S17_write/PortMCP23S17_write.ino
similarity index 60%
rename from unit_tests/PortWrite_MCP23S17/PortWrite_MCP23S17.ino
rename to unit_tests/PortMCP23S17_write/PortMCP23S17_write.ino
index 4aa1566..2bb4307 100644
--- a/unit_tests/PortWrite_MCP23S17/PortWrite_MCP23S17.ino
+++ b/unit_tests/PortMCP23S17_write/PortMCP23S17_write.ino
@@ -1,29 +1,27 @@
-/* unit test for PortWrite_MCP23S17
-Picture of hardware is in unit_tests/PortRead_MCP23S17/PortRead_MCP23S17_bb.JPG
+/* unit test for PortMCP23S17
+Picture of hardware is in unit_tests/PortMCP23S17_read/PortMCP23S17_bb.JPG
 The setup is an MCP23S17 I/O expander on a Teensy LC controller.
 MCP23S17 port-A GPIO pins are not connected to anything.
 Port-A GPIO-pin ouputs alternate between 0 and 3.3 volts.
 
 Use a volt meter to measure port-A GPIO-pin outputs.
-MCP23S17 on 3.3v does not output enough power to reliable light LEDs
-    LED lights w/o resistor
-    LED not light with 56 ohm resistor
+OR low-voltage LED, with forward voltage less than 2 volts.
 */
 #include "PortIOE.h"
-#include "PortWrite_MCP23S17.h"
+#include "PortMCP23S17.h"
 
 const uint8_t PortIOE::DEVICE_ADDR = 0x20;      //MCP23S17 address, all 3 ADDR pins are grounded
-PortIOE portA(0, 0);
+PortIOE portA(0);
 
-PortWrite_MCP23S17 portAWrite(portA);           //PortAWrite needed for begin()
+PortMCP23S17 portAWrite(portA, 0);              //PortAWrite needed for begin()
 
 //const uint8_t GPIOA = 0x12;                     //LEDs are on port A
 
 void setup()
 {
     delay(6000);
-    portAWrite.begin();
-    //Keyboard.print("start blinking");
+    portAWrite.begin(LOW);                     //HIGH or LOW, not matter if readPins=0
+    Keyboard.print("start writing");
 }
 
 void loop()