Commit e6742e10 authored by Stephen Rothwell's avatar Stephen Rothwell Committed by Dave Airlie

drm: linux-next: build failure after merge of the drm-misc tree

Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64
allmodconfig) failed like this:

drivers/staging/vboxvideo/vbox_drv.c:235:2: error: unknown field 'set_busid' specified in initializer
  .set_busid = drm_pci_set_busid,
  ^
drivers/staging/vboxvideo/vbox_drv.c:235:15: error: 'drm_pci_set_busid' undeclared here (not in a function)
  .set_busid = drm_pci_set_busid,
               ^
drivers/staging/vboxvideo/vbox_drv.c: In function 'vbox_init':
drivers/staging/vboxvideo/vbox_drv.c:273:9: error: implicit declaration of function 'drm_pci_init' [-Werror=implicit-function-declaration]
  return drm_pci_init(&driver, &vbox_pci_driver);
         ^
drivers/staging/vboxvideo/vbox_drv.c: In function 'vbox_exit':
drivers/staging/vboxvideo/vbox_drv.c:278:2: error: implicit declaration of function 'drm_pci_exit' [-Werror=implicit-function-declaration]
  drm_pci_exit(&driver, &vbox_pci_driver);
  ^

Caused by commits

  5c484cee ("drm: Remove drm_driver->set_busid hook")
  10631d72 ("drm/pci: Deprecate drm_pci_init/exit completely")

interacting with commit

  dd55d44f ("staging: vboxvideo: Add vboxvideo to drivers/staging")

from the staging.current tree.

I have applied the following merge fix patch - please check that it
is correct.

From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Wed, 19 Jul 2017 11:41:01 +1000
Subject: [PATCH] drm: fixes for staging due to API changes in the drm core
Signed-off-by: default avatarStephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent 0eb2c0ae
...@@ -232,7 +232,6 @@ static struct drm_driver driver = { ...@@ -232,7 +232,6 @@ static struct drm_driver driver = {
.lastclose = vbox_driver_lastclose, .lastclose = vbox_driver_lastclose,
.master_set = vbox_master_set, .master_set = vbox_master_set,
.master_drop = vbox_master_drop, .master_drop = vbox_master_drop,
.set_busid = drm_pci_set_busid,
.fops = &vbox_fops, .fops = &vbox_fops,
.irq_handler = vbox_irq_handler, .irq_handler = vbox_irq_handler,
...@@ -270,12 +269,12 @@ static int __init vbox_init(void) ...@@ -270,12 +269,12 @@ static int __init vbox_init(void)
if (vbox_modeset == 0) if (vbox_modeset == 0)
return -EINVAL; return -EINVAL;
return drm_pci_init(&driver, &vbox_pci_driver); return pci_register_driver(&vbox_pci_driver);
} }
static void __exit vbox_exit(void) static void __exit vbox_exit(void)
{ {
drm_pci_exit(&driver, &vbox_pci_driver); pci_unregister_driver(&vbox_pci_driver);
} }
module_init(vbox_init); module_init(vbox_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment