mbox series

[00/11] sound: Replace deprecated PCI functions

Message ID 20250404121911.85277-2-phasta@kernel.org (mailing list archive)
Headers show
Series sound: Replace deprecated PCI functions | expand

Message

Philipp Stanner April 4, 2025, 12:19 p.m. UTC
pcim_iomap_table() and pcim_iomap_regions() have been deprecated by the
PCI subsystem. In sound/, they can easily be replaced with
pcim_iomap_region().

This series was around some time in late summer last year as a single
patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
requested that I split it by component.

So that's being done here. Feel free to squash the patches as you like.

P.

Philipp Stanner (11):
  ALSA: ad1889: Replace deprecated PCI functions
  ALSA: atiixp: Replace deprecated PCI functions
  ALSA: au88x0: Replace deprecated PCI functions
  ALSA: aw2: Replace deprecated PCI functions
  ALSA: bt87x: Replace deprecated PCI functions
  ALSA: cs4281: Replace deprecated PCI functions
  ALSA: hda_intel: Replace deprecated PCI functions
  ALSA: cs5530: Replace deprecated PCI functions
  ALSA: lola: Replace deprecated PCI functions
  ALSA: hdspm: Replace deprecated PCI functions
  ASoC: loongson: Replace deprecated PCI functions

 sound/pci/ad1889.c                    |  7 +++----
 sound/pci/atiixp.c                    |  7 +++----
 sound/pci/atiixp_modem.c              |  7 +++----
 sound/pci/au88x0/au88x0.c             |  7 +++----
 sound/pci/aw2/aw2-alsa.c              |  7 +++----
 sound/pci/bt87x.c                     |  7 +++----
 sound/pci/cs4281.c                    | 13 +++++++------
 sound/pci/cs5530.c                    |  7 +++----
 sound/pci/hda/hda_intel.c             |  7 +++----
 sound/pci/lola/lola.c                 | 16 +++++++++++-----
 sound/pci/rme9652/hdspm.c             |  7 +++----
 sound/soc/loongson/loongson_i2s_pci.c | 13 +++++++------
 12 files changed, 52 insertions(+), 53 deletions(-)

Comments

Takashi Iwai April 4, 2025, 1:47 p.m. UTC | #1
On Fri, 04 Apr 2025 14:19:01 +0200,
Philipp Stanner wrote:
> 
> pcim_iomap_table() and pcim_iomap_regions() have been deprecated by the
> PCI subsystem. In sound/, they can easily be replaced with
> pcim_iomap_region().
> 
> This series was around some time in late summer last year as a single
> patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
> requested that I split it by component.
> 
> So that's being done here. Feel free to squash the patches as you like.

Thanks for the patches.  I suppose those are no urgent fixes, and can
be postponed for 6.16?  If so, I'll pick them up after 6.15-rc1 merge
window is closed.


Takashi
Philipp Stanner April 4, 2025, 1:49 p.m. UTC | #2
On Fri, 2025-04-04 at 15:47 +0200, Takashi Iwai wrote:
> On Fri, 04 Apr 2025 14:19:01 +0200,
> Philipp Stanner wrote:
> > 
> > pcim_iomap_table() and pcim_iomap_regions() have been deprecated by
> > the
> > PCI subsystem. In sound/, they can easily be replaced with
> > pcim_iomap_region().
> > 
> > This series was around some time in late summer last year as a
> > single
> > patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
> > requested that I split it by component.
> > 
> > So that's being done here. Feel free to squash the patches as you
> > like.
> 
> Thanks for the patches.  I suppose those are no urgent fixes, and can
> be postponed for 6.16?  If so, I'll pick them up after 6.15-rc1 merge
> window is closed.

Sure, those are just improvements that aren't urgent.

Thx,
P.

