| Message ID | 20241028102428.23118-2-pstanner@redhat.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | scsi: ufs: Replace deprecated PCI functions | expand |
On Mon, 2024-10-28 at 11:24 +0100, Philipp Stanner wrote: > pcim_iomap_regions() and pcim_iomap_table() have been deprecated in > commit e354bb84a4c1 ("PCI: Deprecate pcim_iomap_table(), > pcim_iomap_regions_request_all()"). > > Replace these functions with pcim_iomap_region(). > > Signed-off-by: Philipp Stanner <pstanner@redhat.com> Reviewed-by: Bean Huo <beanhuo@micron.com>
On 28/10/24 12:24, Philipp Stanner wrote: > pcim_iomap_regions() and pcim_iomap_table() have been deprecated in > commit e354bb84a4c1 ("PCI: Deprecate pcim_iomap_table(), > pcim_iomap_regions_request_all()"). > > Replace these functions with pcim_iomap_region(). > > Signed-off-by: Philipp Stanner <pstanner@redhat.com> Reviewed-by: Adrian Hunter <adrian.hunter@intel.com> > --- > drivers/ufs/host/tc-dwc-g210-pci.c | 8 +++----- > drivers/ufs/host/ufshcd-pci.c | 8 +++----- > 2 files changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/ufs/host/tc-dwc-g210-pci.c b/drivers/ufs/host/tc-dwc-g210-pci.c > index 876781fd6861..0167d8bef71a 100644 > --- a/drivers/ufs/host/tc-dwc-g210-pci.c > +++ b/drivers/ufs/host/tc-dwc-g210-pci.c > @@ -80,14 +80,12 @@ tc_dwc_g210_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) > > pci_set_master(pdev); > > - err = pcim_iomap_regions(pdev, 1 << 0, UFSHCD); > - if (err < 0) { > + mmio_base = pcim_iomap_region(pdev, 0, UFSHCD); > + if (IS_ERR(mmio_base)) { > dev_err(&pdev->dev, "request and iomap failed\n"); > - return err; > + return PTR_ERR(mmio_base); > } > > - mmio_base = pcim_iomap_table(pdev)[0]; > - > err = ufshcd_alloc_host(&pdev->dev, &hba); > if (err) { > dev_err(&pdev->dev, "Allocation failed\n"); > diff --git a/drivers/ufs/host/ufshcd-pci.c b/drivers/ufs/host/ufshcd-pci.c > index 54e0cc0653a2..ea39c5d5b8cf 100644 > --- a/drivers/ufs/host/ufshcd-pci.c > +++ b/drivers/ufs/host/ufshcd-pci.c > @@ -588,14 +588,12 @@ ufshcd_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) > > pci_set_master(pdev); > > - err = pcim_iomap_regions(pdev, 1 << 0, UFSHCD); > - if (err < 0) { > + mmio_base = pcim_iomap_region(pdev, 0, UFSHCD); > + if (IS_ERR(mmio_base)) { > dev_err(&pdev->dev, "request and iomap failed\n"); > - return err; > + return PTR_ERR(mmio_base); > } > > - mmio_base = pcim_iomap_table(pdev)[0]; > - > err = ufshcd_alloc_host(&pdev->dev, &hba); > if (err) { > dev_err(&pdev->dev, "Allocation failed\n");
Philipp, > pcim_iomap_regions() and pcim_iomap_table() have been deprecated in > commit e354bb84a4c1 ("PCI: Deprecate pcim_iomap_table(), > pcim_iomap_regions_request_all()"). > > Replace these functions with pcim_iomap_region(). Applied to 6.13/scsi-staging, thanks!
On Mon, 28 Oct 2024 11:24:29 +0100, Philipp Stanner wrote: > pcim_iomap_regions() and pcim_iomap_table() have been deprecated in > commit e354bb84a4c1 ("PCI: Deprecate pcim_iomap_table(), > pcim_iomap_regions_request_all()"). > > Replace these functions with pcim_iomap_region(). > > > [...] Applied to 6.13/scsi-queue, thanks! [1/1] scsi: ufs: Replace deprecated PCI functions https://git.kernel.org/mkp/scsi/c/84c1e27e6c64
diff --git a/drivers/ufs/host/tc-dwc-g210-pci.c b/drivers/ufs/host/tc-dwc-g210-pci.c index 876781fd6861..0167d8bef71a 100644 --- a/drivers/ufs/host/tc-dwc-g210-pci.c +++ b/drivers/ufs/host/tc-dwc-g210-pci.c @@ -80,14 +80,12 @@ tc_dwc_g210_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) pci_set_master(pdev); - err = pcim_iomap_regions(pdev, 1 << 0, UFSHCD); - if (err < 0) { + mmio_base = pcim_iomap_region(pdev, 0, UFSHCD); + if (IS_ERR(mmio_base)) { dev_err(&pdev->dev, "request and iomap failed\n"); - return err; + return PTR_ERR(mmio_base); } - mmio_base = pcim_iomap_table(pdev)[0]; - err = ufshcd_alloc_host(&pdev->dev, &hba); if (err) { dev_err(&pdev->dev, "Allocation failed\n"); diff --git a/drivers/ufs/host/ufshcd-pci.c b/drivers/ufs/host/ufshcd-pci.c index 54e0cc0653a2..ea39c5d5b8cf 100644 --- a/drivers/ufs/host/ufshcd-pci.c +++ b/drivers/ufs/host/ufshcd-pci.c @@ -588,14 +588,12 @@ ufshcd_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) pci_set_master(pdev); - err = pcim_iomap_regions(pdev, 1 << 0, UFSHCD); - if (err < 0) { + mmio_base = pcim_iomap_region(pdev, 0, UFSHCD); + if (IS_ERR(mmio_base)) { dev_err(&pdev->dev, "request and iomap failed\n"); - return err; + return PTR_ERR(mmio_base); } - mmio_base = pcim_iomap_table(pdev)[0]; - err = ufshcd_alloc_host(&pdev->dev, &hba); if (err) { dev_err(&pdev->dev, "Allocation failed\n");
pcim_iomap_regions() and pcim_iomap_table() have been deprecated in commit e354bb84a4c1 ("PCI: Deprecate pcim_iomap_table(), pcim_iomap_regions_request_all()"). Replace these functions with pcim_iomap_region(). Signed-off-by: Philipp Stanner <pstanner@redhat.com> --- drivers/ufs/host/tc-dwc-g210-pci.c | 8 +++----- drivers/ufs/host/ufshcd-pci.c | 8 +++----- 2 files changed, 6 insertions(+), 10 deletions(-)