On Thursday 11 September 2014 07:08 AM, Lennart Sorensen wrote: > On Wed, Sep 10, 2014 at 07:07:26PM +0530, Mugunthan V N wrote: >> Add CPSW and MDIO related device tree data for DRA7XX and made as status >> disabled. Phy-id, pinmux for active and sleep state needs to be added in >> board dts files and enable the CPSW device. >> >> Signed-off-by: Mugunthan V N <mugunthanvnm@xxxxxx> >> --- >> arch/arm/boot/dts/dra7.dtsi | 59 +++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 59 insertions(+) >> >> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi >> index d678152..8d79321 100644 >> --- a/arch/arm/boot/dts/dra7.dtsi >> +++ b/arch/arm/boot/dts/dra7.dtsi >> @@ -34,6 +34,8 @@ >> serial3 = &uart4; >> serial4 = &uart5; >> serial5 = &uart6; >> + ethernet0 = &cpsw_emac0; >> + ethernet1 = &cpsw_emac1; >> }; >> >> timer { >> @@ -1262,6 +1264,63 @@ >> ti,irqs-skip = <10 133 139 140>; >> ti,irqs-safe-map = <0>; >> }; >> + >> + mac: ethernet@4a100000 { >> + compatible = "ti,cpsw"; >> + ti,hwmods = "gmac"; >> + cpdma_channels = <8>; >> + ale_entries = <1024>; >> + bd_ram_size = <0x2000>; >> + no_bd_ram = <0>; >> + rx_descs = <64>; >> + mac_control = <0x20>; >> + slaves = <2>; > > How am I supposed to override this in the board dtb when my board only > uses 1 slave? slaves = <1>; > >> + active_slave = <0>; >> + cpts_clock_mult = <0x80000000>; >> + cpts_clock_shift = <29>; >> + reg = <0x48484000 0x1000 >> + 0x48485200 0x2E00>; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + /* >> + * rx_thresh_pend >> + * rx_pend >> + * tx_pend >> + * misc_pend >> + */ >> + interrupts = <GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 336 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 337 IRQ_TYPE_LEVEL_HIGH>; >> + ranges; >> + status = "disabled"; >> + >> + davinci_mdio: mdio@48485000 { >> + compatible = "ti,davinci_mdio"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + ti,hwmods = "davinci_mdio"; >> + bus_freq = <1000000>; >> + reg = <0x48485000 0x100>; >> + }; >> + >> + cpsw_emac0: slave@48480200 { >> + /* Filled in by U-Boot */ >> + mac-address = [ 00 00 00 00 00 00 ]; >> + }; >> + >> + cpsw_emac1: slave@48480300 { >> + /* Filled in by U-Boot */ >> + mac-address = [ 00 00 00 00 00 00 ]; >> + }; > > Should I then be setting this to disabled from my board file? Or does > the disabled for the ethernet overall take care of that? overall disable takes care of this > >> + phy_sel: cpsw-phy-sel@4a002554 { >> + compatible = "ti,dra7xx-cpsw-phy-sel"; >> + reg= <0x4a002554 0x4>; >> + reg-names = "gmii-sel"; >> + }; > > I guess the board file would have to disable that too (we won't have a > phy given we are connecting to a switch chip). Will add disabled in next version. Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html