Read this first :) Re: [patch 0/6] staging: speakup: Introduce TTY-based comms
Samuel Thibault
samuel.thibault at ens-lyon.org
Sat Feb 25 22:38:11 EST 2017
So, just a last mail, which could actually be read first :)
First, I notice that it took you 12 minutes to send the patch series.
This shows that you are not using quilt to do this :) Really, take the
time to look up how "quilt mail" works, and notably its --send option,
it will save you an awful lot of time (notably since I'm asking to
split the series is rather small patches, which is preferred thing when
pushing to Linux)
I also gave a lot of information, most of which are mostly notes for
later. I'd say that the priority is this:
- work on the comments I made on the actual code (patch splitting,
migrating ltlk, not migrating dectlk, moving spk_synth_immediate,
exporting spk_stop_serial_interrupt, taking the ser parameter into
account, renaming spk_ttyio_immediate to spk_ttyio_synth_immediate,
and putting serial_out in a separate spk_io_ops structure)
- maybe post a last round of patches here, so I make a last review
before we push to linux-kernel
- then we push to linux-kernel and get flames :)
- in the meanwhile, you can work on adding a "char *dev" synth parameter
which would replace the "ser" parameter, and writing a function which
translates it into an MKDEV(major,minor) pair (first implement it for
"ttyS*", then for "ttyUSB*", etc.)
- hopefully at that point we can get that series commited mainstream,
even if we only migrate the trivial drivers.
- then we can work on the other drivers as my mail series suggests.
Samuel
More information about the Speakup
mailing list