* [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device()
@ 2022-02-01 18:11 Rafael J. Wysocki
2022-02-05 3:31 ` Lu Baolu
2022-02-14 12:00 ` Joerg Roedel
0 siblings, 2 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2022-02-01 18:11 UTC (permalink / raw)
To: David Woodhouse, Lu Baolu
Cc: LKML, Linux ACPI, Joerg Roedel, Will Deacon, iommu
From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Replace acpi_bus_get_device() that is going to be dropped with
acpi_fetch_acpi_dev().
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
drivers/iommu/intel/dmar.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Index: linux-pm/drivers/iommu/intel/dmar.c
===================================================================
--- linux-pm.orig/drivers/iommu/intel/dmar.c
+++ linux-pm/drivers/iommu/intel/dmar.c
@@ -789,7 +789,8 @@ static int __init dmar_acpi_dev_scope_in
andd->device_name);
continue;
}
- if (acpi_bus_get_device(h, &adev)) {
+ adev = acpi_fetch_acpi_dev(h);
+ if (!adev) {
pr_err("Failed to get device for ACPI object %s\n",
andd->device_name);
continue;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device()
2022-02-01 18:11 [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device() Rafael J. Wysocki
@ 2022-02-05 3:31 ` Lu Baolu
2022-02-14 12:00 ` Joerg Roedel
1 sibling, 0 replies; 3+ messages in thread
From: Lu Baolu @ 2022-02-05 3:31 UTC (permalink / raw)
To: Rafael J. Wysocki, David Woodhouse
Cc: baolu.lu, LKML, Linux ACPI, Joerg Roedel, Will Deacon, iommu
On 2022/2/2 2:11, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> Replace acpi_bus_get_device() that is going to be dropped with
> acpi_fetch_acpi_dev().
>
> No intentional functional impact.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> drivers/iommu/intel/dmar.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> Index: linux-pm/drivers/iommu/intel/dmar.c
> ===================================================================
> --- linux-pm.orig/drivers/iommu/intel/dmar.c
> +++ linux-pm/drivers/iommu/intel/dmar.c
> @@ -789,7 +789,8 @@ static int __init dmar_acpi_dev_scope_in
> andd->device_name);
> continue;
> }
> - if (acpi_bus_get_device(h, &adev)) {
> + adev = acpi_fetch_acpi_dev(h);
> + if (!adev) {
> pr_err("Failed to get device for ACPI object %s\n",
> andd->device_name);
> continue;
Please adjust the patch title to "iommu/vtd: Replace acpi_bus_get_device()"
Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>
Best regards,
baolu
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device()
2022-02-01 18:11 [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device() Rafael J. Wysocki
2022-02-05 3:31 ` Lu Baolu
@ 2022-02-14 12:00 ` Joerg Roedel
1 sibling, 0 replies; 3+ messages in thread
From: Joerg Roedel @ 2022-02-14 12:00 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: David Woodhouse, Lu Baolu, LKML, Linux ACPI, Will Deacon, iommu
On Tue, Feb 01, 2022 at 07:11:40PM +0100, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> Replace acpi_bus_get_device() that is going to be dropped with
> acpi_fetch_acpi_dev().
>
> No intentional functional impact.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> drivers/iommu/intel/dmar.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Applied with adjusted subject, thanks Rafael.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-02-14 12:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-01 18:11 [PATCH] IOMMU: Intel: DMAR: Replace acpi_bus_get_device() Rafael J. Wysocki
2022-02-05 3:31 ` Lu Baolu
2022-02-14 12:00 ` Joerg Roedel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).