On 11/15/2016 04:52 PM, Andre Przywara wrote:
> The whole error message should be within the else clause, not just
> the first part.
> Pointed out by GCC 6.2's -Wmisleading-indentation warning.
>
> Signed-off-by: Andre Przywara <andre.przyw...@arm.com>
> ---
>  board/freescale/ls2080aqds/eth.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/board/freescale/ls2080aqds/eth.c 
> b/board/freescale/ls2080aqds/eth.c
> index 95ff68b..7bf7a5b 100644
> --- a/board/freescale/ls2080aqds/eth.c
> +++ b/board/freescale/ls2080aqds/eth.c
> @@ -196,12 +196,13 @@ static void sgmii_configure_repeater(int serdes_port)
>                                              value);
>                                       i = 5;
>                                       j = 5;
> -                             } else
> +                             } else {
>                                       printf("DPMAC %d :PHY is failed to ",
>                                              dpmac_id);
>                                       printf("configure the repeater 0x%x\n",
>                                              value);
>                               }
> +                     }
>               }
>       }
>  error:
>

Thanks for finding this. This is being addressed by 
http://patchwork.ozlabs.org/patch/682885/. Currently in my test queue.

York
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to