Re: [PATCH 2/2] arm: dts: zynq: Add Digilent ZYBO board

From: SÃren Brinkmann
Date: Sun Nov 09 2014 - 16:48:01 EST


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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/