9front - general discussion about 9front
 help / color / mirror / Atom feed
* Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
       [not found] <hg.95e095480480.1585406323.4892889753185008284@code.9front.org>
@ 2020-03-30 22:56 ` Lyndon Nerenberg
  2020-03-31  1:25   ` [9front] " Sigrid Solveig Haflínudóttir
                     ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Lyndon Nerenberg @ 2020-03-30 22:56 UTC (permalink / raw)
  To: 9front

A bit of a nitpick here, but /dev/repeat is an awfully generic filename.
Wouldn't something like /dev/kbdrate be a bit more obvious?


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-30 22:56 ` [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati Lyndon Nerenberg
@ 2020-03-31  1:25   ` Sigrid Solveig Haflínudóttir
  2020-03-31  1:28   ` Alex Musolino
  2020-04-04 16:19   ` cinap_lenrek
  2 siblings, 0 replies; 9+ messages in thread
From: Sigrid Solveig Haflínudóttir @ 2020-03-31  1:25 UTC (permalink / raw)
  To: 9front

[-- Attachment #1: Type: text/plain, Size: 178 bytes --]

It is indeed. So is all of kbd, like /dev/leds and /dev/scancode. There is
also kbdfs that reads scancodes and provides several /dev/kbd* files. Not
sure what's best to do here.

[-- Attachment #2: Type: text/html, Size: 204 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-30 22:56 ` [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati Lyndon Nerenberg
  2020-03-31  1:25   ` [9front] " Sigrid Solveig Haflínudóttir
@ 2020-03-31  1:28   ` Alex Musolino
  2020-03-31  1:37     ` Sigrid Solveig Haflínudóttir
  2020-04-04 16:19   ` cinap_lenrek
  2 siblings, 1 reply; 9+ messages in thread
From: Alex Musolino @ 2020-03-31  1:28 UTC (permalink / raw)
  To: 9front

Furthermore, as much as I don't really care about PS/2 keyboards, the
interface is really very un-plan9.  I don't want to be dealing with
bitmasks specific to PS/2 controllers to set this kind of thing.
Instead /dev/kbdrate should accept two integers: the initial delay (in
ms), followed by the repeat delay (also in ms).  That kind of
interface will also work for other keyboard controllers.


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-31  1:28   ` Alex Musolino
@ 2020-03-31  1:37     ` Sigrid Solveig Haflínudóttir
  0 siblings, 0 replies; 9+ messages in thread
From: Sigrid Solveig Haflínudóttir @ 2020-03-31  1:37 UTC (permalink / raw)
  To: 9front

[-- Attachment #1: Type: text/plain, Size: 248 bytes --]

Happy to change it (add to kbdfs?) once we can figure a way to do it for
nusb/kb. Btw /dev/leds is a bitmask. The addition of repeat and delay
seemed like extremely ps2 specific thing here in general, considering it's
still _a ps/2 kernel driver_.

[-- Attachment #2: Type: text/html, Size: 275 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-30 22:56 ` [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati Lyndon Nerenberg
  2020-03-31  1:25   ` [9front] " Sigrid Solveig Haflínudóttir
  2020-03-31  1:28   ` Alex Musolino
@ 2020-04-04 16:19   ` cinap_lenrek
  2 siblings, 0 replies; 9+ messages in thread
From: cinap_lenrek @ 2020-04-04 16:19 UTC (permalink / raw)
  To: 9front

yes, i agree. note that scancode filename was choosen because of inferno's
cons(3) scancode file. we use the same format.

--
cinap


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-04-01  8:30 ` Ethan Gardener
@ 2020-04-01  9:37   ` Sigrid Solveig Haflínudóttir
  0 siblings, 0 replies; 9+ messages in thread
From: Sigrid Solveig Haflínudóttir @ 2020-04-01  9:37 UTC (permalink / raw)
  To: 9front

I reverted the change. Feel free to provide a better patch.


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-31  1:53 Alex Musolino
  2020-03-31  2:07 ` Sigrid Solveig Haflínudóttir
@ 2020-04-01  8:30 ` Ethan Gardener
  2020-04-01  9:37   ` Sigrid Solveig Haflínudóttir
  1 sibling, 1 reply; 9+ messages in thread
From: Ethan Gardener @ 2020-04-01  8:30 UTC (permalink / raw)
  To: 9front

On Tue, Mar 31, 2020, at 2:53 AM, Alex Musolino wrote:
> A bit mask is much easier to grok for turning LEDs on and off compared
> to this.  I mean, I have to mask in 5 bits and work out how they
> map (backwards!) into a 30Hz to 2Hz range?!  C'mon...

I haven't looked at the commit, but this sounds like one of the worst evils of Linux: direct explosure of evil hardware nonsense without putting a simple interface on it. What I'm saying is, "lets fix this interface before people enshrine it in their scripts." I'll have a look at the numbers later, if my health permits.


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
  2020-03-31  1:53 Alex Musolino
@ 2020-03-31  2:07 ` Sigrid Solveig Haflínudóttir
  2020-04-01  8:30 ` Ethan Gardener
  1 sibling, 0 replies; 9+ messages in thread
From: Sigrid Solveig Haflínudóttir @ 2020-03-31  2:07 UTC (permalink / raw)
  To: 9front

[-- Attachment #1: Type: text/plain, Size: 41 bytes --]

Yeah that's a typo, thanks for spotting.

[-- Attachment #2: Type: text/html, Size: 67 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [9front] Re: [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati...
@ 2020-03-31  1:53 Alex Musolino
  2020-03-31  2:07 ` Sigrid Solveig Haflínudóttir
  2020-04-01  8:30 ` Ethan Gardener
  0 siblings, 2 replies; 9+ messages in thread
From: Alex Musolino @ 2020-03-31  1:53 UTC (permalink / raw)
  To: 9front

A bit mask is much easier to grok for turning LEDs on and off compared
to this.  I mean, I have to mask in 5 bits and work out how they
map (backwards!) into a 30Hz to 2Hz range?!  C'mon...

Also, why is the man page talking about initial *delays* of 250Hz to
1000Hz?  Shouldn't that me milliseconds?


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-04-04 16:19 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <hg.95e095480480.1585406323.4892889753185008284@code.9front.org>
2020-03-30 22:56 ` [9front-commits] ftrvxmtrx: hg/plan9front: kbd: add "repeat" file to set typemati Lyndon Nerenberg
2020-03-31  1:25   ` [9front] " Sigrid Solveig Haflínudóttir
2020-03-31  1:28   ` Alex Musolino
2020-03-31  1:37     ` Sigrid Solveig Haflínudóttir
2020-04-04 16:19   ` cinap_lenrek
2020-03-31  1:53 Alex Musolino
2020-03-31  2:07 ` Sigrid Solveig Haflínudóttir
2020-04-01  8:30 ` Ethan Gardener
2020-04-01  9:37   ` Sigrid Solveig Haflínudóttir

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).