On Fri, 9 Aug 2019 14:22:22 -0700 Jakub Kicinski <jakub.kicinski@xxxxxxxxxxxxx> wrote: > On Fri, 9 Aug 2019 12:32:29 +0200, Thomas Bogendoerfer wrote: > > SGI IOC3 chip has integrated ethernet, keyboard and mouse interface. > > It also supports connecting a SuperIO chip for serial and parallel > > interfaces. IOC3 is used inside various SGI systemboards and add-on > > cards with different equipped external interfaces. > > > > Support for ethernet and serial interfaces were implemented inside > > the network driver. This patchset moves out the not network related > > parts to a new MFD driver, which takes care of card detection, > > setup of platform devices and interrupt distribution for the subdevices. > > > > Serial portion: Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > > > Signed-off-by: Thomas Bogendoerfer <tbogendoerfer@xxxxxxx> > > There are a lot of changes in the ethernet part which are not easy to > explain by the introduction of the other MFD parts.. Could you possibly > break this change up into smaller chunks? working on it > Also please don't use stdint types in the kernel, please try checkpatch > to catch coding style issues. my patch already reduces them and checkpatch only warns about usage of printk for the network part. Changing that to dev_warn/dev_err in the mfd patch didn't seem the right thing to do. As I'm splitting the conversion patch into a few steps I could also replace the printks. Thomas. -- SUSE Linux GmbH GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 21284 (AG Nürnberg)