Received: by 2002:ac0:a5b6:0:0:0:0:0 with SMTP id m51-v6csp3521794imm; Sun, 17 Jun 2018 22:35:44 -0700 (PDT) X-Google-Smtp-Source: ADUXVKJ1wR+dcEaGe2ITnhlFaV6oepz52SjEyVM0+QGw10/C+eIwS+yZeYYAEI1zNviei99/dIMh X-Received: by 2002:a17:902:5501:: with SMTP id f1-v6mr12476656pli.108.1529300144506; Sun, 17 Jun 2018 22:35:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1529300144; cv=none; d=google.com; s=arc-20160816; b=h56BiHODgvnFbS+5XBWpwTzWNNmN2xQcs0CODXvBJuEGNh8y3Am/9ZEBhcHwdgq4cG 3njsXu2Bd9GP2jgUvTnqOvrHaOMoTUEPuSdnPxFfu34jxfUxNDjjSE9KOWVfTkXEpdh/ O9y5POUB8GJmK4eTfc8EogLDHnhbvYM/CU85JF7c/skiu3aFEioqQily89w90p0cOVnv Q6zPnm6yicho60Ldxl3DAIh+ulOBF4v6isoyp/dx4HQsPkX6w65qo3Xr4rXg7gO/+mk7 3aExP+C5qqxCFea+tpFf07ACrF/XKZf9NB/wt/Q1GF2dzE81Rlafn56sh3yUIJieGsvs uwUg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:to :from:date:dkim-signature:arc-authentication-results; bh=HnmetGA9tuo8pxXkCJJyRdA4X8bVa/Q+Q2Fv3DYXAlo=; b=N8tRFYfnEfnTJHVjOUx6w3LMUYd5Ama6nivV4Vul05OOX4PxFsKqS1oYKKpRPIu6bL OtRJqEx1xz+agI/NkQXsF6GRLzw1+oeRYuVwTJ4d0glWtXAOKV+MwH7QPwqiu7RTX7sM Mr0JoiLJ5raPQaiCXrkGxgZS/6Q+ktXa8iAVra+YJxoeJrGhpHE0Wb9kTa2LLwCV0WXB 6mQ0qeY1LdYPZDs9He78H9rVW2zOHEtAK87guYegE3gMigF7isTSbyrYLmjpyEDtqKQ6 hCp6MzGM61M9UCYdgOVHDU2Xi3Ep7saT+wYxm3WZwHs0HvVcUL2O+v1/XkW6nNfzx2Zu Mw2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@gregn.net header.s=default header.b=Xfbl7umq; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j61-v6si14172328plb.68.2018.06.17.22.35.29; Sun, 17 Jun 2018 22:35:44 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@gregn.net header.s=default header.b=Xfbl7umq; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754283AbeFRFeu (ORCPT + 99 others); Mon, 18 Jun 2018 01:34:50 -0400 Received: from vserver.gregn.net ([174.136.110.154]:37132 "EHLO vserver.gregn.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752361AbeFRFet (ORCPT ); Mon, 18 Jun 2018 01:34:49 -0400 Received: from vbox.gregn.net (unknown [IPv6:2001:470:d:6c5:dccc:19d1:b333:b8a8]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by vserver.gregn.net (Postfix) with ESMTPSA id 5BED422DA; Sun, 17 Jun 2018 22:35:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=gregn.net; s=default; t=1529300119; bh=pOA7Phcl9YSRIPc+RMMCmwSir5FCHjLqAit2LEqUeoY=; h=Date:From:To:Subject:References:In-Reply-To:From; b=Xfbl7umqrN1dKBXC7g9ycCuzXnfravJ0T6WbODVcg+qRSjhulC/60A72Q09Fcs0MA 3u0STLzHxoIvHX7rPLAt+Xi6Ff8WIokih2WoEA0q7il9zbaQpLwDzfuWW2jfkgIwFn IT9Y0wiOrCiVXIdxEpp21QUwVaNgRPv7N5Og3id39JPH9fhY7Hr2gIn2S1WpXyABMD yyE81qogwGHAM9MTRRrYcrOElG45s9WMCTczfXPChmBBKNxNy6JwfDfUlcZMit7GiR KPGv9KGZnqapi4akuifRO0SBZa1YjKy0OOplquAOCvyoD5yuQhs3eCjt0NpW4lFv6Y qVRI6YvDfDnZA== Received: from greg by vbox.gregn.net with local (Exim 4.84_2) (envelope-from ) id 1fUmoO-0001pZ-Sw; Sun, 17 Jun 2018 22:34:44 -0700 Date: Sun, 17 Jun 2018 22:34:44 -0700 From: Gregory Nowak To: Samuel Thibault , "Speakup is a screen review system for Linux." , devel@driverdev.osuosl.org, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: speakup: refactor synths array to use a list Message-ID: <20180618053444.GA1614@gregn.net> References: <20180604095212.GA18381@tanglefoot> <20180606132628.fxykchbznrwzgwqt@var.youpi.perso.aquilenet.fr> <20180611225703.csi47cdafhy6rxcf@var.youpi.perso.aquilenet.fr> <20180611235122.GA28343@gregn.net> <20180612063105.7urgjfckrwjvqs6k@var.youpi.perso.aquilenet.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180612063105.7urgjfckrwjvqs6k@var.youpi.perso.aquilenet.fr> X-PGP-Key: http://www.gregn.net/pubkey.asc User-Agent: Mutt/1.5.23 (2014-03-12) X-Virus-Scanned: clamav-milter 0.99.4 at vserver X-Virus-Status: Clean Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 12, 2018 at 08:31:06AM +0200, Samuel Thibault wrote: > The load/unload is about the module itself, i.e. modprobe speakup_bns ; > modprobe speakup_soft, switch between them, then rmmod speakup_bns ; > speakup_soft or the converse (to exercise both orders). # uname -a Linux p41box 4.17.1 #1 SMP Sat Jun 16 11:19:57 MST 2018 i686 GNU/Linux # lsmod |grep "speakup" speakup_bns 16384 0 speakup_soft 16384 1 speakup 94208 3 speakup_bns,speakup_soft With /sys/accessibility/speakup/synth set to bns, I am getting output alternately from the bns and from soft. It's as if speakup can't make up its mind which synthesizer is being used. When I echo soft >/sys/accessibility/speakup/synth, I get no speech at all from either synthesizer. Doing rmmod of all three speakup modules comes back with no errors. There is also no unusual output in dmesg, I can see both synthesizers being registered and unregistered as I switch between them. I can also reproduce this behavior with speakup_soft, and speakup_dummy specifically: 1. modprobe speakup_soft and modprobe speakup_dummy 2. The synthesizer should now be set to dummy in /sys/accessibility/speakup/synth. 3. Use the speakup review keys, press enter a number of times. You should observe output from both the software speech, and from the serial port alternating between each other. 4. echo soft >/sys/accessibility/speakup/synth 5. You should observe no output from either software speech or the serial port as you use speakup review keys, or press enter repeatedly. 6. echo dummy >/sys/accessibility/speakup/synth 7. You should alternately get speech from the software synthesizer and from the serial port. I built my kernel from the 4.17.1 kernel.org sources, and the patch that Samuel reposted applied cleanly with no errors. Greg -- 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@EU.org