Fri, Nov 18, 2016 at 05:01:14PM CET, a...@arndb.de wrote: >The newly added switchib driver fails to link if MLXSW_PCI=m: > >drivers/net/ethernet/mellanox/mlxsw/mlxsw_switchib.o: In >function^Cmlxsw_sib_module_exit': >switchib.c:(.exit.text+0x8): undefined reference to >`mlxsw_pci_driver_unregister' >switchib.c:(.exit.text+0x10): undefined reference to >`mlxsw_pci_driver_unregister' >drivers/net/ethernet/mellanox/mlxsw/mlxsw_switchib.o: In function >`mlxsw_sib_module_init': >switchib.c:(.init.text+0x28): undefined reference to >`mlxsw_pci_driver_register' >switchib.c:(.init.text+0x38): undefined reference to >`mlxsw_pci_driver_register' >switchib.c:(.init.text+0x48): undefined reference to >`mlxsw_pci_driver_unregister' > >The other two such sub-drivers have a dependency, so add the same one >here. In theory we could allow this driver if MLXSW_PCI is disabled, >but it's probably not worth it. > >Signed-off-by: Arnd Bergmann <a...@arndb.de>
Thanks. Reviewed-by: Jiri Pirko <j...@mellanox.com>