Hello, Okash Khawaja, on Wed 16 Nov 2016 05:36:19 +0000, wrote: > Looks like there are some ideas on what to do. Is there any piece of work that I can take up? AFAIK, there has been no real code written. I've however digged up my mails with Shraddha a bit, to find this: “ Testing with all actual synthesis hardware will be needed at some point, yes, but most of the work can be done with the dummy driver. Then we can ask the speakup community to make tests with actual hardware. I don't think you need actual hardware (I have myself actually worked on speakup for a long time without any particular hardware). The I/O operations are already very well factorized (some drivers do some I/O by hand, but we can forget about them in a first step), into the following functions used by the drivers: - spk_serial_synth_probe - spk_serial_out - spk_synth_immediate To make things go smoothly, I'd say we should add equivalents of those functions that would use proper kernel APIs, testing them with the dummy driver. Then, for each driver, we can have it tested with the new equivalents (that would be a matter of switching the probe and synth_immediate methods of the driver), and once verified by users with the actual hardware, commit the switch to the equivalents. You will notice calls to spk_serial_out() in spk_do_catch_up() and spk_synth_flush(). Actually the very first step of your work could be to add a serial_out() method to drivers, which for now would be set to spk_serial_out() in all drivers, and make spk_do_catch_up() and spk_synth_flush() call the method instead of spk_serial_out(). Direct calls to spk_serial_out() within drivers could be converted into calling the method too, so that switching methods will be trivial. About spk_stop_serial_interrupt, it could be made to return immediately if spk_serial_init was never called. > And also I'll have to convert spk_serial_in() too right? Ah, I missed that one while checking functions. Yes, just like spk_serial_out. ” Samuel _______________________________________________ Speakup mailing list Speakup@xxxxxxxxxxxxxxxxx http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup
