[U-Boot] [PATCH] phy: Fix off-by-one error when parsing DT PHY bindings

[Patrice CHOTARD]

Hi Marek

On 08/08/2018 02:34 PM, Marek Vasut wrote:
> The code fails to copy the last PHY phandle argument, so it is
> missing from the adjusted phandle args and the consumer cannot
> use it to determine what the PHY should do.
> 
> Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> Cc: Patrice Chotard <patrice.chotard at st.com>
> ---
>   drivers/phy/phy-uclass.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/phy/phy-uclass.c b/drivers/phy/phy-uclass.c
> index 6162395e75..a0ac30aa71 100644
> --- a/drivers/phy/phy-uclass.c
> +++ b/drivers/phy/phy-uclass.c
> @@ -64,7 +64,7 @@ int generic_phy_get_by_index(struct udevice *dev, int index,
>   			return ret;
>   
>   		/* insert phy idx at first position into args array */
> -		for (i = args.args_count; i > 1 ; i--)
> +		for (i = args.args_count; i >= 1 ; i--)
>   			args.args[i] = args.args[i - 1];
>   
>   		args.args_count++;
> 

Reviewed-by: Patrice Chotard <patrice.chotard at st.com>

Thanks