>>> +if NET_VENDOR_ASIX >>> + >>> +config SPI_AX88796C >>> + tristate "Asix AX88796C-SPI support" >>> + depends on SPI >> >> That line is redundant (but not harmful). > > Why? Is it because NET_VENDOR_ASIX depends on SPI? Probably it > shouldn't. Thanks for spotting. Yes, that. -- ~Randy