[PATCH v5 09/12] watchdog: wdt-uclass.c: handle all DM watchdogs in watchdog_reset()
Wolfgang Denk
wd at denx.de
Thu Aug 19 13:10:54 CEST 2021
Dear Rasmus,
In message <4798abb5-07d9-fa88-931f-dbaff951e3fb at prevas.dk> you wrote:
> >>
> >> + ret = uclass_get(UCLASS_WDT, &uc);
> >> + if (ret) {
> >> + log_debug("Error getting UCLASS_WDT: %d\n", ret);
> >> + return 0;
> >> + }
> >
> > Here the error goes silent, so we should fix the callers to report
> > it.
>
> The caller (singular) is the initr sequence, so returning an error is
> effectively the same as halting the boot process, and as I've already
> explained, I'm not going to change the semantics of initr_watchdog in
> this regard.
In this case you must print an error message here.
> Feel free to submit a patch if you feel a change in this area is in
> order. That's completely unrelated to what these patches are trying to
> achieve.
You add new code here, so please make sure not to add known issues.
> >> + uclass_foreach_dev(dev, uc) {
> >> + ret = device_probe(dev);
> >> + if (ret) {
> >> + log_debug("Error probing %s: %d\n", dev->name, ret);
> >> + continue;
> >> }
> >
> > Here the situation is different. The probing error is never
> > reported anywhere. Is it really a normal condition that a
> > device_probe() fails here?
>
> No, it is not a normal condition. I added the log_debug() after a
> request from Simon.
But log_debug() is nothing any user will see in the field. We need
an error message here, too.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If the odds are a million to one against something occuring, chances
are 50-50 it will.
More information about the U-Boot
mailing list