Re: [U-Boot] [PATCH v3 03/14] fdt: Switch to using the verbose overlay application method

2017-09-15 Thread Simon Glass 
On 8 September 2017 at 22:53, Simon Glass  wrote:
> On 4 September 2017 at 14:12, Pantelis Antoniou
>  wrote:
>> The verbose overlay application method prints out more helpful
>> messages, so switch to it.
>>
>> Signed-off-by: Pantelis Antoniou 
>> ---
>>  cmd/fdt.c | 7 +++
>>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> Acked-by: Simon Glass 

Applied to u-boot-fdt, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3 03/14] fdt: Switch to using the verbose overlay application method

2017-09-08 Thread Simon Glass 
On 4 September 2017 at 14:12, Pantelis Antoniou
 wrote:
> The verbose overlay application method prints out more helpful
> messages, so switch to it.
>
> Signed-off-by: Pantelis Antoniou 
> ---
>  cmd/fdt.c | 7 +++
>  1 file changed, 3 insertions(+), 4 deletions(-)

Acked-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v3 03/14] fdt: Switch to using the verbose overlay application method

2017-09-04 Thread Pantelis Antoniou 
The verbose overlay application method prints out more helpful
messages, so switch to it.

Signed-off-by: Pantelis Antoniou 
---
 cmd/fdt.c | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/cmd/fdt.c b/cmd/fdt.c
index 118613f..362a621 100644
--- a/cmd/fdt.c
+++ b/cmd/fdt.c
@@ -669,11 +669,10 @@ static int do_fdt(cmd_tbl_t *cmdtp, int flag, int argc, 
char * const argv[])
if (!fdt_valid())
return CMD_RET_FAILURE;
 
-   ret = fdt_overlay_apply(working_fdt, blob);
-   if (ret) {
-   printf("fdt_overlay_apply(): %s\n", fdt_strerror(ret));
+   /* apply method prints messages on error */
+   ret = fdt_overlay_apply_verbose(working_fdt, blob);
+   if (ret)
return CMD_RET_FAILURE;
-   }
}
 #endif
/* resize the fdt */
-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot