On 07/23/2014 06:07 PM, Sören Brinkmann wrote: > On Wed, 2014-07-23 at 03:05PM +0200, Michal Simek wrote: >> Add node describing Zynq's CAN controller. >> >> Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx> >> --- >> >> arch/arm/boot/dts/zynq-7000.dtsi | 13 ++++++++++++- >> 1 file changed, 12 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi >> index 366ca6434f54..2287d9b4ed1a 100644 >> --- a/arch/arm/boot/dts/zynq-7000.dtsi >> +++ b/arch/arm/boot/dts/zynq-7000.dtsi >> @@ -71,7 +71,18 @@ >> interrupts = <0 7 4>; >> interrupt-parent = <&intc>; >> clocks = <&clkc 12>; >> - }; >> + }; >> + >> + can0: can@e0008000 { >> + compatible = "xlnx,zynq-can-1.0"; >> + clocks = <&clkc 19>, <&clkc 36>; >> + clock-names = "can_clk", "pclk"; >> + reg = <0xe0008000 0x1000>; >> + interrupts = <0 28 4>; >> + interrupt-parent = <&intc>; >> + tx-fifo-depth = <0x40>; >> + rx-fifo-depth = <0x40>; >> + }; > > What about the second CAN core? You also probably want to add > 'status = "disabled"' in the dtsi. Second can core - yes. I can add status = "disabled" but then question is if make sense to change status for any zynq dts file. Or just keep enable both. In our repo is enabled can0 on zc702. What do you think? Thanks, Michal -- 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