Messages in this thread | | | Date | Sun, 9 Nov 2014 13:47:51 -0800 | From | Sören Brinkmann <> | Subject | Re: [PATCH 2/2] arm: dts: zynq: Add Digilent ZYBO board |
| |
Hi Peter,
On Sun, 2014-11-09 at 01:38PM +1000, Peter Crosthwaite wrote: > Add a DTS describing the Digilent ZYBO board. Similar to ZED but with
"Digilent ZYBO" here...
[...] > +/ { > + model = "Zynq ZYBO Development Board"; > + compatible = "xlnx,zynq-zybo", "xlnx,zynq-7000";
... "xlnx,zynq-zybo" here. Seems inconsistent. IMHO, there should rather be a digilent vendor prefix.
Everything else looks pretty straight forward and good.
Soren
| |