On Sat, Jul 24, 2021 at 11:54:40AM +0200, Petr Vorel wrote: > Hi, > > I'd like to cleanup and mainline Linux Sunxi SoC AW USB host driver [1]. Great! > What are the most ugly parts which should be replaced? Loading module does not > complain about anything? > [267044.912155] awusb: loading out-of-tree module taints kernel. That will be fixed by moving it into the tree. > [267044.912257] awusb: module verification failed: signature and/or required key missing - tainting kernel That shows a mis-match between you building a kernel and what keys were used to build the distro kernel you are using. > [267044.913399] usbcore: registered new interface driver allwinner > [267044.913401] awusb: v0.5:AW USB driver Looks fine. > BTW should it go to the host/Kconfig or misc/Kconfig or elsewhere? No idea, is this a host driver or does it control a USB device you plug into the system? > [1] https://github.com/linux-sunxi/sunxi-livesuite/blob/master/awusb/awusb.c I looked at this, and it's a device driver, not a host controller driver. But it looks to need a userspace program for the ioctls, where is that code at? And why does it need any ioctls at all? Why is this even a driver at all, it looks like you can write a small userspace program using libusb to do everything it does, right? What exactly is this driver needed for? thanks, greg k-h