From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by befuddled.reisers.ca (Postfix, from userid 65534) id BF15D1F0A2A; Tue, 14 Mar 2017 09:45:22 -0400 (EDT) Received: from mail-wr0-x241.google.com (mail-wr0-x241.google.com [IPv6:2a00:1450:400c:c0c::241]) by befuddled.reisers.ca (Postfix) with ESMTPS id 3BE401F0A24 for ; Tue, 14 Mar 2017 09:45:20 -0400 (EDT) Received: by mail-wr0-x241.google.com with SMTP id u108so24003338wrb.2 for ; Tue, 14 Mar 2017 06:45:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:user-agent:date:from:to:cc:subject:references :mime-version:content-disposition; bh=Ydld1Qbn5R2Qg8L4WJI/aOQbt0h4VoU6fqVSNn0bqBk=; b=lrAkg95ZAP2aonXOxvXWOOC/gQ2XxH0sR7l/LKsQeYSoYPYMiBl9wJYUcnh6lu8i0Q sEFEBF233WifA3tto8gMXXBTSlg8kELgyBhldP1WPc9Ac8TsxUqVGY564YG7aRWlL3cb wecO8sXeP/nO2r4E5dwWNs8pDmkUYbPl+/mPkAsMy67oz8jAtXcF89q4szqL7IbCYyFr fE5SfKJGVfNmHdI5+/SQAnVlFOGdc9/AeXPQWpNcm2EqNxKyqnBiedeiHrCR2hZZy8mX +pc4Z4obA16zf4I23EbHmlxsbcqwQfPGrcsLw/FMtTSoGISdThGGggKhvqoIil1FF+eq yVhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:user-agent:date:from:to:cc:subject :references:mime-version:content-disposition; bh=Ydld1Qbn5R2Qg8L4WJI/aOQbt0h4VoU6fqVSNn0bqBk=; b=GmGbKFqM5AEcVJoKfbdYmYIB4skvSKxadROk4y/ucSYxXbGOB8Cob8myM4QwKIKSp3 sBNvseI2f6FomQj+XsLX7ZZjFxVbUz3/HtZI43CRpqn+U/8/Prs+FAR4MkBwEcRWtCPs kpLoa/UzC/vwGNdJUEovI0F4df50FHoIMAp/Y2cn49HxE3iinUlDLwHTRhq6RU435nTn KS0fDOr+/pz0oNXfSklx+BRM15CXwAeQfBlr8vv6UISASSnHJgA8EE4EW/nEayM+DFoH r4sguIIRyC1YtrOnuJNbIhVPKLVP/UVzGEyipStBma6SMvR3yaWFZnSuc4mPH3dpvgLH 1bNQ== X-Gm-Message-State: AMke39ksxb3SPBtkYFaxi13jMHp4UWFaurMbh/E3iN5A+g8tYM2BTNbZOTZ/qYX7FcvwgA== X-Received: by 10.223.176.7 with SMTP id f7mr34053091wra.124.1489499119312; Tue, 14 Mar 2017 06:45:19 -0700 (PDT) Received: from sanghar (188.29.165.27.threembb.co.uk. [188.29.165.27]) by smtp.gmail.com with ESMTPSA id i133sm15596030wmg.26.2017.03.14.06.45.18 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 14 Mar 2017 06:45:18 -0700 (PDT) Message-Id: <20170314134517.395770078@gmail.com> User-Agent: quilt/0.63-1 Date: Tue, 14 Mar 2017 13:41:52 +0000 From: Okash Khawaja To: Greg Kroah-Hartman , Samuel Thibault , linux-kernel@vger.kernel.org Cc: William Hubbs , Chris Brannon , Kirk Reiser , speakup@linux-speakup.org, devel@driverdev.osuosl.org, Okash Khawaja Subject: [patch 1/4] staging: speakup: spk_serial_out and spk_wait_for_xmitr to take synth arg References: <20170314134151.603195427@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Disposition: inline; filename=01_spk_serial_out_and_xmitr_to_take_synth X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 X-BeenThere: speakup@linux-speakup.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Speakup is a screen review system for Linux." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 14 Mar 2017 13:45:23 -0000 These two functions are always called from a context where spk_synth instance is available. They also use the spk_synth instance but instead of taking it as an argument, they rely on a global spk_synth instance inside synth.c which points to the same synth as the one being passed in as argument. Signed-off-by: Okash Khawaja Reviewed-by: Samuel Thibault Index: linux-4.11-rc2/drivers/staging/speakup/serialio.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/serialio.c +++ linux-4.11-rc2/drivers/staging/speakup/serialio.c @@ -144,14 +144,14 @@ free_irq(serstate->irq, (void *)synth_readbuf_handler); } -int spk_wait_for_xmitr(void) +int spk_wait_for_xmitr(struct spk_synth *in_synth) { int tmout = SPK_XMITR_TIMEOUT; - if ((synth->alive) && (timeouts >= NUM_DISABLE_TIMEOUTS)) { + if ((in_synth->alive) && (timeouts >= NUM_DISABLE_TIMEOUTS)) { pr_warn("%s: too many timeouts, deactivating speakup\n", - synth->long_name); - synth->alive = 0; + in_synth->long_name); + in_synth->alive = 0; /* No synth any more, so nobody will restart TTYs, and we thus * need to do it ourselves. Now that there is no synth we can * let application flood anyway @@ -162,7 +162,7 @@ } while (spk_serial_tx_busy()) { if (--tmout == 0) { - pr_warn("%s: timed out (tx busy)\n", synth->long_name); + pr_warn("%s: timed out (tx busy)\n", in_synth->long_name); timeouts++; return 0; } @@ -207,9 +207,9 @@ } EXPORT_SYMBOL_GPL(spk_serial_in_nowait); -int spk_serial_out(const char ch) +int spk_serial_out(struct spk_synth *in_synth, const char ch) { - if (synth->alive && spk_wait_for_xmitr()) { + if (in_synth->alive && spk_wait_for_xmitr(in_synth)) { outb_p(ch, speakup_info.port_tts); return 1; } Index: linux-4.11-rc2/drivers/staging/speakup/speakup_apollo.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/speakup_apollo.c +++ linux-4.11-rc2/drivers/staging/speakup/speakup_apollo.c @@ -168,7 +168,7 @@ set_current_state(TASK_INTERRUPTIBLE); full_time_val = full_time->u.n.value; spin_unlock_irqrestore(&speakup_info.spinlock, flags); - if (!spk_serial_out(ch)) { + if (!spk_serial_out(synth, ch)) { outb(UART_MCR_DTR, speakup_info.port_tts + UART_MCR); outb(UART_MCR_DTR | UART_MCR_RTS, speakup_info.port_tts + UART_MCR); @@ -181,7 +181,7 @@ full_time_val = full_time->u.n.value; delay_time_val = delay_time->u.n.value; spin_unlock_irqrestore(&speakup_info.spinlock, flags); - if (spk_serial_out(synth->procspeech)) + if (spk_serial_out(synth, synth->procspeech)) schedule_timeout(msecs_to_jiffies (delay_time_val)); else @@ -194,7 +194,7 @@ synth_buffer_getc(); spin_unlock_irqrestore(&speakup_info.spinlock, flags); } - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); } module_param_named(ser, synth_apollo.ser, int, 0444); Index: linux-4.11-rc2/drivers/staging/speakup/speakup_audptr.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/speakup_audptr.c +++ linux-4.11-rc2/drivers/staging/speakup/speakup_audptr.c @@ -135,7 +135,7 @@ udelay(1); } outb(SYNTH_CLEAR, speakup_info.port_tts); - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); } static void synth_version(struct spk_synth *synth) Index: linux-4.11-rc2/drivers/staging/speakup/speakup_decext.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/speakup_decext.c +++ linux-4.11-rc2/drivers/staging/speakup/speakup_decext.c @@ -185,7 +185,7 @@ spin_unlock_irqrestore(&speakup_info.spinlock, flags); if (ch == '\n') ch = 0x0D; - if (synth_full() || !spk_serial_out(ch)) { + if (synth_full() || !spk_serial_out(synth, ch)) { schedule_timeout(msecs_to_jiffies(delay_time_val)); continue; } @@ -199,10 +199,10 @@ in_escape = 0; else if (ch <= SPACE) { if (!in_escape && strchr(",.!?;:", last)) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); if (time_after_eq(jiffies, jiff_max)) { if (!in_escape) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); spin_lock_irqsave(&speakup_info.spinlock, flags); jiffy_delta_val = jiffy_delta->u.n.value; @@ -217,7 +217,7 @@ last = ch; } if (!in_escape) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); } static void synth_flush(struct spk_synth *synth) Index: linux-4.11-rc2/drivers/staging/speakup/speakup_dectlk.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/speakup_dectlk.c +++ linux-4.11-rc2/drivers/staging/speakup/speakup_dectlk.c @@ -250,7 +250,7 @@ spin_unlock_irqrestore(&speakup_info.spinlock, flags); if (ch == '\n') ch = 0x0D; - if (synth_full_val || !spk_serial_out(ch)) { + if (synth_full_val || !spk_serial_out(synth, ch)) { schedule_timeout(msecs_to_jiffies(delay_time_val)); continue; } @@ -264,10 +264,10 @@ in_escape = 0; else if (ch <= SPACE) { if (!in_escape && strchr(",.!?;:", last)) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); if (time_after_eq(jiffies, jiff_max)) { if (!in_escape) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); spin_lock_irqsave(&speakup_info.spinlock, flags); jiffy_delta_val = jiffy_delta->u.n.value; @@ -282,17 +282,17 @@ last = ch; } if (!in_escape) - spk_serial_out(PROCSPEECH); + spk_serial_out(synth, PROCSPEECH); } static void synth_flush(struct spk_synth *synth) { if (in_escape) /* if in command output ']' so we don't get an error */ - spk_serial_out(']'); + spk_serial_out(synth, ']'); in_escape = 0; is_flushing = 1; - spk_serial_out(SYNTH_CLEAR); + spk_serial_out(synth, SYNTH_CLEAR); } module_param_named(ser, synth_dectlk.ser, int, 0444); Index: linux-4.11-rc2/drivers/staging/speakup/spk_priv.h =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/spk_priv.h +++ linux-4.11-rc2/drivers/staging/speakup/spk_priv.h @@ -42,10 +42,10 @@ const struct old_serial_port *spk_serial_init(int index); void spk_stop_serial_interrupt(void); -int spk_wait_for_xmitr(void); +int spk_wait_for_xmitr(struct spk_synth *in_synth); unsigned char spk_serial_in(void); unsigned char spk_serial_in_nowait(void); -int spk_serial_out(const char ch); +int spk_serial_out(struct spk_synth *in_synth, const char ch); void spk_serial_release(void); char synth_buffer_getc(void); Index: linux-4.11-rc2/drivers/staging/speakup/synth.c =================================================================== --- linux-4.11-rc2.orig/drivers/staging/speakup/synth.c +++ linux-4.11-rc2/drivers/staging/speakup/synth.c @@ -119,7 +119,7 @@ spin_unlock_irqrestore(&speakup_info.spinlock, flags); if (ch == '\n') ch = synth->procspeech; - if (!spk_serial_out(ch)) { + if (!spk_serial_out(synth, ch)) { schedule_timeout(msecs_to_jiffies(full_time_val)); continue; } @@ -129,7 +129,7 @@ delay_time_val = delay_time->u.n.value; full_time_val = full_time->u.n.value; spin_unlock_irqrestore(&speakup_info.spinlock, flags); - if (spk_serial_out(synth->procspeech)) + if (spk_serial_out(synth, synth->procspeech)) schedule_timeout( msecs_to_jiffies(delay_time_val)); else @@ -142,7 +142,7 @@ synth_buffer_getc(); spin_unlock_irqrestore(&speakup_info.spinlock, flags); } - spk_serial_out(synth->procspeech); + spk_serial_out(synth, synth->procspeech); } EXPORT_SYMBOL_GPL(spk_do_catch_up); @@ -153,7 +153,7 @@ while ((ch = *buff)) { if (ch == '\n') ch = synth->procspeech; - if (spk_wait_for_xmitr()) + if (spk_wait_for_xmitr(synth)) outb(ch, speakup_info.port_tts); else return buff; @@ -165,7 +165,7 @@ void spk_synth_flush(struct spk_synth *synth) { - spk_serial_out(synth->clear); + spk_serial_out(synth, synth->clear); } EXPORT_SYMBOL_GPL(spk_synth_flush); @@ -180,7 +180,7 @@ { if (synth->alive) return 1; - if (spk_wait_for_xmitr() > 0) { + if (spk_wait_for_xmitr(synth) > 0) { /* restart */ synth->alive = 1; synth_printf("%s", synth->init);