On Thu, Nov 29, 2012 at 3:51 PM, Marek Vasut marex@denx.de wrote:
Dear Yegor Yefremov,
On Thu, Nov 29, 2012 at 3:29 PM, Marek Vasut marex@denx.de wrote:
Dear Yegor Yefremov,
On Thu, Nov 29, 2012 at 1:55 PM, Marek Vasut marex@denx.de wrote:
Dear Yegor Yefremov,
On Thu, Nov 29, 2012 at 8:53 AM, Henrik Nordström
henrik@henriknordstrom.net wrote: > tor 2012-11-29 klockan 07:27 +0100 skrev Marek Vasut: >> Dear yegorslists@googlemail.com, >> >> > From: Yegor Yefremov yegorslists@googlemail.com >> > >> > The driver code was taken from Linux kernel source: >> > drivers/net/phy/icplus.c >> > >> > Signed-off-by: Yegor Yefremov yegorslists@googlemail.com >> >> [...] >> >> Is there any user for this PHY please? > > I don't have one, but know at least one user having one in an A10 > based "netbook". > > http://linux-sunxi.org/File:H6-netbook-mainboard.jpg
I found these boards in kernel:
arch/mips/configs/malta_defconfig:CONFIG_ICPLUS_PHY=m arch/mips/configs/bcm47xx_defconfig:CONFIG_ICPLUS_PHY=m arch/mips/configs/ip27_defconfig:CONFIG_ICPLUS_PHY=m arch/mips/configs/sead3_defconfig:CONFIG_ICPLUS_PHY=y arch/powerpc/configs/mpc512x_defconfig:CONFIG_ICPLUS_PHY=y arch/powerpc/configs/52xx/motionpro_defconfig:CONFIG_ICPLUS_PHY=y arch/powerpc/configs/83xx/mpc832x_rdb_defconfig:CONFIG_ICPLUS_PHY=y arch/powerpc/configs/mpc83xx_defconfig:CONFIG_ICPLUS_PHY=y arch/powerpc/configs/ppc6xx_defconfig:CONFIG_ICPLUS_PHY=m
Sure, but the question/point is ... won't this be dead code in u-boot? It won't once you add at least one board that actually uses it ... adding it only so it's there will only cause it to bitrot and it'll actually only add more burden to maintainers.
It won't be dead code, because we will use it :-) Our board is derivative of BeagleBone & Co, so the whole support stuff is already in u-boot.
You need to enable it for that board though ;-)
Will do this as soon as we come near to the release.
Release of what? The hardware?
Release of hardware and when two more patches hit main line u-boot:
http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/147860 http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/147859
I'd like to have basic stuff mainlined before.
Well, I'll leave that up to Joe anyway
O.K.
Yegor