[BRLTTY] Braillex EL 66S problem

[Dave Mielke]

[quoted lines by Delaunay Christophe on 2009/07/15 at 11:11 +0200]

>First, the display has been changed by the manufacturer. In its 125
>version I have, it has no status cell, 66 regular cells with only one
>routing key each, the usual easy access bar and two buttons only: one at
>the left of the bar, and one at its right.

Thanks. That matches our description.

>I did not understand all of what was placed in the attached log file.

No problem. That's the data sent by the EL66S to BRLTTY. Only people crazy 
enough to write drivers try to figure that kind of stuff out. :-)

>Since I noticed that the patch only modified two .c files, I did not
>./configure before doing make.

That's fine.

>>For the test, I'd like you to press the first primary routing key,
>
>Ok. But, as soon as I did this, I noticed that the display was filled
>with spurious unreadable garbage in its right side. The login prompt on
>the left was not affected.

That's okay for a login prompt. Those are the escape sequences for the arrow 
keys sent by brltty as it was trying to move the cursor to a place where it 
couldn't go.

Please try the attached patch (pm-bar-1.patch) to see if it fixes the bar.

Also, can you test and confirm that the rear and front of both keys work 
correctly?

-- 
Dave Mielke           | 2213 Fox Crescent | The Bible is the very Word of God.
Phone: 1-613-726-0014 | Ottawa, Ontario   | 2011 May 21 is the Day of Judgement.
EMail: dave at mielke.cc | Canada  K2A 1H7   | 2011 Oct 21 is the End of the World.
http://FamilyRadio.com/                   | http://Mielke.cc/bible/
-------------- next part --------------
Index: Drivers/Braille/Papenmeier/braille.c
===================================================================
--- Drivers/Braille/Papenmeier/braille.c	(revision 4685)
+++ Drivers/Braille/Papenmeier/braille.c	(working copy)
@@ -1127,6 +1127,7 @@
     }
   }
 
+  module.bit = 0;
   nextInputModule2(&module, 8);
   addInputMapping2(&module, 0, PM_SET_NavigationKeys, PM_KEY_BarUp2);
   addInputMapping2(&module, 1, PM_SET_NavigationKeys, PM_KEY_BarUp1);