On 08/06/2018 12:16 AM, Zhao Qiang wrote:
> Recently  the FMan Port and MAC compatibles were changed.
> This patch aligns the FMan Port and MAC compatibles
> to the new FMan device tree binding document.
> The FMan device tree binding document can be found in the Linux
> kernel:
> version 3.18.0
> commit 297d35fd2a7d3fbd4e5c0f0c1c18213117ba11ba
> ./Documentation/devicetree/bindings/powerpc/fsl/fman.txt
> 
> This patch doesn't affect legacy compatibles support.
> 
> Signed-off-by: Zhao Qiang <qiang.z...@nxp.com>
> ---
> Changes for v2
>       - add kernel version and commit id modifing the binding
> 
>  drivers/net/fm/init.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/net/fm/init.c b/drivers/net/fm/init.c
> index 71c3abe..48344f0 100644
> --- a/drivers/net/fm/init.c
> +++ b/drivers/net/fm/init.c
> @@ -329,7 +329,8 @@ void fdt_fixup_fman_ethernet(void *blob)
>                               ft_fixup_port(blob, &fm_info[i],
>                                             "fsl,fman-1g-mac");
>               } else {
> -                     if (ft_fixup_port(blob, &fm_info[i], "fsl,fman-tgec"))
> +                     if (ft_fixup_port(blob, &fm_info[i], "fsl,fman-tgec") &&
> +                         ft_fixup_port(blob, &fm_info[i], "fsl,fman-xgec"))

Is this logic correct? You want success return from fixing up _both_
fman-tgec and fman-xgec before fixup fman-10-mac?

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

Reply via email to