Re: [PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-18 Thread Peter Korsgaard
Michal == Michal Bachraty michal.bachr...@gmail.com writes: Hi, Michal I'm ok with your changes. Have you tried to set MAC from Michal u-boot? I haven't seen any code for that purpose in cpsw. I Michal tried put MAC with kernel command parameter (in u-boot), but Michal nothing happen. I

Re: [PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-17 Thread Michal Bachraty
Hi Peter, I made changes to cpsw driver two weeks ago, which add support for reading MAC address from CPU and also I posted it to review. You can find patch here : https://patchwork.kernel.org/patch/1966481/ I will create updated patch next week, depended on reactions. Can you try that

Re: [PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-17 Thread Peter Korsgaard
Michal == Michal Bachraty michal.bachr...@gmail.com writes: Hi Michael, Michal I made changes to cpsw driver two weeks ago, which add support Michal for reading MAC address from CPU and also I posted it to Michal review. You can find patch here : Michal

Re: [PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-17 Thread Tony Lindgren
* Peter Korsgaard jac...@sunsite.dk [130117 08:46]: Michal == Michal Bachraty michal.bachr...@gmail.com writes: Hi Michael, Michal I made changes to cpsw driver two weeks ago, which add support Michal for reading MAC address from CPU and also I posted it to Michal review. You can

Re: [PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-17 Thread Peter Korsgaard
Tony == Tony Lindgren t...@atomide.com writes: Hi, The way I've handled it is similar to how it is done on atleast one other arm subarchicture, see arch/arm/mach-mxs/mach-mxs.c::update_fec_mac_prop() Tony Cool this seems like the cleanest way to deal with it so far. Can Tony you please

[PATCH] am33xx: cpsw: default to ethernet hwaddr from efuse if not defined in dt

2013-01-16 Thread Peter Korsgaard
When booting with CONFIG_ARM_APPENDED_DTB (either because of using an old U-Boot, not wanting the hassle of 2 files or when using Falcon fast boot mode in U-Boot), nothing updates the ethernet hwaddr specified for the CPSW slaves, causing the driver to use a random hwaddr, which is some times