[U-Boot] [PATCH 1/2] mtd: ubi debug: Remove the pid print from ubi_assert
Eran Matityahu
eran.m at variscite.com
Mon Feb 18 07:44:40 UTC 2019
Hi Heiko.
On Mon, Feb 18, 2019 at 7:06 AM Heiko Schocher <hs at denx.de> wrote:
>
> Hello Eran,
>
> Am 13.02.2019 um 19:55 schrieb Eran Matityahu:
> > Add a new definition for ubi_assert and keep
> > the original one in an ifndef __UBOOT__.
> >
> > Signed-off-by: Eran Matityahu <eran.m at variscite.com>
> > ---
> > drivers/mtd/ubi/debug.h | 10 ++++++++++
> > 1 file changed, 10 insertions(+)
>
> Is there any reason for this change?
>
> If I see it correct, pid is for U-Boot always set to one in
> ./lib/linux_compat.c ... so I see no reason for introducing here
> an U-Boot specific version of ubi_assert() ...
>
> bye,
> Heiko
> --
> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: hs at denx.de
Sure, it works with the pid print, however:
1. The pid print is useless in U-Boot.
2. I wanted to align it with ubifs_assert() and the rest of the macros in
fs/ubifs/debug.h, which also have U-Boot specific versions without the
pid print.
3. If you agree with the next patch I sent (using pr_debug), then it's
probably best to have a U-Boot specific version for ubi_assert()
anyway.
Regards,
Eran
--
Eran Matityahu | Software manager
Website | www.variscite.com Variscite wiki | www.variwiki.com
More information about the U-Boot
mailing list