All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: "John G. Heim" <jheim@math.wisc.edu>
Cc: <linux-kernel@vger.kernel.org>
Subject: Re: speakup bug
Date: Sat, 3 Mar 2012 17:39:16 +0000	[thread overview]
Message-ID: <20120303173916.7eca9391@pyramind.ukuu.org.uk> (raw)
In-Reply-To: <BFFDBC9B6BA14BE1AC8E266A3A456A63@hinkle>

On Sat, 3 Mar 2012 11:18:12 -0600
"John G. Heim" <jheim@math.wisc.edu> wrote:

> I need help fixing a bug in the driver for serial hardware  speech synths in 
> the speakup screen reader. According to the comments in the code, it is in a 
> part of the code that is trying to "steal" the serial port.

Yes - and the code is broken. To start with it's assumig a legacy PC
serial port at 0x3F8 and that it can beat the serial layer to it.

> returns an error code. But it looks as if the region for a serial port, 
> 0x3f8 - 0x3ff,  in ioport_resource cannot be reserved because the entire 
> range from 0x000 through 0xcf7 is already taken by something named "PCI Bus 
> 0000:00".  Therefore calling request_resource always fails and the driver 
> for the speech synth errors out.

It's a heirarchical space, so you can allocate things within it. Look
at /proc/ioports.

> And therefore I can't use my hardware speech synth without modifying the 
> kernel code.  If you comment out the line that checks the return code from 
> request_region, it works. So you have to modify  the kernel code and compile 
> a custom kernel to use a hardware speech synth. That's not such a problem 
> for me but it is for a lot of people. Plus, the grml live CD doesn't work 
> with hardware speech.  That is a problem for me.
> 
> Can anyone tell me how to fix this so it can be patched in the official 
> kernel code?

The proper fix is to make the drivers work via the serial layer properly.
The speakup people have been told this repeatedly for years and years
which is why their drivers work on less and less systems and won't run
with things PCI or USB serial ports, and why they are forever buried in
staging.

Alan

  reply	other threads:[~2012-03-03 17:37 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-03 17:18 speakup bug John G. Heim
2012-03-03 17:39 ` Alan Cox [this message]
2012-03-03 22:01   ` Ted Ts'o
2012-03-04  0:01   ` John G. Heim
2012-03-04  0:18     ` Samuel Thibault
2012-03-04  0:21       ` Samuel Thibault
2012-03-04  0:41         ` Ted Ts'o
2012-03-04  0:43           ` Samuel Thibault
2012-03-04  0:44             ` Samuel Thibault
2012-03-04  1:41             ` Ted Ts'o
2012-03-04  0:26     ` Ted Ts'o
2012-03-04  2:03   ` John G. Heim
2012-03-04 21:06     ` Alan Cox

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120303173916.7eca9391@pyramind.ukuu.org.uk \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=jheim@math.wisc.edu \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.