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
