Hey Greg, Thanks for that. You're right, acntpc shouldn't have been affected. I will investigate this today. Cheers, Okash > On 14 Jun 2017, at 05:50, Gregory Nowak <greg@xxxxxxxxx> wrote: > > Okash and list, I tried building the modules from your repo, and am > running into a compilation error. I'll describe what I did. I'm > building on the same machine and in the same environment that I used > for my previous speakup testing. This is on a devuan 1.0 system. > > 1. git clone https://github.com/bytefire/speakup-decext > The repo cloned without issues. > > 2. Since the speakup-decext.txt file still mentions using the 4.10.9 > kernel tree, I used the source I already had built previously, and > just got rid of drivers/staging/speakup. > > 3. Move speakup2.tgz to drivers/staging, and extract it. All went ok. > > 4. cd to the root of the kernel source, and run make modules: > > CHK include/config/kernel.release > CHK include/generated/uapi/linux/version.h > CHK include/generated/utsrelease.h > CHK include/generated/bounds.h > CHK include/generated/timeconst.h > CHK include/generated/asm-offsets.h > CALL scripts/checksyscalls.sh > CC [M] drivers/staging/speakup/speakup_acntpc.o > drivers/staging/speakup/speakup_acntpc.c:313:42: error: expected ) > before int > module_param_hw_named(port, port_forced, int, ioport, 0444); > ^ > scripts/Makefile.build:300: recipe for target > 'drivers/staging/speakup/speakup_acntpc.o' failed > make[3]: *** [drivers/staging/speakup/speakup_acntpc.o] Error 1 > scripts/Makefile.build:553: recipe for target > 'drivers/staging/speakup' failed > make[2]: *** [drivers/staging/speakup] Error 2 > scripts/Makefile.build:553: recipe for target 'drivers/staging' failed > make[1]: *** [drivers/staging] Error 2 > Makefile:988: recipe for target 'drivers' failed > make: *** [drivers] Error 2 > > What I find strange here is that the acntpc driver is for the internal > isa card, so it shouldn't have been effected by the serial changes, > correct? > > I don't need the acntpc module, so I could just remove it from the > config, and go on. However, others who build a kernel using debian's > default kernel config will have the module included, and will probably > run into this. Thanks. > > Greg > > >> On Sat, Jun 10, 2017 at 12:29:29PM +0100, Okash Khawaja wrote: >>> On Sat, Jun 10, 2017 at 12:24:10PM +0100, Okash Khawaja wrote: >>> Hi, >>> >>> These patches extend speakup support to ttyS* and lp*. They introduce a >>> new module param dev whose purpose is similar to ser but instead of >>> taking serial port number as argument, it takes strings like ttyS0 or >>> ttyUSB0. First patch just adds functionality to convert such strings >>> into dev_t. Second patch makes use of that functionlity. >> >> I have updated the speakup-decext repo [1] to latest, including bns >> migration and support for more than ttyS*. Please do test with 'dev' >> module param. >> >> One thing I noticed in my testing is that when loading with incorrect >> 'dev' value several times in a row, it eventually succeeds in loading. >> That seems like an existing issue with synths[] array which caches >> already loaded synths. I am investigating it now. >> >> [1] https://github.com/bytefire/speakup-decext >> >> Cheers! >> Okash >> _______________________________________________ >> Speakup mailing list >> Speakup@xxxxxxxxxxxxxxxxx >> http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup > > -- > web site: http://www.gregn.net > gpg public key: http://www.gregn.net/pubkey.asc > skype: gregn1 > (authorization required, add me to your contacts list first) > If we haven't been in touch before, e-mail me before adding me to your contacts. > > -- > Free domains: http://www.eu.org/ or mail dns-manager@xxxxxx > _______________________________________________ > Speakup mailing list > Speakup@xxxxxxxxxxxxxxxxx > http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup _______________________________________________ Speakup mailing list Speakup@xxxxxxxxxxxxxxxxx http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup
