On 11/13/2014 04:07 PM, Marc Kleine-Budde wrote: > On 11/13/2014 01:22 PM, Roger Quadros wrote: >> The SoC contains 2 DCAN modules. Add them. >> >> Signed-off-by: Roger Quadros <rogerq@xxxxxx> >> --- >> arch/arm/boot/dts/am4372.dtsi | 22 ++++++++++++++++++++++ >> 1 file changed, 22 insertions(+) >> >> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi >> index 899c57c..12fb1db 100644 >> --- a/arch/arm/boot/dts/am4372.dtsi >> +++ b/arch/arm/boot/dts/am4372.dtsi >> @@ -901,6 +901,28 @@ >> compatible = "mmio-sram"; >> reg = <0x40300000 0x40000>; /* 256k */ >> }; >> + >> + dcan0: can@481cc000 { >> + compatible = "ti,am3352-d_can"; > > You should add "ti,am4372-d_can" as first compatible here. > We don't have a separate compatible id for am4372-d_can as it the IP exactly same as am3352-d_can. cheers, -roger >> + ti,hwmods = "d_can0"; >> + clocks = <&dcan0_fck>; >> + clock-names = "fck"; >> + reg = <0x481cc000 0x2000>; >> + syscon-raminit = <&am43xx_control_module 0x644 0>; >> + interrupts = <GIC_SPI 52 IRQ_TYPE_LEVEL_HIGH>; >> + status = "disabled"; >> + }; >> + >> + dcan1: can@481d0000 { >> + compatible = "ti,am3352-d_can"; > > Same here. > >> + ti,hwmods = "d_can1"; >> + clocks = <&dcan1_fck>; >> + clock-names = "fck"; >> + reg = <0x481d0000 0x2000>; >> + syscon-raminit = <&am43xx_control_module 0x644 1>; >> + interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>; >> + status = "disabled"; >> + }; >> }; >> }; >> >> > > Marc > -- 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