On Wednesday 18 December 2013 10:38 PM, Felipe Balbi wrote: > On Wed, Dec 18, 2013 at 10:30:55PM +0530, Mugunthan V N wrote: >> On Wednesday 18 December 2013 10:17 PM, Markus Pargmann wrote: >>> Hi, >>> >>> This series introduces a driver to read and use the MACIDs stored in the am335x >>> control module. These are read-only registers for a unique MACID. At the moment >>> the MACIDs are generated randomly or they are set by the bootloader. >>> >>> A device node is added in am33xx dtsi and used by the cpsw slaves in the bone >>> board files. >>> >>> Regards, >>> >>> Markus >>> >>> >>> Markus Pargmann (6): >>> DT doc: net: cpsw mac-address is optional >>> net: cpsw: header, Add missing include >>> net: cpsw: Add control-module macid driver >>> net: cpsw: Use cpsw-ctrl-macid driver >>> arm: dts: am33xx, Add device node for cpsw-ctrl-macid >>> arm: dts: am335x beagle bone use processor macids >>> >>> .../devicetree/bindings/net/cpsw-ctrl-macid.txt | 31 +++++ >>> Documentation/devicetree/bindings/net/cpsw.txt | 7 +- >>> arch/arm/boot/dts/am335x-bone.dts | 8 ++ >>> arch/arm/boot/dts/am335x-boneblack.dts | 8 ++ >>> arch/arm/boot/dts/am33xx.dtsi | 7 ++ >>> drivers/net/ethernet/ti/Kconfig | 8 ++ >>> drivers/net/ethernet/ti/Makefile | 1 + >>> drivers/net/ethernet/ti/cpsw-ctrl-macid.c | 138 +++++++++++++++++++++ >>> drivers/net/ethernet/ti/cpsw.c | 18 ++- >>> drivers/net/ethernet/ti/cpsw.h | 3 + >>> 10 files changed, 224 insertions(+), 5 deletions(-) >>> create mode 100644 Documentation/devicetree/bindings/net/cpsw-ctrl-macid.txt >>> create mode 100644 drivers/net/ethernet/ti/cpsw-ctrl-macid.c >>> >> Mac ID is to be filled by U-Boot and this kind of approach is already >> rejected in linux-omap list. >> >> If proper ethaddr/eth*addr is populated in U-boot environment variable >> then mac-address dt property in ethernet* device nodes will be populated >> before boot kernel in U-boot. So I don't think this patch series is >> required. > but will u-boot read MACID from control module ? > Yes, U-Boot will read the MACID from control module and if a customer wants to have his own MACID, U-boot ENV variable ethaddr/eth1addr must be updated. Regards Mugunthan V N -- 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