> > This is correct. But it would be nice if somebody in the future could > > move the disabling of the switch to the inverse of the gswip_setup() > > function to make the code symmetrical. > > Should we add an uninit callback? Yes, that would be good. It looks like lan9303-core.c could use it as well for lan9303_disable_processing(chip); Thanks Andrew