diff mbox series

[8/8] arm: socfpga: vining: Let DWMAC configure PHY reset GPIO

Message ID 20210914032536.273530-8-marex@denx.de
State Accepted
Commit 9de0f144c70899f498ba2b90ae2758dcf79ce481
Delegated to: Simon Goldschmidt
Headers show
Series [1/8] arm: socfpga: vining: Drop meaningless comment | expand

Commit Message

Marek Vasut Sept. 14, 2021, 3:25 a.m. UTC
The DM DWMAC driver is perfectly capable of configuring the ethernet
PHY reset GPIO, let the driver do it instead of doing it in the board
file.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Siew Chin Lim <elly.siew.chin.lim@intel.com>
Cc: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Cc: Tien Fong Chee <tien.fong.chee@intel.com>
---
 board/softing/vining_fpga/socfpga.c | 7 -------
 1 file changed, 7 deletions(-)

Comments

Chee, Tien Fong Sept. 27, 2021, 2:40 a.m. UTC | #1
> -----Original Message-----
> From: Marek Vasut <marex@denx.de>
> Sent: Tuesday, 14 September, 2021 11:26 AM
> To: u-boot@lists.denx.de
> Cc: Marek Vasut <marex@denx.de>; Lim, Elly Siew Chin
> <elly.siew.chin.lim@intel.com>; Simon Goldschmidt
> <simon.k.r.goldschmidt@gmail.com>; Chee, Tien Fong
> <tien.fong.chee@intel.com>
> Subject: [PATCH 8/8] arm: socfpga: vining: Let DWMAC configure PHY reset
> GPIO
> 
> The DM DWMAC driver is perfectly capable of configuring the ethernet PHY
> reset GPIO, let the driver do it instead of doing it in the board file.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Siew Chin Lim <elly.siew.chin.lim@intel.com>
> Cc: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
> Cc: Tien Fong Chee <tien.fong.chee@intel.com>
> ---
>  board/softing/vining_fpga/socfpga.c | 7 -------
>  1 file changed, 7 deletions(-)
> 

Reviewed-by: Tien Fong Chee <tien.fong.chee@intel.com>

Regards,
TF
diff mbox series

Patch

diff --git a/board/softing/vining_fpga/socfpga.c b/board/softing/vining_fpga/socfpga.c
index aaedf034504..22992273911 100644
--- a/board/softing/vining_fpga/socfpga.c
+++ b/board/softing/vining_fpga/socfpga.c
@@ -23,7 +23,6 @@  DECLARE_GLOBAL_DATA_PTR;
  */
 int board_late_init(void)
 {
-	const unsigned int phy_nrst_gpio = 0;
 	const unsigned int usb_nrst_gpio = 35;
 	int ret;
 
@@ -33,12 +32,6 @@  int board_late_init(void)
 	/* Address of boot parameters for ATAG (if ATAG is used) */
 	gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100;
 
-	ret = gpio_request(phy_nrst_gpio, "phy_nrst_gpio");
-	if (!ret)
-		gpio_direction_output(phy_nrst_gpio, 1);
-	else
-		printf("Cannot remove PHY from reset!\n");
-
 	ret = gpio_request(usb_nrst_gpio, "usb_nrst_gpio");
 	if (!ret)
 		gpio_direction_output(usb_nrst_gpio, 1);