linux-kernel - Re: [PATCH] leds: ledtrig-morse: send out morse code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdXaPz3ckT1cPZed4e=iwk3nb75UBGC66hm+b+xRDNNnNg@mail.gmail.com>
Date:   Fri, 29 Jun 2018 09:21:24 +0200
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     Pavel Machek <pavel@....cz>
Cc:     Andreas Klinger <ak@...klinger.de>,
        Jacek Anaszewski <jacek.anaszewski@...il.com>,
        Ben Whitten <ben.whitten@...il.com>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Willy Tarreau <w@....eu>,
        Philippe Ombredanne <pombredanne@...b.com>,
        Greg KH <gregkh@...uxfoundation.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-leds@...r.kernel.org
Subject: Re: [PATCH] leds: ledtrig-morse: send out morse code

Hi Pavel,

On Fri, Jun 29, 2018 at 9:17 AM Pavel Machek <pavel@....cz> wrote:
> > > Yeah, well... I don't think decoding sentences in morse code is going
> > > to be much fun.
> > >
> > > And we don't really want encoder in kernel. Just do encoding in
> > > userspace, and use pattern trigger to display it.
> > >
> > > For many uses, morse code is "too geeky", and other patterns will be
> > > used.
> > >
> > > Like " X    " for error 1, " X X     " for error 2, " .xX .xX " for
> > > charging, " .xXx. " for everything okay...
> > >
> >
> > The hardware i'm using is not able to adjust brightness. It can just switch the
> > LED on or off. That's it.
> >
> > If anybody is interested i can submit a version 2 of the morse trigger with the
> > improvements suggested by Greg and Geert. Please let me know.
>
> I'd prefer more general pattern trigger.

There are two parts here:
  1. The general pattern trigger (which is not yet upstream),
  2. A way to supply a string, convert it into morse code, and feed it to
     the LED subsystem.

Given 1 is not yet upstream, I think Andreas solution is fine.
Once 1 is upstream, 2 can be modified to feed into 1, if available.

Do you agree?
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds