Hi Tim,

On 09/05/2015 03:28, Tim Harvey wrote:
> The re-assignment of pcie_rst gpio for GW522x needs to occur earlier, before
> the PCI subsystem calls the toggle funciton.
> 
> Signed-off-by: Tim Harvey <thar...@gateworks.com>
> ---
>  board/gateworks/gw_ventana/gw_ventana.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/board/gateworks/gw_ventana/gw_ventana.c 
> b/board/gateworks/gw_ventana/gw_ventana.c
> index 554cd84..8cfc1f2 100644
> --- a/board/gateworks/gw_ventana/gw_ventana.c
> +++ b/board/gateworks/gw_ventana/gw_ventana.c
> @@ -1184,10 +1184,6 @@ static void setup_board_gpio(int board)
>       }
>  
>  #if !defined(CONFIG_CMD_PCI)
> -     /* GW522x Uses GPIO3_IO23 for PCIE_RST# */
> -     if (board_type == GW52xx && info->model[4] == '2')
> -             gpio_cfg[board].pcie_rst = IMX_GPIO_NR(3, 23);
> -

This code runs now without #ifdef protection. Is it desired ?

>       /* assert PCI_RST# (released by OS when clock is valid) */
>       gpio_request(gpio_cfg[board].pcie_rst, "pci_rst#");
>       gpio_direction_output(gpio_cfg[board].pcie_rst, 0);
> @@ -1435,6 +1431,10 @@ int board_init(void)
>               int count = gpio_cfg[board_type].num_pads;
>  
>               imx_iomux_v3_setup_multiple_pads(p, count);
> +
> +             /* GW522x Uses GPIO3_IO23 for PCIE_RST# */
> +             if (board_type == GW52xx && ventana_info.model[4] == '2')
> +                     gpio_cfg[board_type].pcie_rst = IMX_GPIO_NR(3, 23);
>       }
>  
>       return 0;
> 

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to