[PATCH v3 4/4] ARM: dts: Enable Broadcom iProc mailbox controller

[Jonathan Richardson]

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 at broadcom.com>
>> Reviewed-by: Shreesha Rajashekar <shreesha.rajashekar at broadcom.com>
>> Reviewed-by: Ray Jui <ray.jui at broadcom.com>
>> Tested-by: Jonathan Richardson <jonathan.richardson at broadcom.com>
>> Reviewed-by: Vikram Prakash <vikram.prakash at broadcom.com>
>> Reviewed-by: Scott Branden <scott.branden at broadcom.com>
>> Signed-off-by: Jonathan Richardson <jonathan.richardson at broadcom.com>
>> ---
>>  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 at 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.