[U-Boot] [PATCH] malloc: Turn on DEBUG when enabling unit tests
Unit tests require mallinfo which in turn requires DEBUG on dlmalloc to be enabled.
The dependancy on CONFIG_SANDBOX is wrong.
Signed-off-by: Pantelis Antoniou pantelis.antoniou@konsulko.com --- common/dlmalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/dlmalloc.c b/common/dlmalloc.c index adc680e..fc1e8b3 100644 --- a/common/dlmalloc.c +++ b/common/dlmalloc.c @@ -1,6 +1,6 @@ #include <common.h>
-#ifdef CONFIG_SANDBOX +#if defined(CONFIG_UNIT_TEST) #define DEBUG #endif
On Thu, May 25, 2017 at 07:24:06PM +0300, Pantelis Antoniou wrote:
Unit tests require mallinfo which in turn requires DEBUG on dlmalloc to be enabled.
The dependancy on CONFIG_SANDBOX is wrong.
Signed-off-by: Pantelis Antoniou pantelis.antoniou@konsulko.com
Reviewed-by: Tom Rini trini@konsulko.com
On 25 May 2017 at 10:24, Pantelis Antoniou pantelis.antoniou@konsulko.com wrote:
Unit tests require mallinfo which in turn requires DEBUG on dlmalloc to be enabled.
The dependancy on CONFIG_SANDBOX is wrong.
Signed-off-by: Pantelis Antoniou pantelis.antoniou@konsulko.com
common/dlmalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
On Thu, May 25, 2017 at 07:24:06PM +0300, Pantelis Antoniou wrote:
Unit tests require mallinfo which in turn requires DEBUG on dlmalloc to be enabled.
The dependancy on CONFIG_SANDBOX is wrong.
Signed-off-by: Pantelis Antoniou pantelis.antoniou@konsulko.com Reviewed-by: Tom Rini trini@konsulko.com Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot/master, thanks!
participants (3)
-
Pantelis Antoniou -
Simon Glass -
Tom Rini