On Tue, Aug 09, 2022 at 06:00:50PM +0000, Will McVicker wrote: > + bool msi_64b = false; Spellt out bit here? > + msi_capabilities = dw_pcie_msi_capabilities(pci); > + if (msi_capabilities & PCI_MSI_FLAGS_ENABLE) > + msi_64b = msi_capabilities & PCI_MSI_FLAGS_64BIT ? true : false; No need for the tenary operator here: msi_64bit = msi_capabilities & PCI_MSI_FLAGS_64BIT;