[U-Boot] [PATCH 1/2] mtd: ubi debug: Remove the pid print from ubi_assert

Heiko Schocher hs at denx.de
Mon Feb 18 12:15:37 UTC 2019


Hello Eran,

Am 18.02.2019 um 08:44 schrieb Eran Matityahu:
> 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.

Ah, I see. Ok, I have no objections.

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


More information about the U-Boot mailing list