On Fri, Dec 11, 2020 at 03:10:56AM +0300, Dmitry Baryshkov wrote: > Convert soc/qcom/qcom,smem.txt bindings to YAML format. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > .../bindings/soc/qcom/qcom,smem.txt | 57 --------------- > .../bindings/soc/qcom/qcom,smem.yaml | 73 +++++++++++++++++++ > 2 files changed, 73 insertions(+), 57 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,smem.txt > create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,smem.yaml > diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,smem.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,smem.yaml > new file mode 100644 > index 000000000000..832bf54239c8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,smem.yaml > @@ -0,0 +1,73 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/soc/qcom/qcom,smem.yaml#"; > +$schema: "http://devicetree.org/meta-schemas/core.yaml#"; > + > +title: Qualcomm Shared Memory Manager binding > + > +maintainers: > + - Andy Gross <agross@xxxxxxxxxx> > + - Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > + > +description: | > + This binding describes the Qualcomm Shared Memory Manager, used to share data > + between various subsystems and OSes in Qualcomm platforms. > + > +properties: > + compatible: > + const: "qcom-smem" Don't need quotes and the the string is wrong. Makes validation easy to pass because it never matches. > + > + memory-region: > + maxItems: 1 > + description: handle to memory reservation for main SMEM memory region. > + > + hwlocks: > + $ref: /schemas/types.yaml#/definitions/phandle hwlocks has a type already. You just need to define how many. > + description: reference to a hwspinlock used to protect allocations from the shared memory Isn't that every hwlocks? > + > + qcom,rpm-msg-ram: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: handle to RPM message memory resource > + > +required: > + - compatible > + - memory-region > + - hwlocks > + > +additionalProperties: false > + > +examples: > + - | > + reserved-memory { > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + smem_region: smem@fa00000 { > + reg = <0xfa00000 0x200000>; > + no-map; > + }; > + }; > + > + smem { > + compatible = "qcom,smem"; > + > + memory-region = <&smem_region>; > + qcom,rpm-msg-ram = <&rpm_msg_ram>; > + > + hwlocks = <&tcsr_mutex 3>; > + }; > + > + soc { > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + rpm_msg_ram: sram@fc428000 { > + compatible = "qcom,rpm-msg-ram"; > + reg = <0xfc428000 0x4000>; > + }; > + }; > + > +... > -- > 2.29.2 >
