soc: qcom: Update properties for smem state referencing
authorBjorn Andersson <bjorn.andersson@linaro.org>
Fri, 10 Jun 2016 04:11:22 +0000 (21:11 -0700)
committerAndy Gross <andy.gross@linaro.org>
Sat, 25 Jun 2016 03:53:41 +0000 (22:53 -0500)
Update the property names to match device tree bindings, the correct
values should be qcom,smem-states and qcom,smem-state-names.

Also update the #qcom,smem-state-cells for consistency, before we merge
any users of these properties.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Andy Gross <andy.gross@linaro.org>
Documentation/devicetree/bindings/soc/qcom/qcom,smp2p.txt
Documentation/devicetree/bindings/soc/qcom/qcom,smsm.txt
drivers/soc/qcom/smem_state.c
drivers/soc/qcom/smsm.c

index 5cc82b8353d89659600743e4fed17e48a004134a..af9ca37221ceac6f33872d9ae4fa438cab435259 100644 (file)
@@ -68,7 +68,7 @@ important.
        Value type: <u32>
        Definition: must be 2 - denoting the bit in the entry and IRQ flags
 
-- #qcom,state-cells:
+- #qcom,smem-state-cells:
        Usage: required for outgoing entries
        Value type: <u32>
        Definition: must be 1 - denoting the bit in the entry
@@ -92,7 +92,7 @@ wcnss-smp2p {
        wcnss_smp2p_out: master-kernel {
                qcom,entry-name = "master-kernel";
 
-               #qcom,state-cells = <1>;
+               #qcom,smem-state-cells = <1>;
        };
 
        wcnss_smp2p_in: slave-kernel {
index a6634c70850d76fdd68318d95433f07612aeced4..2993b5a97dd648c1da9c560bc431aa5b99ea6615 100644 (file)
@@ -51,7 +51,7 @@ important.
        Definition: specifies the offset, in words, of the first bit for this
                    entry
 
-- #qcom,state-cells:
+- #qcom,smem-state-cells:
        Usage: required for local entry
        Value type: <u32>
        Definition: must be 1 - denotes bit number
@@ -91,7 +91,7 @@ smsm {
        apps_smsm: apps@0 {
                reg = <0>;
 
-               #qcom,state-cells = <1>;
+               #qcom,smem-state-cells = <1>;
        };
 
        wcnss_smsm: wcnss@7 {
index 54261decb369250c7cef09108d63517a972bb6ee..d5437ca76ed926bbb10d5f2140ef22a667fa0d73 100644 (file)
@@ -104,26 +104,26 @@ struct qcom_smem_state *qcom_smem_state_get(struct device *dev,
 
        if (con_id) {
                index = of_property_match_string(dev->of_node,
-                                                "qcom,state-names",
+                                                "qcom,smem-state-names",
                                                 con_id);
                if (index < 0) {
-                       dev_err(dev, "missing qcom,state-names\n");
+                       dev_err(dev, "missing qcom,smem-state-names\n");
                        return ERR_PTR(index);
                }
        }
 
        ret = of_parse_phandle_with_args(dev->of_node,
-                                        "qcom,state",
-                                        "#qcom,state-cells",
+                                        "qcom,smem-states",
+                                        "#qcom,smem-state-cells",
                                         index,
                                         &args);
        if (ret) {
-               dev_err(dev, "failed to parse qcom,state property\n");
+               dev_err(dev, "failed to parse qcom,smem-states property\n");
                return ERR_PTR(ret);
        }
 
        if (args.args_count != 1) {
-               dev_err(dev, "invalid #qcom,state-cells\n");
+               dev_err(dev, "invalid #qcom,smem-state-cells\n");
                return ERR_PTR(-EINVAL);
        }
 
index 6b777af1bc19d382997bb99f7d1fa2c784370cd2..d0337b2a71c81b57eaae7ec4e5bf921055cbb766 100644 (file)
@@ -495,7 +495,7 @@ static int qcom_smsm_probe(struct platform_device *pdev)
        if (!smsm->hosts)
                return -ENOMEM;
 
-       local_node = of_find_node_with_property(pdev->dev.of_node, "#qcom,state-cells");
+       local_node = of_find_node_with_property(pdev->dev.of_node, "#qcom,smem-state-cells");
        if (!local_node) {
                dev_err(&pdev->dev, "no state entry\n");
                return -EINVAL;