> 
> 
> Takashi
Takashi Iwai April 7, 2025, 7:24 a.m. UTC | #3
On Fri, 04 Apr 2025 15:49:36 +0200,
Philipp Stanner wrote:
> 
> On Fri, 2025-04-04 at 15:47 +0200, Takashi Iwai wrote:
> > On Fri, 04 Apr 2025 14:19:01 +0200,
> > Philipp Stanner wrote:
> > > 
> > > pcim_iomap_table() and pcim_iomap_regions() have been deprecated by
> > > the
> > > PCI subsystem. In sound/, they can easily be replaced with
> > > pcim_iomap_region().
> > > 
> > > This series was around some time in late summer last year as a
> > > single
> > > patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
> > > requested that I split it by component.
> > > 
> > > So that's being done here. Feel free to squash the patches as you
> > > like.
> > 
> > Thanks for the patches.  I suppose those are no urgent fixes, and can
> > be postponed for 6.16?  If so, I'll pick them up after 6.15-rc1 merge
> > window is closed.
> 
> Sure, those are just improvements that aren't urgent.

Now all patches applied to for-next branch.  Thanks.


Takashi
Mark Brown April 7, 2025, 11:35 p.m. UTC | #4
On Fri, 04 Apr 2025 14:19:01 +0200, Philipp Stanner wrote:
> pcim_iomap_table() and pcim_iomap_regions() have been deprecated by the
> PCI subsystem. In sound/, they can easily be replaced with
> pcim_iomap_region().
> 
> This series was around some time in late summer last year as a single
> patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
> requested that I split it by component.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[11/11] ASoC: loongson: Replace deprecated PCI functions
        commit: 7288aa73e5cfb3f37ae93b55d7b7d63eca5140a8

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
Takashi Iwai April 8, 2025, 5:22 a.m. UTC | #5
On Tue, 08 Apr 2025 01:35:54 +0200,
Mark Brown wrote:
> 
> On Fri, 04 Apr 2025 14:19:01 +0200, Philipp Stanner wrote:
> > pcim_iomap_table() and pcim_iomap_regions() have been deprecated by the
> > PCI subsystem. In sound/, they can easily be replaced with
> > pcim_iomap_region().
> > 
> > This series was around some time in late summer last year as a single
> > patch. I lost track of it for a while, but Takashi Iwai (AFAIR)
> > requested that I split it by component.
> > 
> > [...]
> 
> Applied to
> 
>    https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
> 
> Thanks!
> 
> [11/11] ASoC: loongson: Replace deprecated PCI functions
>         commit: 7288aa73e5cfb3f37ae93b55d7b7d63eca5140a8
> 
> All being well this means that it will be integrated into the linux-next
> tree (usually sometime in the next 24 hours) and sent to Linus during
> the next merge window (or sooner if it is a bug fix), however if
> problems are discovered then the patch may be dropped or reverted.
> 
> You may get further e-mails resulting from automated or manual testing
> and review of the tree, please engage with people reporting problems and
> send followup patches addressing any issues that are reported if needed.
> 
> If any updates are required or you are submitting further changes they
> should be sent as incremental updates against current git, existing
> patches will not be replaced.
> 
> Please add any relevant lists and maintainers to the CCs when replying
> to this mail.

Oh, I already applied this one blindly as a part of series to my
tree.  But it's a trivial fix and shouldn't be a big problem to apply
twice...


Takashi
Mark Brown April 8, 2025, 9:26 a.m. UTC | #6
On Tue, Apr 08, 2025 at 07:22:54AM +0200, Takashi Iwai wrote:
> Mark Brown wrote:

> > [11/11] ASoC: loongson: Replace deprecated PCI functions
> >         commit: 7288aa73e5cfb3f37ae93b55d7b7d63eca5140a8

> Oh, I already applied this one blindly as a part of series to my
> tree.  But it's a trivial fix and shouldn't be a big problem to apply
> twice...

Yeah, I saw you applied it after it was already well down the pipeline
in a branch in my CI and thought it'd be fine if we ended up with two
copies.