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

[Jonathan Richardson <jonathan.richardson-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>]

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-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Reviewed-by: Shreesha Rajashekar <shreesha.rajashekar-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Reviewed-by: Ray Jui <ray.jui-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Tested-by: Jonathan Richardson <jonathan.richardson-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Reviewed-by: Vikram Prakash <vikram.prakash-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Reviewed-by: Scott Branden <scott.branden-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> Signed-off-by: Jonathan Richardson <jonathan.richardson-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
>> ---
>>  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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html