[lkml]   [2007]   [May]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/2] leds:arch/sh/boards/landisk LEDs supports
    On Tue, 2007-05-08 at 21:27 +0900, kogiidena wrote:
    > I wrote LED Blink Trigger. Pleas apply if there is no problem.
    > This allows LEDs to be controlled by a programmable timer.
    > "blink" blinks LED at a constant cycle.
    > "bitshift" turns LED or Buzzer on and off by the value of pattern.
    > Kernel version: linux-2.6.21-rc4
    > Signed-off-by: kogiidena <>

    As far as I can tell, the existing timer trigger can do everything the
    blink trigger can (and more besides). The driver looks like it would be
    a lot easier to read if it just contained the bitshift trigger. To be
    accepted, the bitshift trigger needs more documentation, particularly
    the Kconfig entry. Its not clear for example that the bits are just
    shifted once and don't repeat. Most people would probably want it to
    repeat and with a repeat mode, it might be able to replace the heartbeat
    trigger too?



    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-05-08 14:49    [W:0.019 / U:168.512 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site