On Sun, Jun 13, 2021 at 05:29:13PM +0800, Dongdong Liu wrote:
> +static void pci_configure_10bit_tags(struct pci_dev *dev)
> +{
> + struct pci_dev *bridge;
> +
> + if (!pci_is_pcie(dev))
> + return;
> +
> + if (!(dev->pcie_devcap2 & PCI_EXP_DEVCAP2_10BIT_TAG_COMP))
> + return;
Doesn't the second check imply the first one?
Otherwise looks good:
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
