On 17-01-25 04:27 PM, Florian Fainelli wrote: > On 01/25/2017 04:27 PM, Jonathan Richardson wrote: >> Add node for iProc mailbox controller and enable. >> >> Reviewed-by: Jonathan Richardson <jonathan.richardson@xxxxxxxxxxxx> >> Reviewed-by: Shreesha Rajashekar <shreesha.rajashekar@xxxxxxxxxxxx> >> Reviewed-by: Ray Jui <ray.jui@xxxxxxxxxxxx> >> Tested-by: Jonathan Richardson <jonathan.richardson@xxxxxxxxxxxx> >> Reviewed-by: Vikram Prakash <vikram.prakash@xxxxxxxxxxxx> >> Reviewed-by: Scott Branden <scott.branden@xxxxxxxxxxxx> >> Signed-off-by: Jonathan Richardson <jonathan.richardson@xxxxxxxxxxxx> >> --- >> arch/arm/boot/dts/bcm-cygnus.dtsi | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi b/arch/arm/boot/dts/bcm-cygnus.dtsi >> index cf02271..606e11a 100644 >> --- a/arch/arm/boot/dts/bcm-cygnus.dtsi >> +++ b/arch/arm/boot/dts/bcm-cygnus.dtsi >> @@ -121,6 +121,12 @@ >> <0x0301d24c 0x2c>; >> }; >> >> + mailbox: mailbox@03024024 { >> + compatible = "brcm,iproc-mailbox"; >> + reg = <0x03024024 0x8>; >> + #mbox-cells = <1>; >> + }; > Can you squash this change with the previous one? It seems to me like > you are just removing the interrupt properties, and updating the reg > property to be only 0x8 bytes. > > Thank you And removing the interrupt controller properties from the mailbox controller. I don't mind squashing it if there are no objections. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html