On 12/10/18 1:27 AM, Florinel Iordache wrote:
> u-boot makes a fixup for LS1046AQDS board to setup the properties
>  'fixed-link' and 'phy-connection-type' to 'xgmii' but in case of
>  backplane mode this fixup is not correct because it causes the KR link
>  to fail and so it must be bypassed in order to keep the link in KR mode
>  as it is defined in DTS.
> 
> Signed-off-by: Florinel Iordache <florinel.iorda...@nxp.com>
> ---
>  board/freescale/ls1046aqds/eth.c | 35 +++++++++++++++++++++++++----------
>  1 file changed, 25 insertions(+), 10 deletions(-)
> 
> diff --git a/board/freescale/ls1046aqds/eth.c 
> b/board/freescale/ls1046aqds/eth.c
> index d3e8831..1fd755c 100644
> --- a/board/freescale/ls1046aqds/eth.c
> +++ b/board/freescale/ls1046aqds/eth.c
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0+
>  /*
>   * Copyright 2016 Freescale Semiconductor, Inc.
> + * Copyright 2018 NXP
>   */
>  
>  #include <common.h>
> @@ -153,6 +154,9 @@ void board_ft_fman_fixup_port(void *fdt, char *compat, 
> phys_addr_t addr,
>                             enum fm_port port, int offset)
>  {
>       struct fixed_link f_link;
> +     u32 *handle;
> +     char *prop = NULL;
> +     int off;
>  
>       if (fm_info_get_enet_if(port) == PHY_INTERFACE_MODE_SGMII) {
>               switch (port) {
> @@ -208,16 +212,27 @@ void board_ft_fman_fixup_port(void *fdt, char *compat, 
> phys_addr_t addr,
>                                  "qsgmii");
>       } else if (fm_info_get_enet_if(port) == PHY_INTERFACE_MODE_XGMII &&
>                  (port == FM1_10GEC1 || port == FM1_10GEC2)) {
> -             /* XFI interface */
> -             f_link.phy_id = cpu_to_fdt32(port);
> -             f_link.duplex = cpu_to_fdt32(1);
> -             f_link.link_speed = cpu_to_fdt32(10000);
> -             f_link.pause = 0;
> -             f_link.asym_pause = 0;
> -             /* no PHY for XFI */
> -             fdt_delprop(fdt, offset, "phy-handle");
> -             fdt_setprop(fdt, offset, "fixed-link", &f_link, sizeof(f_link));
> -             fdt_setprop_string(fdt, offset, "phy-connection-type", "xgmii");
> +             handle = fdt_getprop(fdt, offset, "phy-handle", NULL);

warning: assignment discards ‘const’ qualifier from pointer target type
[-Wdiscarded-qualifiers]

Pleas fix.

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

Reply via email to