Re: [U-Boot] [v3 patch] net: fman: Support both new and legacy FMan Compatibles

2018-09-28 Thread York Sun
On 08/28/2018 08:49 PM, 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 
> ---
> Changes for v2
>   - add kernel version and commit id modifing the binding
> Changes for v3
>   - put "fsl,fman-xgec" before "fsl,fman-tgec"
>   - because it use the former in latest kernel version
> 

Applied to fsl-qoriq master, awaiting upstream. Thanks.

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


Re: [U-Boot] [v3 patch] net: fman: Support both new and legacy FMan Compatibles

2018-09-21 Thread York Sun
On 09/20/2018 07:57 PM, Qiang Zhao wrote:
> Hi York,
> 
> It seems nobody have comments on this patch,
> Could you help to take action on it?
> 

I will try when I get a chance. Thanks.

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


Re: [U-Boot] [v3 patch] net: fman: Support both new and legacy FMan Compatibles

2018-09-20 Thread Qiang Zhao
Hi York,

It seems nobody have comments on this patch,
Could you help to take action on it?

Best Regards
Qiang Zhao

> -Original Message-
> From: Zhao Qiang 
> Sent: 2018年8月29日 11:47
> To: York Sun ; joe.hershber...@ni.com
> Cc: u-boot@lists.denx.de; Qiang Zhao 
> Subject: [v3 patch] net: fman: Support both new and legacy FMan Compatibles
> 
> 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 
> ---
> Changes for v2
>   - add kernel version and commit id modifing the binding Changes for v3
>   - put "fsl,fman-xgec" before "fsl,fman-tgec"
>   - because it use the former in latest kernel version
> 
>  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 
> 147d043..f896e80
> 100644
> --- a/drivers/net/fm/init.c
> +++ b/drivers/net/fm/init.c
> @@ -328,7 +328,8 @@ void fdt_fixup_fman_ethernet(void *blob)
>   ft_fixup_port(blob, _info[i],
> "fsl,fman-1g-mac");
>   } else {
> - if (ft_fixup_port(blob, _info[i], "fsl,fman-tgec"))
> + if (ft_fixup_port(blob, _info[i], "fsl,fman-xgec") &&
> + ft_fixup_port(blob, _info[i], "fsl,fman-tgec"))
>   ft_fixup_port(blob, _info[i],
> "fsl,fman-10g-mac");
>   }
> --
> 1.7.1

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


[U-Boot] [v3 patch] net: fman: Support both new and legacy FMan Compatibles

2018-08-28 Thread Zhao Qiang
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 
---
Changes for v2
- add kernel version and commit id modifing the binding
Changes for v3
- put "fsl,fman-xgec" before "fsl,fman-tgec"
- because it use the former in latest kernel version

 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 147d043..f896e80 100644
--- a/drivers/net/fm/init.c
+++ b/drivers/net/fm/init.c
@@ -328,7 +328,8 @@ void fdt_fixup_fman_ethernet(void *blob)
ft_fixup_port(blob, _info[i],
  "fsl,fman-1g-mac");
} else {
-   if (ft_fixup_port(blob, _info[i], "fsl,fman-tgec"))
+   if (ft_fixup_port(blob, _info[i], "fsl,fman-xgec") &&
+   ft_fixup_port(blob, _info[i], "fsl,fman-tgec"))
ft_fixup_port(blob, _info[i],
  "fsl,fman-10g-mac");
}
-- 
1.7.1

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