On Tue, Sep 10, 2013 at 01:26:49AM +0000, Duan Fugang-B38611 wrote: > Shawn, you cannot drop this patch. I have to drop the patch *for now*. Look, with the patch landed on linux-next, people report that fec is broken as below on imx6sl-evk. fec 2188000.ethernet eth0: no PHY, assuming direct connection to switch libphy: PHY fixed-0:00 not found fec 2188000.ethernet eth0: could not attach to PHY IP-Config: Failed to open eth0 IP-Config: No network devices available While before the patch, we can boot the board into console with NFS when downloading kernel image from network. This is the most used setup for kernel developers. > 1. Why do you get a working fec driver in kernel if U-Boot sets it up properly ? > Because the phy reset done by uboot, kernel cannot depend on uboot. > And even so, the driver still have problem for unit test. Yeah, it should be fixed/improved without any breakage/regression on the current use case. > 2. You see freescale internal kernel(3.10.9) work fine. There have one driver patch don't upstream. > I will send it out to net tree for review. That's where problem comes. You should push required driver changes upstream before sending me this patch. You need to resolve the dependencies between patches properly without introducing any breakages or regressions. That said, I'm not saying the patch is wrong, but the timing of sending this patch for merging is wrong. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
