From: Crystal Kolipe Subject: Re: remove IOPENER keyboard support (Re: improve wscons man pages) To: Matthieu Herrb Cc: tech@openbsd.org Date: Mon, 8 Sep 2025 19:54:20 +0100 On Sat, Aug 16, 2025 at 01:30:46PM +0200, Matthieu Herrb wrote: > On Fri, Aug 15, 2025 at 09:06:21PM +0100, Crystal Kolipe wrote: > > 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. > > That looks ok to me. Someone else? There have been no objections to this, can it be committed? > > > > --- 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), > > > > -- > Matthieu Herrb >