From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: phasta@kernel.org
Cc: Guenter Roeck <linux@roeck-us.net>,
Jiri Slaby <jirislaby@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org
Subject: Re: [PATCH] tty: serial: Replace deprecated PCI API
Date: Wed, 17 Dec 2025 15:06:24 +0100 [thread overview]
Message-ID: <2025121710-audacious-slot-fa2f@gregkh> (raw)
In-Reply-To: <de540d7789d0e9f77efde05fab4705b8d97245d8.camel@mailbox.org>
On Thu, Dec 11, 2025 at 02:57:46PM +0100, Philipp Stanner wrote:
> On Wed, 2025-11-26 at 09:02 -0800, Guenter Roeck wrote:
> > On 11/26/25 01:10, Philipp Stanner wrote:
> > > pcim_iomap_table() is deprecated. Moreover, its special usage in 8250,
> > > causes a WARN_ON to fire (in pcim_add_mapping_to_legacy_table()).
> > >
> > > 8250's function serial8250_pci_setup_port() effectively maps the same
> > > BAR multiple times and adds an offset to the start address. While
> > > mapping and adding offsets is not a bug, it can be achieved in a far
> > > more straightforward way by using the specialized function
> > > pcim_iomap_range().
> > >
> > > pcim_iomap_range() does not add the mapping addresses to the deprecated
> > > iomap table - that's not a problem, however, because non of the users of
> > > serial8250_pci_setup_port() uses pcim_iomap_table().
> > >
> > > Replace the deprecated PCI functions with pcim_iomap_range().
> > >
> > > Cc: Guenter Roeck <linux@roeck-us.net>
> > > Link: https://lore.kernel.org/dri-devel/16cd212f-6ea0-471d-bf32-34f55d7292fe@roeck-us.net/
> > > Signed-off-by: Philipp Stanner <phasta@kernel.org>
> > > ---
> > > @Guenther: Can you please test this? I hope it fixes your issue.
> >
> > Yes, it does. Thanks a lot for fixing this!
> >
> > Tested-by: Guenter Roeck <linux@roeck-us.net>
>
> @Greg:
> Can you apply this?
Does not apply at all to 6.19-rc1 :(
Can you please rebase and resend?
thanks,
greg k-h
next prev parent reply other threads:[~2025-12-17 14:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-26 9:10 [PATCH] tty: serial: Replace deprecated PCI API Philipp Stanner
2025-11-26 17:02 ` Guenter Roeck
2025-12-11 13:57 ` Philipp Stanner
2025-12-17 14:06 ` Greg Kroah-Hartman [this message]
2025-12-17 16:11 ` Guenter Roeck
2025-12-18 9:02 ` Philipp Stanner
2025-12-18 9:30 ` Philipp Stanner
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=2025121710-audacious-slot-fa2f@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=jirislaby@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=peterz@infradead.org \
--cc=phasta@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.