Dear Nikita, In message <20200121221005.9303-1-coff...@gmail.com> you wrote: > In the case of error while device look up (e.g. wrong interface type > was specified) put a message about it even if debug was not enable. > > Signed-off-by: Nikita Ermakov <coff...@gmail.com> > --- > disk/part.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/disk/part.c b/disk/part.c > index 8982ef3bae..280e3e4f42 100644 > --- a/disk/part.c > +++ b/disk/part.c > @@ -513,7 +513,10 @@ int blk_get_device_part_str(const char *ifname, const > char *dev_part_str, > /* Look up the device */ > dev = blk_get_device_by_str(ifname, dev_str, dev_desc); > if (dev < 0) > + { > + printf("** Look up for the device %s %s failed **\n", ifname, > dev_str); > goto cleanup; > + }
Please see my other patch instead. You are using incorrect brace style, and I think your error message is a bit misleading - "look up failed" sounds like some internal error, the reason of which is still unclear. We should instead say clearly the the given device type is not supported/known in the given configuration. Thanks. 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: w...@denx.de That's the thing about people who think they hate computers. What they really hate is lousy programmers. - Larry Niven and Jerry Pournelle in "Oath of Fealty"