Re: [U-Boot] [PATCH 3/3] microblaze: bootm: Fix compiler warning

[Michal Simek]

On 12.2.2018 15:46, Tom Rini wrote:
> On Mon, Feb 12, 2018 at 05:54:37PM +0800, Bin Meng wrote:
> 
>> Fix build warning in arch/microblaze/lib/bootm.c with gcc 7.3.0:
>>
>>   warning: this 'if' clause does not guard... [-Wmisleading-indentation]
>>
>> Signed-off-by: Bin Meng 
> 
> Reviewed-by: Tom Rini 
> 

Reviewed-by: Michal Simek 

If you want me to apply via my tree, I am happy to do that.
You have sent it as series that's why I am asking.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs




signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] microblaze: bootm: Fix compiler warning

[Tom Rini]

On Mon, Feb 12, 2018 at 05:54:37PM +0800, Bin Meng wrote:

> Fix build warning in arch/microblaze/lib/bootm.c with gcc 7.3.0:
> 
>   warning: this 'if' clause does not guard... [-Wmisleading-indentation]
> 
> Signed-off-by: Bin Meng 

Reviewed-by: Tom Rini 

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

[U-Boot] [PATCH 3/3] microblaze: bootm: Fix compiler warning

[Bin Meng]

Fix build warning in arch/microblaze/lib/bootm.c with gcc 7.3.0:

  warning: this 'if' clause does not guard... [-Wmisleading-indentation]

Signed-off-by: Bin Meng 
---

 arch/microblaze/lib/bootm.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/microblaze/lib/bootm.c b/arch/microblaze/lib/bootm.c
index 0a286e8..93525cc 100644
--- a/arch/microblaze/lib/bootm.c
+++ b/arch/microblaze/lib/bootm.c
@@ -62,11 +62,12 @@ int do_bootm_linux(int flag, int argc, char * const argv[],
of_flat_tree = (char *)simple_strtoul(argv[1], NULL, 16);
 
/* fixup the initrd now that we know where it should be */
-   if (images->rd_start && images->rd_end && of_flat_tree)
+   if (images->rd_start && images->rd_end && of_flat_tree) {
ret = fdt_initrd(of_flat_tree, images->rd_start,
 images->rd_end);
if (ret)
return 1;
+   }
 
 #ifdef DEBUG
printf("## Transferring control to Linux (at address 0x%08lx) ",
-- 
2.7.4

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