On 08/09/2018 09:04 AM, Patrice CHOTARD wrote: > 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 <[email protected]> >> Cc: Patrice Chotard <[email protected]> >> --- >> 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 <[email protected]>
Who's picking this up ? -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
