Linux Serial subsystem development
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Joshua Droney <josh@fastcomproducts.com>
Cc: linux-serial@vger.kernel.org
Subject: Re: Incorrect UART in /tty/serial/8250
Date: Fri, 29 Mar 2024 07:01:31 +0100	[thread overview]
Message-ID: <2024032920-replay-ruined-d92d@gregkh> (raw)
In-Reply-To: <CAJqSBBNc5mPcKfFSH2jXnjp9PB6fQGWN0vB9JPKHENogUw-D+A@mail.gmail.com>

On Thu, Mar 28, 2024 at 02:34:10PM -0500, Joshua Droney wrote:
> Hello!
> 
> My name is Josh. I'm currently employed at Commtech, Inc.
> (https://fastcomproducts.com/). I'm following the instructions at
> https://kernelnewbies.org/FoundBug to report this issue. Please let me
> know if this is incorrect.
> 
> We (Commtech) have several serial boards, but the board in question is
> the '422/2-PCIe'. This board does not exist, but someone added it to
> the Linux kernel at some point in the past. This wouldn't be a
> problem, except that the board ID that was used for this non-existent
> board (0x22) was ultimately used for a different REAL board, with a
> different UART. Instead of a xr172358, we use a 16C950 on the real
> board. But the Linux kernel finds our other board ID, and tries to
> initialize it as an exar chip and fails. I don't know much about the
> kernel, but I believe the lines in question are:
> 
> https://github.com/torvalds/linux/blob/master/drivers/tty/serial/8250/8250_exar.c#L47
> https://github.com/torvalds/linux/blob/master/drivers/tty/serial/8250/8250_exar.c#L925
> 
> You can see that we have a different board with device ID 0x22 here:
> https://admin.pci-ids.ucw.cz/read/PC/18f7
> 
> The 422/4-PCIe exists (0x20), the 422/8-PCIe exists (0x21), but the
> 422/2-PCIe was never created and its device id was used for a
> different board (0x22, the SuperFSCC/4-LVDS-PCIe).

Can you please submit a patch that fixes this?  We'll be glad to take
it.  If you need help making it, please let us know.

thanks,

greg k-h

  reply	other threads:[~2024-03-29  6:01 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-28 19:34 Incorrect UART in /tty/serial/8250 Joshua Droney
2024-03-29  6:01 ` Greg KH [this message]
2024-03-29 14:23   ` Joshua Droney
2024-04-03 22:26     ` Andy Shevchenko

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=2024032920-replay-ruined-d92d@gregkh \
    --to=gregkh@linuxfoundation.org \
    --cc=josh@fastcomproducts.com \
    --cc=linux-serial@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox