Barry,
On 05/26/2015 03:28 AM, Barry Song wrote:
> From: Wei Chen <Wei.Chen@xxxxxxx>
>
> This patch moves to use generic dt-binding for hwspinlock providers and
> clients.
> add #hwlock-cells for the provider and hwlocks for clients.
>
> Cc: Suman Anna <s-anna@xxxxxx>
> Cc: Bjorn Andersson <bjorn@xxxxxxx>
> Signed-off-by: Wei Chen <Wei.Chen@xxxxxxx>
> Signed-off-by: Barry Song <Baohua.Song@xxxxxxx>
> ---
> arch/arm/boot/dts/atlas7.dtsi | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/atlas7.dtsi b/arch/arm/boot/dts/atlas7.dtsi
> index a753178..66d3f0e 100644
> --- a/arch/arm/boot/dts/atlas7.dtsi
> +++ b/arch/arm/boot/dts/atlas7.dtsi
> @@ -84,17 +84,17 @@
> #address-cells = <1>;
> #size-cells = <1>;
>
> - hwspinlock {
> + hwlock: hwspinlock {
> compatible = "sirf,hwspinlock";
> reg = <0x13240000 0x00010000>;
An unrelated question here, why the reg is same for all the child nodes
of the parent ipc node? If this is partitioned properly, then the
driver can be simplified a bit by using platform_get_resource and
devm_ioremap_resource?
> -
> - num-spinlocks = <30>;
> + #hwlock-cells = <1>;
> };
>
> ns_m3_rproc@0 {
> compatible = "sirf,ns2m30-rproc";
> reg = <0x13240000 0x00010000>;
> interrupts = <0 123 0>;
> + hwlocks = <&hwlock 0>, <&hwlock 1>;
Does this need to be added for the other nodes like ns_m3_rproc@1 as well?
regards
Suman
> };
>
> ns_m3_rproc@1 {
>
--
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
