fbdev/sisfb: Unexport symbols
authorThomas Zimmermann <tzimmermann@suse.de>
Thu, 12 Jun 2025 08:16:35 +0000 (10:16 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Mon, 16 Jun 2025 07:08:06 +0000 (09:08 +0200)
Fix the compile-time warning

  drivers/video/fbdev/sis/sis_main.c: warning: EXPORT_SYMBOL() is used, but #include <linux/export.h> is missing

The affected symbols are not used outside of their module. Some of
the symbols are not used anywhere, so remove the functions entirely.
Also remove the related, but unused, define SISFB_HAVE_MALLOC_NEW.

v2:
- remove unused functions (Helge)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Helge Deller <deller@gmx.de>
Fixes: a934a57a42f6 ("scripts/misc-check: check missing #include <linux/export.h> when W=1")
Cc: Masahiro Yamada <masahiroy@kernel.org>
Cc: Nathan Chancellor <nathan@kernel.org>
Link: https://lore.kernel.org/r/20250612081738.197826-13-tzimmermann@suse.de
drivers/video/fbdev/sis/sis.h
drivers/video/fbdev/sis/sis_main.c
include/video/sisfb.h

index d632f09..3d65848 100644 (file)
@@ -673,9 +673,7 @@ unsigned int        sisfb_read_mio_pci_word(struct SiS_Private *SiS_Pr, int reg);
 
 /* SiS-specific exported functions */
 void                   sis_malloc(struct sis_memreq *req);
-void                   sis_malloc_new(struct pci_dev *pdev, struct sis_memreq *req);
 void                   sis_free(u32 base);
-void                   sis_free_new(struct pci_dev *pdev, u32 base);
 
 /* Routines from init.c/init301.c */
 extern unsigned short  SiS_GetModeID_LCD(int VGAEngine, unsigned int VBFlags, int HDisplay,
index 75033e6..84567d6 100644 (file)
@@ -3421,14 +3421,6 @@ sis_malloc(struct sis_memreq *req)
                req->offset = req->size = 0;
 }
 
-void
-sis_malloc_new(struct pci_dev *pdev, struct sis_memreq *req)
-{
-       struct sis_video_info *ivideo = pci_get_drvdata(pdev);
-
-       sis_int_malloc(ivideo, req);
-}
-
 /* sis_free: u32 because "base" is offset inside video ram, can never be >4GB */
 
 static void
@@ -3455,14 +3447,6 @@ sis_free(u32 base)
        sis_int_free(ivideo, base);
 }
 
-void
-sis_free_new(struct pci_dev *pdev, u32 base)
-{
-       struct sis_video_info *ivideo = pci_get_drvdata(pdev);
-
-       sis_int_free(ivideo, base);
-}
-
 /* --------------------- SetMode routines ------------------------- */
 
 static void
@@ -6832,12 +6816,3 @@ MODULE_PARM_DESC(videoram,
 #endif
 
 #endif            /*  /MODULE  */
-
-/* _GPL only for new symbols. */
-EXPORT_SYMBOL(sis_malloc);
-EXPORT_SYMBOL(sis_free);
-EXPORT_SYMBOL_GPL(sis_malloc_new);
-EXPORT_SYMBOL_GPL(sis_free_new);
-
-
-
index 76ff628..54e6632 100644 (file)
 #define        SIS_300_VGA  1
 #define        SIS_315_VGA  2
 
-#define SISFB_HAVE_MALLOC_NEW
-extern void sis_malloc(struct sis_memreq *req);
-extern void sis_malloc_new(struct pci_dev *pdev, struct sis_memreq *req);
-
-extern void sis_free(u32 base);
-extern void sis_free_new(struct pci_dev *pdev, u32 base);
 #endif