video: fbdev: cyber2000fb: remove superfluous CONFIG_PCI ifdef

Submitted by Bartlomiej Zolnierkiewicz on May 21, 2019, 1:44 p.m.

Details

Message ID 214f05e0-a448-b1cf-7475-4fa7eeaa9949@samsung.com
State New
Headers show
Series "video: fbdev: cyber2000fb: remove superfluous CONFIG_PCI ifdef" ( rev: 1 ) in DRI devel

Not browsing as part of any series.

Commit Message

Bartlomiej Zolnierkiewicz May 21, 2019, 1:44 p.m.
This is a PCI driver and FB_CYBER2000 depends on PCI in Kconfig so
there is no need to check for PCI inside the driver code.

Cc: Russell King <linux@armlinux.org.uk>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
---
 drivers/video/fbdev/cyber2000fb.c |    5 -----
 1 file changed, 5 deletions(-)

Patch hide | download patch | download mbox

Index: b/drivers/video/fbdev/cyber2000fb.c
===================================================================
--- a/drivers/video/fbdev/cyber2000fb.c
+++ b/drivers/video/fbdev/cyber2000fb.c
@@ -1642,10 +1642,6 @@  static void cyberpro_common_resume(struc
 }
 
 /*
- * PCI specific support.
- */
-#ifdef CONFIG_PCI
-/*
  * We need to wake up the CyberPro, and make sure its in linear memory
  * mode.  Unfortunately, this is specific to the platform and card that
  * we are running on.
@@ -1861,7 +1857,6 @@  static struct pci_driver cyberpro_driver
 	.resume		= cyberpro_pci_resume,
 	.id_table	= cyberpro_pci_table
 };
-#endif
 
 /*
  * I don't think we can use the "module_init" stuff here because

Comments

On 5/21/19 3:44 PM, Bartlomiej Zolnierkiewicz wrote:
> This is a PCI driver and FB_CYBER2000 depends on PCI in Kconfig so
> there is no need to check for PCI inside the driver code.
> 
> Cc: Russell King <linux@armlinux.org.uk>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>

I queued the patch for v5.3.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics