On Fri, Jun 2, 2017 at 11:07 AM, Florian Fainelli [off-list ref] wrote:

On 05/26/2017 02:24 AM, Anup Patel wrote:

quoted

The Broadcom Stingray SoC is a new member in Broadcom iProc
SoC family.

This patch adds initial DTS files for Broadcom Stingray SoC
and two of its reference boards (bcm958742k and bcm958742t).

We have lot of reference boards and large number of devices
in Broadcom Stingray SoC so eventually we will have quite
a few DTS files for Stingray. To tackle, we have added a
separate directory for Stingray DTS files.

So I was going to apply this, but...

quoted

+     scr {
+             compatible = "simple-bus";
+             #address-cells = <1>;
+             #size-cells = <1>;
+             ranges = <0x0 0x0 0x61000000 0x05000000>;

So you corrected the ranges property here...

quoted

+
+             gic: interrupt-controller@63c00000 {

but you kept the original unit address here, which does not match the
reg property, could you fix that through your entire submission?

Thanks for catching. I quickly update this and send revised patchset.

Regards,
Anup
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html