Hi, > Sparse complains about 'mt7621_pci_ops' symbol is not declared and asks if > it should be declared as 'static' instead. Sparse is right. Hence declare > symbol as static. Thank you for taking care of this! [...] > -struct pci_ops mt7621_pci_ops = { > +static struct pci_ops mt7621_pci_ops = { > .map_bus = mt7621_pcie_map_bus, > .read = pci_generic_config_read, > .write = pci_generic_config_write, Thank you! Reviewed-by: Krzysztof Wilczyński <kw@xxxxxxxxx> Krzysztof