Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Fri, 23 Feb 2024 09:47:09 +0300
Cc: Ethan Zhao <haifeng.zhao@xxxxxxxxxxxxxxx>, bhelgaas@xxxxxxxxxx, robin.murphy@xxxxxxx, jgg@xxxxxxxx, kevin.tian@xxxxxxxxx, dwmw2@xxxxxxxxxxxxx, will@xxxxxxxxxx, lukas@xxxxxxxxx, yi.l.liu@xxxxxxxxx, iommu@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-pci@xxxxxxxxxxxxxxx, Haorong Ye <yehaorong@xxxxxxxxxxxxx>
In-reply-to: <9c4637d5-6496-4c68-b2db-4be1e56ca746@linux.intel.com>
References: <20240222090251.2849702-1-haifeng.zhao@linux.intel.com> <20240222090251.2849702-2-haifeng.zhao@linux.intel.com> <9c4637d5-6496-4c68-b2db-4be1e56ca746@linux.intel.com>

I'm not a PCI maintainer, but first two patches seem good to me.  For
the third patch, my complaints were really minor.  Let's just add a
Fixes tag for sure, the rest is okay.

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

regards,
dan carpenter

Follow-Ups:
- Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
  - From: Ethan Zhao

References:
- [PATCH v13 0/3] fix vt-d hard lockup when hotplug ATS capable device
  - From: Ethan Zhao
- [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
  - From: Ethan Zhao
- Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
  - From: Baolu Lu

Prev by Date: Re: [PATCH 0/3] sysfs: Group visibility fixups
Next by Date: Re: [PATCH v13 3/3] iommu/vt-d: improve ITE fault handling if target device isn't valid
Previous by thread: Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
Next by thread: Re: [PATCH v13 1/3] PCI: make pci_dev_is_disconnected() helper public for other drivers
Index(es):
- Date
- Thread

[Index of Archives] [DMA Engine] [Linux Coverity] [Linux USB] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [Greybus]