Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Dec 2019 03:00:20 +0000 (UTC)
From:      Kyle Evans <kevans@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r356110 - in stable: 11/sys/arm/broadcom/bcm2835 11/sys/arm/freescale/imx 11/sys/arm/ti/am335x 11/sys/arm/versatile 11/sys/dev/terasic/mtl 11/sys/powerpc/ofw 12/sys/arm/broadcom/bcm2835...
Message-ID:  <201912270300.xBR30Kw8012573@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kevans
Date: Fri Dec 27 03:00:18 2019
New Revision: 356110
URL: https://svnweb.freebsd.org/changeset/base/356110

Log:
  MFC r355936, r356045: remove unused kbd drivers
  
  r355936:
  Kill off dummy kbd drivers
  
  As far as I can tell, these are an artifact of times when linker sets
  couldn't be empty, otherwise the kernel build would fail due to unresolved
  symbols. hselasky fixed this in r268138, and I've audited the kbd portions
  to make sure nothing would blow up due to the empty linker set and
  successfully compiled+ran a kernel with no keyboard support at all.
  
  Kill them off now since they're no longer required.
  
  r356045:
  terasic_mtl: kill off final dummy keyboard driver
  
  The rest were removed in r355936, which speculated that the cause of this
  phenomenon was due to an inability to have an empty linker set. The comment
  included with this one shows that this was, in fact, not the reason.
  
  Regardless, syscons no longer seems to have an issue with not having any
  keyboard drivers and in-fact ignores the keyboard probe anyways.

Modified:
  stable/12/sys/arm/broadcom/bcm2835/bcm2835_fb.c
  stable/12/sys/arm/freescale/imx/imx51_ipuv3.c
  stable/12/sys/arm/ti/am335x/am335x_lcd_syscons.c
  stable/12/sys/arm/versatile/versatile_clcd.c
  stable/12/sys/dev/terasic/mtl/terasic_mtl_syscons.c
  stable/12/sys/powerpc/ofw/ofw_syscons.c
Directory Properties:
  stable/12/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/11/sys/arm/broadcom/bcm2835/bcm2835_fb.c
  stable/11/sys/arm/freescale/imx/imx51_ipuv3.c
  stable/11/sys/arm/ti/am335x/am335x_lcd_syscons.c
  stable/11/sys/arm/versatile/versatile_clcd.c
  stable/11/sys/dev/terasic/mtl/terasic_mtl_syscons.c
  stable/11/sys/powerpc/ofw/ofw_syscons.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/12/sys/arm/broadcom/bcm2835/bcm2835_fb.c
==============================================================================
--- stable/12/sys/arm/broadcom/bcm2835/bcm2835_fb.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/arm/broadcom/bcm2835/bcm2835_fb.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -850,22 +850,3 @@ bcmfb_putm(video_adapter_t *adp, int x, int y, uint8_t
 
 	return (0);
 }
-
-/*
- * Define a stub keyboard driver in case one hasn't been
- * compiled into the kernel
- */
-#include <sys/kbio.h>
-#include <dev/kbd/kbdreg.h>
-
-static int dummy_kbd_configure(int flags);
-
-keyboard_switch_t bcmdummysw;
-
-static int
-dummy_kbd_configure(int flags)
-{
-
-	return (0);
-}
-KEYBOARD_DRIVER(bcmdummy, bcmdummysw, dummy_kbd_configure);

Modified: stable/12/sys/arm/freescale/imx/imx51_ipuv3.c
==============================================================================
--- stable/12/sys/arm/freescale/imx/imx51_ipuv3.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/arm/freescale/imx/imx51_ipuv3.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -878,22 +878,3 @@ ipu3fb_putm(video_adapter_t *adp, int x, int y, uint8_
 
 	return (0);
 }
-
-/*
- * Define a stub keyboard driver in case one hasn't been
- * compiled into the kernel
- */
-#include <sys/kbio.h>
-#include <dev/kbd/kbdreg.h>
-
-static int dummy_kbd_configure(int flags);
-
-keyboard_switch_t ipu3dummysw;
-
-static int
-dummy_kbd_configure(int flags)
-{
-
-	return (0);
-}
-KEYBOARD_DRIVER(ipu3dummy, ipu3dummysw, dummy_kbd_configure);

