From: Crystal Kolipe <kolipe.c@exoticsilicon.com>
Subject: Re: improve wscons man pages
To: Matthieu Herrb <matthieu@openbsd.org>
Cc: tech@openbsd.org
Date: Fri, 15 Aug 2025 21:06:21 +0100

On Fri, Aug 15, 2025 at 09:19:46PM +0200, Matthieu Herrb wrote:
> On Fri, Aug 15, 2025 at 06:36:12PM +0100, Crystal Kolipe wrote:
> > On Fri, Aug 15, 2025 at 04:23:33PM +0200, Matthieu Herrb wrote:
> > > The diff below adds some documentation about the supported ioctls()
> > > on wskbd(4), wsmouse(4) and wsmux(4). For wskbd(4) I also included the
> > > description of the supported keymaps.
> > 
> > The list of keymaps is missing any referene to KB_IOPENER, is this intentional
> > or an oversight?
> > 
> > Otherwise looks OK to me.
> > 
> 
> Probably. it also not documented on NetBSD (where I copied the keymap
> text from) and I had never heard about this option before.

Almost certainly, nobody is running any recent OpenBSD version on this
hardware.

It's time for the keyboard support to go, rather than being documented.

--- dev/wscons/wsksymdef.h	Mon Apr  7 04:25:23 2025
+++ dev/wscons/wsksymdef.h	Fri Aug 15 20:47:30 2025
@@ -748,7 +748,6 @@
 #define KB_SWAPCTRLCAPS		0x00000008 /* swap Left-Control and Caps-Lock */
 #define KB_DVORAK		0x00000010 /* Dvorak layout */
 #define KB_METAESC		0x00000020 /* generate ESC prefix on ALT-key */
-#define KB_IOPENER		0x00000040 /* f1-f12 -> ESC,f1-f11 */
 #define KB_NOENCODING		0x00000080 /* no encodings available */
 #define KB_APPLE		0x00010000 /* Apple specific layout */
 #define KB_COLEMAK		0x02000000 /* Colemak layout */
@@ -792,7 +791,6 @@
 	{ KB_SWAPCTRLCAPS, "swapctrlcaps" }, \
 	{ KB_DVORAK,	"dvorak" }, \
 	{ KB_METAESC,	"metaesc" }, \
-	{ KB_IOPENER,	"iopener" }, \
 	{ KB_NOENCODING, "noencoding" }, \
 	{ KB_APPLE,	"apple" }, \
 	{ KB_COLEMAK,	"colemak" }
--- dev/pckbc/wskbdmap_mfii.c	Mon Apr  7 04:25:22 2025
+++ dev/pckbc/wskbdmap_mfii.c	Fri Aug 15 20:45:51 2025
@@ -623,22 +623,6 @@
     KC(58),  KS_Cmd1,		KS_Control_L,
 };
 
-static const keysym_t pckbd_keydesc_iopener[] = {
-/*  pos      command		normal		shifted */
-    KC(59),  KS_Cmd_Debugger,	KS_Escape,
-    KC(60),  KS_Cmd_Screen0,	KS_f1,
-    KC(61),  KS_Cmd_Screen1,	KS_f2,
-    KC(62),  KS_Cmd_Screen2,	KS_f3,
-    KC(63),  KS_Cmd_Screen3,	KS_f4,
-    KC(64),  KS_Cmd_Screen4,	KS_f5,
-    KC(65),  KS_Cmd_Screen5,	KS_f6,
-    KC(66),  KS_Cmd_Screen6,	KS_f7,
-    KC(67),  KS_Cmd_Screen7,	KS_f8,
-    KC(68),  KS_Cmd_Screen8,	KS_f9,
-    KC(87),  KS_Cmd_Screen9,	KS_f10,
-    KC(88),			KS_f11,
-};
-
 static const keysym_t pckbd_keydesc_ru[] = {
 /*  pos      normal		shifted		altgr			shift-altgr */
     KC(7),   KS_6,		KS_asciicircum,	KS_6,			KS_comma,
@@ -1151,7 +1135,6 @@
 	KBD_MAP(KB_US | KB_DVORAK,	KB_US,	pckbd_keydesc_us_dvorak),
 	KBD_MAP(KB_US | KB_COLEMAK,	KB_US,	pckbd_keydesc_us_colemak),
 	KBD_MAP(KB_US | KB_SWAPCTRLCAPS, KB_US,	pckbd_keydesc_swapctrlcaps),
-	KBD_MAP(KB_US | KB_IOPENER,	KB_US,	pckbd_keydesc_iopener),
 	KBD_MAP(KB_UK | KB_SWAPCTRLCAPS, KB_UK,	pckbd_keydesc_swapctrlcaps),
 	KBD_MAP(KB_JP | KB_SWAPCTRLCAPS, KB_JP,	pckbd_keydesc_swapctrlcaps),
 	KBD_MAP(KB_FR | KB_SWAPCTRLCAPS, KB_FR,	pckbd_keydesc_swapctrlcaps),
@@ -1161,8 +1144,6 @@
 	KBD_MAP(KB_US | KB_DVORAK | KB_SWAPCTRLCAPS,	KB_US | KB_DVORAK,
 		pckbd_keydesc_swapctrlcaps),
 	KBD_MAP(KB_US | KB_COLEMAK | KB_SWAPCTRLCAPS,	KB_US | KB_COLEMAK,
-		pckbd_keydesc_swapctrlcaps),
-	KBD_MAP(KB_US | KB_IOPENER | KB_SWAPCTRLCAPS,	KB_US | KB_IOPENER,
 		pckbd_keydesc_swapctrlcaps),
 	KBD_MAP(KB_ES,			KB_US,	pckbd_keydesc_es),
 	KBD_MAP(KB_BE,			KB_US,	pckbd_keydesc_be),