net: fec: prevent undesired de-assertion of phy-reset on request
When gpio_request_by_name allocates a gpio output it by default will de-assert the gpio which for phy-reset will take the PHY out of reset. As this occurs before fec_gpio_reset is called to assert the reset line it can cause undesired affects if reset timings are not properly met. Configure the gpio with GPIOD_IS_OUT_ACTIVE so that reset is kept active (reset asserted) to avoid this. Cc: Sean Anderson <sean.anderson@seco.com> Signed-off-by: Tim Harvey <tharvey@gateworks.com> Acked-by: Joe Hershberger <joe.hershberger@ni.com> Tested-by: Adam Ford <aford173@gmail.com> #imx8mm-beacon Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
This commit is contained in:
		
							parent
							
								
									e24b58f5ed
								
							
						
					
					
						commit
						4223fb0ee1
					
				|  | @ -1324,7 +1324,7 @@ static int fecmxc_of_to_plat(struct udevice *dev) | ||||||
| 
 | 
 | ||||||
| #if CONFIG_IS_ENABLED(DM_GPIO) | #if CONFIG_IS_ENABLED(DM_GPIO) | ||||||
| 	ret = gpio_request_by_name(dev, "phy-reset-gpios", 0, | 	ret = gpio_request_by_name(dev, "phy-reset-gpios", 0, | ||||||
| 				   &priv->phy_reset_gpio, GPIOD_IS_OUT); | 				   &priv->phy_reset_gpio, GPIOD_IS_OUT | GPIOD_IS_OUT_ACTIVE); | ||||||
| 	if (ret < 0) | 	if (ret < 0) | ||||||
| 		return 0; /* property is optional, don't return error! */ | 		return 0; /* property is optional, don't return error! */ | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue