| Message ID | 1477505640-26658-1-git-send-email-stefan.wahren@i2se.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
Stefan Wahren <stefan.wahren@i2se.com> writes: > The address of the mailbox node in the bcm283x.dts has also a typo. > So fix it accordingly. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > Fixes: 05b682b7a3b2 ("ARM: bcm2835: dt: Add the mailbox to the device tree") I've marked these to be applied once Rob acks the docs change. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Am 26.10.2016 um 20:13 schrieb Stefan Wahren: > The address of the mailbox node in the example has a typo. > So fix it accordingly. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > Fixes: d4b5c782b9f4 ("dt/bindings: Add binding for the BCM2835 mailbox driver") > --- > .../bindings/mailbox/brcm,bcm2835-mbox.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Andreas Färber <afaerber@suse.de> Regards, Andreas
Hi, Am 26.10.2016 um 20:14 schrieb Stefan Wahren: > The address of the mailbox node in the bcm283x.dts has also a typo. .dts -> .dtsi "also has" or maybe "has the same typo as in the binding example"? > So fix it accordingly. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > Fixes: 05b682b7a3b2 ("ARM: bcm2835: dt: Add the mailbox to the device tree") > --- > arch/arm/boot/dts/bcm283x.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/bcm283x.dtsi b/arch/arm/boot/dts/bcm283x.dtsi > index 46d46d8..74dd21b 100644 > --- a/arch/arm/boot/dts/bcm283x.dtsi > +++ b/arch/arm/boot/dts/bcm283x.dtsi > @@ -104,7 +104,7 @@ > reg = <0x7e104000 0x10>; > }; > > - mailbox: mailbox@7e00b800 { > + mailbox: mailbox@7e00b880 { > compatible = "brcm,bcm2835-mbox"; > reg = <0x7e00b880 0x40>; > interrupts = <0 1>; Otherwise, Reviewed-by: Andreas Färber <afaerber@suse.de> At one point in time dtc did complain about such mismatches... Regards, Andreas
On Wed, Oct 26, 2016 at 06:13:59PM +0000, Stefan Wahren wrote: > The address of the mailbox node in the example has a typo. > So fix it accordingly. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > Fixes: d4b5c782b9f4 ("dt/bindings: Add binding for the BCM2835 mailbox driver") > --- > .../bindings/mailbox/brcm,bcm2835-mbox.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Stefan Wahren <stefan.wahren@i2se.com> writes: > The address of the mailbox node in the bcm283x.dtsi also has a typo. > So fix it accordingly. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > Reviewed-by: Andreas Färber <afaerber@suse.de> > Fixes: 05b682b7a3b2 ("ARM: bcm2835: dt: Add the mailbox to the device tree") Pulled to bcm2835-dt-next. Thanks!
diff --git a/Documentation/devicetree/bindings/mailbox/brcm,bcm2835-mbox.txt b/Documentation/devicetree/bindings/mailbox/brcm,bcm2835-mbox.txt index e893615..b48d7d3 100644 --- a/Documentation/devicetree/bindings/mailbox/brcm,bcm2835-mbox.txt +++ b/Documentation/devicetree/bindings/mailbox/brcm,bcm2835-mbox.txt @@ -12,7 +12,7 @@ Required properties: Example: -mailbox: mailbox@7e00b800 { +mailbox: mailbox@7e00b880 { compatible = "brcm,bcm2835-mbox"; reg = <0x7e00b880 0x40>; interrupts = <0 1>;
The address of the mailbox node in the example has a typo. So fix it accordingly. Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> Fixes: d4b5c782b9f4 ("dt/bindings: Add binding for the BCM2835 mailbox driver") --- .../bindings/mailbox/brcm,bcm2835-mbox.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)