On Thu, Sep 02, 2021 at 10:26:31PM +0100, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The function prototype is missing an identifier name. Add one. > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > --- > drivers/vfio/pci/vfio_pci_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) It seems fine, but is there a reason to just pick this one case and not clean the whole subsystem? Eg i see a couple more cases in the headers Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx> Jason