lkml.org 
[lkml]   [2018]   [Jul]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2] leds: ledtrig-morse: send out morse code
    Hi Pavel,

    On Wed, Jul 04, 2018 at 08:53:05AM +0200, Pavel Machek wrote:
    > As I stated before, I don't think morse encoder belongs in kernel.

    On the opposite, I think that the kernel needs to be a bit more autonomous
    when it comes to reporting its own issues. Being able to report a panic
    when userland cannot be accessed for example is the reason why we've seen
    various features such as blinking keyboard LEDs for this.

    > LED pattern trigger should be merged, instead.

    Well, just like we have LED and LED triggers in the kernel, I think having
    a generic way to use patterns could be nice and in this case Morse could be
    one such pattern, but if that means it's limited to userland to configure
    it then it sadly voids all of its benefits.

    Last, as I showed on my previous mail in this thread, the Morse encoding
    can be brought to 36 bytes, which is much less than even the registration
    code needed to feed it. At some point we need to focus on code efficiency
    and doing things right instead of how too many layers look like from far
    away. While I don't need Morse right now I consider it as a nice addition
    I would definitely enable by default on all my boards if it helps provide
    details about what prevents my system from booting or what just killed it.

    Cheers,
    Willy

    \
     
     \ /
      Last update: 2018-07-04 09:36    [W:2.221 / U:0.136 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site