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
