All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joshua Droney <josh@fastcomproducts.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: linux-serial@vger.kernel.org
Subject: Re: Incorrect UART in /tty/serial/8250
Date: Fri, 29 Mar 2024 09:23:49 -0500	[thread overview]
Message-ID: <CAJqSBBMETm4UVv3F8iUAdMi2tpaSY+-QcxAO2qHOC_O9J=cSqw@mail.gmail.com> (raw)
In-Reply-To: <2024032920-replay-ruined-d92d@gregkh>

I would appreciate some guidance on this. I don't know enough about
the kernel's serial driver to be confident about what changes need to
be made, nor anything about creating/submitting a patch.

Thanks!
Josh

On Fri, Mar 29, 2024 at 1:01 AM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> 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 14:23 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
2024-03-29 14:23   ` Joshua Droney [this message]
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='CAJqSBBMETm4UVv3F8iUAdMi2tpaSY+-QcxAO2qHOC_O9J=cSqw@mail.gmail.com' \
    --to=josh@fastcomproducts.com \
    --cc=gregkh@linuxfoundation.org \
    --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 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.