Re: Staging status of speakup

2019-03-15 Thread Chris Brannon
Okash Khawaja writes: > Finally there is an issue where text in output buffer sometimes gets > garbled on SMP systems, but we can continue working on it after the > driver is moved out of staging, if that's okay. Basically we need a > reproducer of this issue. What kind of reproducer do you need

Re: [HELP REQUESTED from the community] Was: Staging status of speakup

2019-07-24 Thread Chris Brannon
Gregory Nowak writes: > keymap > I believe this is the currently active kernel keymap. I'm not sure of > the format, probably what dumpkeys(1) and showkey(1) use. Echoing > different values here should allow for remapping speakup's review > commands besides remapping the keyboard as a whole. AFA

Re: [patch 1/3] speakup: extend synth buffer to 16bit unicode characters

2017-03-03 Thread Chris Brannon
ngs. > synth_putwc, synth_putwc_s, synth_putws and synth_putws_s helpers are > however added to put 16bit characters and strings. > > Signed-off-by: Samuel Thibault Reviewed-by: Chris Brannon Looks good to me. ___ devel m

Re: [patch 3/3] speakup: add unicode variant of /dev/softsynth

2017-03-03 Thread Chris Brannon
e Linux only supports 16bit characters, > we can just hardcode the UTF-8 encoding. > > Signed-off-by: Samuel Thibault Reviewed-by: Chris Brannon Looks good to me. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject

Re: [PATCHv2 1/2] speakup: make get_char actually get unicode characters

2017-03-13 Thread Chris Brannon
t; VC. This fixes by just turning the use_unicode parameter to 1. > > Signed-off-by: Samuel Thibault > Tested-by: Zahari Yurukov Reviewed-by: Chris Brannon ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCHv2 2/2] speakup: Support spelling unicode characters

2017-03-14 Thread Chris Brannon
Samuel Thibault writes: > This supports spelling unicode characters by just passing them to > the speech synthesis in direct mode. > > Signed-off-by: Samuel Thibault > > Index: linux-4.10/drivers/staging/speakup/main.c > === > --- l

Re: [PATCH] speakup: Fix bogus caps on speaking unicode characters

2017-03-14 Thread Chris Brannon
Samuel Thibault writes: > commit 33590c185299 ("speakup: Support spelling unicode characters") > mistakenly passed unicode characters to IS_CHAR(), which only > supports latin1. Let's assume non-latin1 is lower case, like is done > in spell_word().

Re: [PATCHv2 2/2] speakup: Support spelling unicode characters

2017-03-14 Thread Chris Brannon
Samuel Thibault writes: > This supports spelling unicode characters by just passing them to > the speech synthesis in direct mode. > > Signed-off-by: Samuel Thibault Reviewed-by: Chris Brannon ___ devel mailing list de...@linuxdriverproj

Re: [PATCH v4 4/7] staging: speakup: replace spk_strlwr() with strlcpytolower()

2016-07-22 Thread Chris Brannon
Markus Mayer writes: > After introducing generic strltolower() and strtolower(), spk_strlwr() > is no longer needed. > > Signed-off-by: Markus Mayer > Acked-by: Samuel Thibault Acked-by: Chris Brannon Haven't looked at much kernel code in a while, but this l

Re: [PATCH] [STYLE]staging:MAINTAINERS email revision speakup

2016-11-12 Thread Chris Brannon
Walt Feasel writes: > Modified email address per the TODO file in > speakup's email listing, also verified email > address from speakup's website NAK. The website needs updating, because it has Kirk's old addresses. k...@reisers.ca is correct. ___ dev

Re: [PATCH v2 3/3] staging: speakup: TODO Correct email

2016-11-19 Thread Chris Brannon
Walt Feasel writes: > Make email correction for k...@reisers.ca > > Signed-off-by: Walt Feasel Acked-by: Chris Brannon > --- > v2 makes changes to correct for email format patch submission > > drivers/staging/speakup/TODO | 2 +- > 1 file changed, 1 insertion(+)

Re: [PATCH] staging: speakup: Fix warning reported by checkpatch

2015-09-01 Thread Chris Brannon
Aleksei Fedotov writes: > This patch fixes the checkpatch.pl warnings: > WARNING: Block comments use a trailing */ on a separate line > WARNING: Block comments use * on subsequent lines > > Signed-off-by: Aleksey Fedotov Acked-by: Chris Brannon Seems fine t

Re: [PATCH 1/1] staging/speakup/kobjects.c: Code improvement.

2013-09-10 Thread Chris Brannon
Raphael S Carvalho writes: > Wouldn't the following code (right before the statement: if > (param->var_id == VOICE)) > check if value is out of range? > > value = simple_strtol(cp, NULL, 10); > ret = spk_set_num_var(value, param, len); > if (ret == -ERANGE) { > var_data = param->data; > pr_warn(

Re: [PATCH 1/1] staging/speakup/kobjects.c: Code improvement.

2013-09-10 Thread Chris Brannon
"Raphael S.Carvalho" writes: > + /* > + * If voice was just changed, we might need to reset our default > + * pitch and volume. > + */ > + if (param->var_id == VOICE) { > + spk_reset_default_value("pitch", synth->defa

Re: [PATCH 1/1] staging/speakup/kobjects.c: Code improvement.

2013-09-10 Thread Chris Brannon
Dan Carpenter writes: > Good eye for spotting the memory corruption bug! > > This is a bug fix, so the fix should go in a separate patch and not > merged with a code cleanup patch. Ordinary users can trigger this so > it's a security bug and separating it out is extra important. Ok. I just sen