On 05/10/2018 06:43 AM, andy pugh wrote:
On 10 May 2018 at 04:55, Jon Elson <el...@pico-systems.com> wrote:
I can't match this commit with
your 1b7feb83
That commit shows up as de7bbf4 in github.
1b7feb83
is
https://github.com/LinuxCNC/linuxcnc/commit/1b7feb839c15952d6ff867bcfc4cdea7f873976e
(You can search on github with the hash, but need to click on the
"commits" label on the screen that says "no code results")
OK, thanks to you and Rene, I new see the commit that
matches that hash.
And, I guess this is what Chris was pointing to:
+static int slotnum; /* made global so SelRead can see which
parport is being handled */
+ /* to deal with epp_dir option */
But, then I see there's a declaration of slotnum in
rtapi_app_main(), and this function calls
the export_encoder() function that sets the master and slave
condition of the boards.
So, yes, I think Chris had it, it just took me a day to
understand it.
Now, the REAL mess here is that slotnum is being used for
two totally different things.
in rtapi_app_main() it indicates which board is being
addresses on ONE parport.
But, for SelRead() is controls which PARPORT is being used!
Well, I think I now see what needs to be done.
But, I'm not sure this is going to be easy to test, as it
seems to work in a number of cases.
I guess one can just make the code right, and then see that
it works in as many cases as I can test.
Thanks, everybody for the help!
Jon
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Emc-developers mailing list
Emc-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/emc-developers
