Re: [RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property

2018-07-05 Thread Luis Araneda
Hi Rob, On Tue, Jul 3, 2018 at 7:07 PM Rob Herring wrote: > Maybe so, but you can't just go around changing compatible strings. Once > set, it's an ABI. Ok, I'll drop the patch from v2. Thanks, Luis Araneda.

Re: [RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property

2018-07-03 Thread Rob Herring
On Tue, Jul 03, 2018 at 03:18:49AM -0400, Luis Araneda wrote: > Hi Michal, > > On Thu, Jun 28, 2018 at 2:42 AM Michal Simek wrote: > > zynq name here is correct which is saying that hardblock is going to be > > described. Without it it is suggesting that hardblock is not used and it > > is for ex

Re: [RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property

2018-07-03 Thread Luis Araneda
Hi Michal, On Thu, Jun 28, 2018 at 2:42 AM Michal Simek wrote: > zynq name here is correct which is saying that hardblock is going to be > described. Without it it is suggesting that hardblock is not used and it > is for example on zybo with microblaze. In my opinion, the zynq name is just addin

Re: [RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property

2018-06-27 Thread Michal Simek
On 28.6.2018 07:42, Luis Araneda wrote: > The value "zynq" isn't officially part of the model on any board. > Additionally, the value is redundant as it's included in a > subsequent value of the property. > > Signed-off-by: Luis Araneda > --- > .../devicetree/bindings/arm/xilinx.txt| 22

[RFC PATCH 5/5] arm: dts: zynq: remove "zynq" prefix from compatible property

2018-06-27 Thread Luis Araneda
The value "zynq" isn't officially part of the model on any board. Additionally, the value is redundant as it's included in a subsequent value of the property. Signed-off-by: Luis Araneda --- .../devicetree/bindings/arm/xilinx.txt| 22 +-- arch/arm/boot/dts/zynq-cc108.dts