Modified: stable/12/sys/arm/ti/am335x/am335x_lcd_syscons.c
==============================================================================
--- stable/12/sys/arm/ti/am335x/am335x_lcd_syscons.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/arm/ti/am335x/am335x_lcd_syscons.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -770,22 +770,3 @@ int am335x_lcd_syscons_setup(vm_offset_t vaddr, vm_pad
 
 	return (0);
 }
-
-/*
- * Define a stub keyboard driver in case one hasn't been
- * compiled into the kernel
- */
-#include <sys/kbio.h>
-#include <dev/kbd/kbdreg.h>
-
-static int dummy_kbd_configure(int flags);
-
-keyboard_switch_t am335x_dummysw;
-
-static int
-dummy_kbd_configure(int flags)
-{
-
-	return (0);
-}
-KEYBOARD_DRIVER(am335x_dummy, am335x_dummysw, dummy_kbd_configure);

Modified: stable/12/sys/arm/versatile/versatile_clcd.c
==============================================================================
--- stable/12/sys/arm/versatile/versatile_clcd.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/arm/versatile/versatile_clcd.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -920,22 +920,3 @@ versatilefb_putm(video_adapter_t *adp, int x, int y, u
 
 	return (0);
 }
-
-/*
- * Define a stub keyboard driver in case one hasn't been
- * compiled into the kernel
- */
-#include <sys/kbio.h>
-#include <dev/kbd/kbdreg.h>
-
-static int dummy_kbd_configure(int flags);
-
-keyboard_switch_t bcmdummysw;
-
-static int
-dummy_kbd_configure(int flags)
-{
-
-	return (0);
-}
-KEYBOARD_DRIVER(bcmdummy, bcmdummysw, dummy_kbd_configure);

Modified: stable/12/sys/dev/terasic/mtl/terasic_mtl_syscons.c
==============================================================================
--- stable/12/sys/dev/terasic/mtl/terasic_mtl_syscons.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/dev/terasic/mtl/terasic_mtl_syscons.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -406,22 +406,6 @@ terasic_mtl_vidsw_putm(video_adapter_t *adp, int x, in
 	return (ENODEV);
 }
 
-/*
- * XXXRW: For historical reasons, syscons can't register video consoles
- * without a keyboard implementation.  Provide a dummy.
- */
-static keyboard_switch_t	terasic_mtl_keyboard_switch;
-
-static int
-terasic_mtl_kbd_configure(int flags)
-{
-
-	return (0);
-}
-
-KEYBOARD_DRIVER(mtl_kbd, terasic_mtl_keyboard_switch,
-    terasic_mtl_kbd_configure);
-
 int
 terasic_mtl_syscons_attach(struct terasic_mtl_softc *sc)
 {

Modified: stable/12/sys/powerpc/ofw/ofw_syscons.c
==============================================================================
--- stable/12/sys/powerpc/ofw/ofw_syscons.c	Fri Dec 27 01:50:16 2019	(r356109)
+++ stable/12/sys/powerpc/ofw/ofw_syscons.c	Fri Dec 27 03:00:18 2019	(r356110)
@@ -1049,25 +1049,6 @@ static devclass_t	sc_devclass;
 DRIVER_MODULE(ofwfb, nexus, ofwfb_sc_driver, sc_devclass, 0, 0);
 
 /*
- * Define a stub keyboard driver in case one hasn't been
- * compiled into the kernel
- */
-#include <sys/kbio.h>
-#include <dev/kbd/kbdreg.h>
-
-static int dummy_kbd_configure(int flags);
-
-keyboard_switch_t dummysw;
-
-static int
-dummy_kbd_configure(int flags)
-{
-
-	return (0);
-}
-KEYBOARD_DRIVER(dummy, dummysw, dummy_kbd_configure);
-
-/*
  * Utility routines from <dev/fb/fbreg.h>
  */
 void



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201912270300.xBR30Kw8012573>