Kumar Gala wrote:

quoted

+        mpic: pic@40000 {
+            clock-frequency = <0>;

remove clock-frequency

Where would one express the PIC timer frequency, then?

quoted

+            interrupt-controller;
+            #address-cells = <0>;
+            #interrupt-cells = <2>;
+            reg = <0x40000 0x40000>;
+            compatible = "chrp,open-pic";
+            device_type = "open-pic";
+                        big-endian;

remove big-endian.

I thought this was part of the open-pic binding.

In any case, if these are to go, we should remove them from existing 
trees and from the examples in bwof.txt before complaining about them in 
new boards...

-Scott