Re: [PATCH -ac] Panicking in morse code

Andrew Rodland (arodland@noln.com)
Fri, 19 Jul 2002 13:00:40 -0400


On Fri, 19 Jul 2002 16:36:55 +0000
Thorsten Kranzkowski <dl8bcu@dl8bcu.de> wrote:

> On Fri, Jul 19, 2002 at 01:13:00AM -0400, Andrew Rodland wrote:
> > I was researching panic_blink() for someone who needed a little
> > help, when I noticed the comment above the function definition, not
> > being the kind to step down from a challenge (unless it's just
> > really hard), I decided to write morse code output code.
>
> nice idea :)
>
> > diff -u -r -d linux.old/drivers/char/pc_keyb.c
> > linux/drivers/char/pc_keyb.c--- linux.old/drivers/char/pc_keyb.c Fri Jul 19 00:59:04 2002
> > +++ linux/drivers/char/pc_keyb.c Fri Jul 19 01:00:34 2002
> > @@ -1244,37 +1244,126 @@
> > #endif /* CONFIG_PSMOUSE */
>
> [...]
>
> > +static const char * morse[] = {
> > + ".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", /* A-H
> > */+ "..", ".---.", "-.-", ".-..", "--", "-.", "---", ".--.", /* I-P
> > */
>
> This should read:
>
> "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", /* I-P
> */
>
> i.e. there's a dot too much :)
>

Thank you. Fixed locally.
Since you seem to be something of a morse-wiz (the only letters I know
by heart are 'S', 'O', and 'S'), could you either make corrections on
my timings (ditlen, dahlen, inter-letter space, inter-word space), or
tell me that they're good?

--Andrew

P.S. Yes, in case anyone is wondering, I did create a module that does
nothing but generate a user-supplied panic. :)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/