Re: [U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
On Fri, Aug 11, 2017 at 3:30 PM, Jagan Teki wrote: > On Tue, Jul 4, 2017 at 4:13 PM, Icenowy Zheng wrote: >> The Ethernet function is enabled in the Orange Pi PC2 device tree and >> defconfig, however, CONFIG_MACPWR is not properly set, which left the >> PHY being disabled when booting, which makes the Ethernet function not >> usable. >> >> Add the proper value of this option in the PC2 defconfig. >> >> Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") >> Signed-off-by: Icenowy Zheng > > Reviewed-by: Jagan Teki Applied to u-boot-sunxi/master thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
On Tue, Jul 4, 2017 at 4:13 PM, Icenowy Zheng wrote: > The Ethernet function is enabled in the Orange Pi PC2 device tree and > defconfig, however, CONFIG_MACPWR is not properly set, which left the > PHY being disabled when booting, which makes the Ethernet function not > usable. > > Add the proper value of this option in the PC2 defconfig. > > Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") > Signed-off-by: Icenowy Zheng Reviewed-by: Jagan Teki thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
Hi, On 04/07/17 11:48, Icenowy Zheng wrote: > 于 2017年7月4日 GMT+08:00 下午6:47:17, Andre Przywara 写到: >> Hi, >> >> On 04/07/17 11:43, Icenowy Zheng wrote: >>> The Ethernet function is enabled in the Orange Pi PC2 device tree and >>> defconfig, however, CONFIG_MACPWR is not properly set, which left the >>> PHY being disabled when booting, which makes the Ethernet function >> not >>> usable. >> >> Indeed, so I can remove this very same patch from my queue here ;-) >> >>> Add the proper value of this option in the PC2 defconfig. >>> >>> Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") >> >> I believe the support for CONFIG_MACPWR wasn't there when I wrote this >> patch, so I hacked this into my ATF port at the time ;-) > > It used to be in CONFIG_EXTRA_OPTIONS, see the history of > H3 GbE OPis' defconfig. I know, but I didn't want to let this legacy hack creep into a new board. Cheers, Andre. >>> Signed-off-by: Icenowy Zheng >> >> Acked-by: Andre Przywara >> >> Thanks, >> Andre. >> >>> --- >>> configs/orangepi_pc2_defconfig | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/configs/orangepi_pc2_defconfig >> b/configs/orangepi_pc2_defconfig >>> index 5a64ad3f41..b72514d3b9 100644 >>> --- a/configs/orangepi_pc2_defconfig >>> +++ b/configs/orangepi_pc2_defconfig >>> @@ -4,6 +4,7 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y >>> CONFIG_MACH_SUN50I_H5=y >>> CONFIG_DRAM_CLK=672 >>> CONFIG_DRAM_ZQ=3881977 >>> +CONFIG_MACPWR="PD6" >>> CONFIG_DEFAULT_DEVICE_TREE="sun50i-h5-orangepi-pc2" >>> # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set >>> CONFIG_SPL=y >>> ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
于 2017年7月4日 GMT+08:00 下午6:47:17, Andre Przywara 写到: >Hi, > >On 04/07/17 11:43, Icenowy Zheng wrote: >> The Ethernet function is enabled in the Orange Pi PC2 device tree and >> defconfig, however, CONFIG_MACPWR is not properly set, which left the >> PHY being disabled when booting, which makes the Ethernet function >not >> usable. > >Indeed, so I can remove this very same patch from my queue here ;-) > >> Add the proper value of this option in the PC2 defconfig. >> >> Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") > >I believe the support for CONFIG_MACPWR wasn't there when I wrote this >patch, so I hacked this into my ATF port at the time ;-) It used to be in CONFIG_EXTRA_OPTIONS, see the history of H3 GbE OPis' defconfig. > >> Signed-off-by: Icenowy Zheng > >Acked-by: Andre Przywara > >Thanks, >Andre. > >> --- >> configs/orangepi_pc2_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/configs/orangepi_pc2_defconfig >b/configs/orangepi_pc2_defconfig >> index 5a64ad3f41..b72514d3b9 100644 >> --- a/configs/orangepi_pc2_defconfig >> +++ b/configs/orangepi_pc2_defconfig >> @@ -4,6 +4,7 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y >> CONFIG_MACH_SUN50I_H5=y >> CONFIG_DRAM_CLK=672 >> CONFIG_DRAM_ZQ=3881977 >> +CONFIG_MACPWR="PD6" >> CONFIG_DEFAULT_DEVICE_TREE="sun50i-h5-orangepi-pc2" >> # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set >> CONFIG_SPL=y >> ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
Hi, On 04/07/17 11:43, Icenowy Zheng wrote: > The Ethernet function is enabled in the Orange Pi PC2 device tree and > defconfig, however, CONFIG_MACPWR is not properly set, which left the > PHY being disabled when booting, which makes the Ethernet function not > usable. Indeed, so I can remove this very same patch from my queue here ;-) > Add the proper value of this option in the PC2 defconfig. > > Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") I believe the support for CONFIG_MACPWR wasn't there when I wrote this patch, so I hacked this into my ATF port at the time ;-) > Signed-off-by: Icenowy Zheng Acked-by: Andre Przywara Thanks, Andre. > --- > configs/orangepi_pc2_defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configs/orangepi_pc2_defconfig b/configs/orangepi_pc2_defconfig > index 5a64ad3f41..b72514d3b9 100644 > --- a/configs/orangepi_pc2_defconfig > +++ b/configs/orangepi_pc2_defconfig > @@ -4,6 +4,7 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y > CONFIG_MACH_SUN50I_H5=y > CONFIG_DRAM_CLK=672 > CONFIG_DRAM_ZQ=3881977 > +CONFIG_MACPWR="PD6" > CONFIG_DEFAULT_DEVICE_TREE="sun50i-h5-orangepi-pc2" > # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set > CONFIG_SPL=y > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] sunxi: add MACPWR in Orange Pi PC2 defconfig
The Ethernet function is enabled in the Orange Pi PC2 device tree and defconfig, however, CONFIG_MACPWR is not properly set, which left the PHY being disabled when booting, which makes the Ethernet function not usable. Add the proper value of this option in the PC2 defconfig. Fixes: e7bd15ea156f ("sunxi: Add OrangePi PC 2 initial support") Signed-off-by: Icenowy Zheng --- configs/orangepi_pc2_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/orangepi_pc2_defconfig b/configs/orangepi_pc2_defconfig index 5a64ad3f41..b72514d3b9 100644 --- a/configs/orangepi_pc2_defconfig +++ b/configs/orangepi_pc2_defconfig @@ -4,6 +4,7 @@ CONFIG_SPL_SPI_FLASH_SUPPORT=y CONFIG_MACH_SUN50I_H5=y CONFIG_DRAM_CLK=672 CONFIG_DRAM_ZQ=3881977 +CONFIG_MACPWR="PD6" CONFIG_DEFAULT_DEVICE_TREE="sun50i-h5-orangepi-pc2" # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set CONFIG_SPL=y -- 2.13.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot