Hello Thomas, On 7/21/22 10:16, Thomas Zimmermann wrote: > Reverse the order of the final two arguments when calling > aperture_remove_conflicting_devices(). An error report is available > at [1]. > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx> > Fixes: 8d69d008f44c ("fbdev: Convert drivers to aperture helpers") > Cc: Thomas Zimmermann <tzimmermann@xxxxxxx> > Cc: Javier Martinez Canillas <javierm@xxxxxxxxxx> > Cc: Sudip Mukherjee <sudipm.mukherjee@xxxxxxxxx> > Cc: Teddy Wang <teddy.wang@xxxxxxxxxxxxxxxxx> > Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> > Cc: "K. Y. Srinivasan" <kys@xxxxxxxxxxxxx> > Cc: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx> > Cc: Stephen Hemminger <sthemmin@xxxxxxxxxxxxx> > Cc: Wei Liu <wei.liu@xxxxxxxxxx> > Cc: Dexuan Cui <decui@xxxxxxxxxxxxx> > Cc: linux-fbdev@xxxxxxxxxxxxxxx > Cc: linux-hyperv@xxxxxxxxxxxxxxx > Link: https://lore.kernel.org/lkml/202207202040.jS1WcTzN-lkp@xxxxxxxxx/ # 1 > --- > drivers/video/fbdev/aty/radeon_base.c | 2 +- > drivers/video/fbdev/hyperv_fb.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/video/fbdev/aty/radeon_base.c b/drivers/video/fbdev/aty/radeon_base.c > index e5e362b8c9da..0a8199985d52 100644 > --- a/drivers/video/fbdev/aty/radeon_base.c > +++ b/drivers/video/fbdev/aty/radeon_base.c > @@ -2243,7 +2243,7 @@ static int radeon_kick_out_firmware_fb(struct pci_dev *pdev) > resource_size_t base = pci_resource_start(pdev, 0); > resource_size_t size = pci_resource_len(pdev, 0); > > - return aperture_remove_conflicting_devices(base, size, KBUILD_MODNAME, false); > + return aperture_remove_conflicting_devices(base, size, false, KBUILD_MODNAME); > } I also missed that the order wasn't correct when reviewing. Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> -- Best regards, Javier Martinez Canillas Linux Engineering Red Hat