Re: [RFC PATCH 2/2] leds: trigger: implement block trigger

From: Marek Behun
Date: Fri Apr 30 2021 - 16:11:47 EST


On Fri, 30 Apr 2021 15:32:11 -0300
Enzo Matsumiya <ematsumiya@xxxxxxx> wrote:

> Activity is then represented in an accumulated manner (part_read_stat_accum()),
> with a fixed blinking interval of 50ms.

part_stat_read_accum, not part_read_stat_accum

Why only accum? With the netdev trigger, you can choose whether rx, tx,
or both are blinking the LED.

Also I think the trigger should be called "blockdev" instead of
"block". This is consistent with "netdev", and avoids misinterpretation
with the verb "to block".

Marek