Hi Fabio, On 06/01/2016 10:47, Fabio Estevam wrote: > On Wed, Jan 6, 2016 at 3:17 AM, Qiang Zhao <[email protected]> wrote: >> On Tue, Jan 5, 2016 at 1:02 PM, Fabio Estevam <[email protected]> wrote: >> >>> -----Original Message----- >>> From: Fabio Estevam [mailto:[email protected]] >>> Sent: Wednesday, January 06, 2016 3:03 AM >>> To: [email protected] >>> Cc: [email protected]; [email protected]; [email protected]; >>> [email protected]; [email protected]; Fabio Estevam >>> <[email protected]> >>> Subject: [PATCH v2 1/3] phy: atheros: Use ar8035_config for AR8031 >>> >>> From: Fabio Estevam <[email protected]> >>> >>> Commit 08ad9b068afb88 (" ar8031: modify the config func of ar8031 to >>> ar8021_config") selected 'ar8021_config' as the configuration function for >>> AR8031. >>> >>> The correct would be to use 'ar8035_config' instead as AR8031/AR8035 have >>> the same programming model and even share the same phy driver in the linux >>> kernel: drivers/net/phy/at803x.c. >>> >>> Tested on a mx6qsabresd and wandboard, which now can work without any >>> PHY setup code in the board files. >>> >>> Signed-off-by: Fabio Estevam <[email protected]> >>> --- >>> This is 2016.04 material. >>> >>> Zhao Qiang, could you please test this on your platform? >> >> Yes, have tested, it can work on my platform. > > Excellent, thanks for testing, Zhao. > > Stefano, > > With Zhao's test I think it is safe to add this for 2016.01. What do you > think? >
Agree, I pick it up. Regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: [email protected] ===================================================================== _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
