Hi all
On Wed, Aug 14, 2024 at 6:34 AM Heiko Schocher <h...@denx.de> wrote:
>
> Hello Christian,
>
> On 12.08.24 12:32, Christian Marangi wrote:
> > Implement support for LED activity. If the feature is enabled,
> > make the defined ACTIVITY LED to signal ubi write operation.
> >
> > Signed-off-by: Christian Marangi <ansuels...@gmail.com>
> > ---
> > cmd/ubi.c | 17 +++++++++++++++--
> > 1 file changed, 15 insertions(+), 2 deletions(-)
> >
> > diff --git a/cmd/ubi.c b/cmd/ubi.c
> > index 0e62e449327..6f679eae9c3 100644
> > --- a/cmd/ubi.c
> > +++ b/cmd/ubi.c
> > @@ -14,6 +14,7 @@
> > #include <command.h>
> > #include <env.h>
> > #include <exports.h>
> > +#include <led.h>
> > #include <malloc.h>
> > #include <memalign.h>
> > #include <mtd.h>
> > @@ -488,10 +489,22 @@ exit:
> >
> > int ubi_volume_write(char *volume, void *buf, loff_t offset, size_t size)
> > {
> > + int ret;
> > +
> > +#ifdef CONFIG_LED_ACTIVITY_ENABLE
> > + led_activity_blink();
> > +#endif
>
> Do we really need ifdef? May it is possible to declare an empty function
> when CONFIG_LED_ACTIVITY_ENABLE is not set? May this applies for the whole
> series?
>
> > +
> > if (!offset)
> > - return ubi_volume_begin_write(volume, buf, size, size);
> > + ret = ubi_volume_begin_write(volume, buf, size, size);
> > + else
> > + ret = ubi_volume_offset_write(volume, buf, offset, size);
> >
> > - return ubi_volume_offset_write(volume, buf, offset, size);
> > +#ifdef CONFIG_LED_ACTIVITY_ENABLE
> > + led_activity_off();
> > +#endif
> > +
> > + return ret;
> > }
> >
> > int ubi_volume_read(char *volume, char *buf, loff_t offset, size_t size)
> >
>
I rather prefer to have some registration of events that need to be executed for
a particular i/o activity and then a subscription process from led
subsystem if that
particular event is connected to the dts or just on a board file
Michael
> bye,
> Heiko
> --
> DENX Software Engineering GmbH, Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: h...@denx.de
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
mich...@amarulasolutions.com
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
i...@amarulasolutions.com
www.amarulasolutions.com
