* [PATCH v2 3/4] dt-bindings: firmware: Add Qualcomm QSEECOM interface [not found] <20230127184650.756795-1-luzmaximilian@gmail.com> @ 2023-01-27 18:46 ` Maximilian Luz 2023-01-30 21:05 ` Rob Herring 0 siblings, 1 reply; 3+ messages in thread From: Maximilian Luz @ 2023-01-27 18:46 UTC (permalink / raw) To: Bjorn Andersson Cc: Maximilian Luz, Andy Gross, Konrad Dybcio, Ard Biesheuvel, Rob Herring, Krzysztof Kozlowski, Johan Hovold, Sudeep Holla, Ilias Apalodimas, Srinivas Kandagatla, Sumit Garg, Steev Klimaszewski, linux-arm-msm, linux-kernel, devicetree Add bindings for the Qualcomm Secure Execution Environment interface (QSEECOM). Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com> --- Changes in v2: - Replaces uefisecapp bindings. - Fix various dt-checker complaints. --- .../bindings/firmware/qcom,qseecom.yaml | 49 +++++++++++++++++++ MAINTAINERS | 1 + 2 files changed, 50 insertions(+) create mode 100644 Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml diff --git a/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml new file mode 100644 index 000000000000..540a604f81bc --- /dev/null +++ b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml @@ -0,0 +1,49 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/firmware/qcom,qseecom.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Qualcomm Secure Execution Environment Communication Interface + +maintainers: + - Maximilian Luz <luzmaximilian@gmail.com> + +description: | + QSEECOM provides an interface to Qualcomm's Secure Execution Environment + (SEE) running in the Trust Zone via SCM calls. In particular, it allows + communication with secure applications running therein. + + Applications running in this environment can, for example, include + 'uefisecapp', which is required for accessing UEFI variables on certain + systems as these cannot be accessed directly. + +properties: + compatible: + items: + - enum: + - qcom,qseecom-sc8280xp + - const: qcom,qseecom + + qcom,scm: + $ref: '/schemas/types.yaml#/definitions/phandle' + description: + A phandle pointing to the QCOM SCM device (see ./qcom,scm.yaml). + +required: + - compatible + - qcom,scm + +additionalProperties: false + +examples: + - | + firmware { + scm { + compatible = "qcom,scm-sc8280xp", "qcom,scm"; + }; + qseecom { + compatible = "qcom,qseecom-sc8280xp", "qcom,qseecom"; + qcom,scm = <&scm>; + }; + }; diff --git a/MAINTAINERS b/MAINTAINERS index 846d6c927840..5493bcc5792c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -17391,6 +17391,7 @@ QUALCOMM SECURE EXECUTION ENVIRONMENT COMMUNICATION DRIVER M: Maximilian Luz <luzmaximilian@gmail.com> L: linux-arm-msm@vger.kernel.org S: Maintained +F: Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml F: drivers/firmware/qcom_qseecom.c F: include/linux/qcom_qseecom.h -- 2.39.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2 3/4] dt-bindings: firmware: Add Qualcomm QSEECOM interface 2023-01-27 18:46 ` [PATCH v2 3/4] dt-bindings: firmware: Add Qualcomm QSEECOM interface Maximilian Luz @ 2023-01-30 21:05 ` Rob Herring 2023-01-30 21:46 ` Maximilian Luz 0 siblings, 1 reply; 3+ messages in thread From: Rob Herring @ 2023-01-30 21:05 UTC (permalink / raw) To: Maximilian Luz Cc: Bjorn Andersson, Andy Gross, Konrad Dybcio, Ard Biesheuvel, Krzysztof Kozlowski, Johan Hovold, Sudeep Holla, Ilias Apalodimas, Srinivas Kandagatla, Sumit Garg, Steev Klimaszewski, linux-arm-msm, linux-kernel, devicetree On Fri, Jan 27, 2023 at 07:46:49PM +0100, Maximilian Luz wrote: > Add bindings for the Qualcomm Secure Execution Environment interface > (QSEECOM). > > Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com> > --- > > Changes in v2: > - Replaces uefisecapp bindings. > - Fix various dt-checker complaints. > > --- > .../bindings/firmware/qcom,qseecom.yaml | 49 +++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 50 insertions(+) > create mode 100644 Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml > > diff --git a/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml > new file mode 100644 > index 000000000000..540a604f81bc > --- /dev/null > +++ b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml > @@ -0,0 +1,49 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/firmware/qcom,qseecom.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Secure Execution Environment Communication Interface > + > +maintainers: > + - Maximilian Luz <luzmaximilian@gmail.com> > + > +description: | > + QSEECOM provides an interface to Qualcomm's Secure Execution Environment > + (SEE) running in the Trust Zone via SCM calls. In particular, it allows SCM is SMCCC or something else? > + communication with secure applications running therein. > + > + Applications running in this environment can, for example, include > + 'uefisecapp', which is required for accessing UEFI variables on certain > + systems as these cannot be accessed directly. > + > +properties: > + compatible: > + items: > + - enum: > + - qcom,qseecom-sc8280xp > + - const: qcom,qseecom > + > + qcom,scm: > + $ref: '/schemas/types.yaml#/definitions/phandle' > + description: > + A phandle pointing to the QCOM SCM device (see ./qcom,scm.yaml). > + > +required: > + - compatible > + - qcom,scm > + > +additionalProperties: false > + > +examples: > + - | > + firmware { > + scm { > + compatible = "qcom,scm-sc8280xp", "qcom,scm"; > + }; > + qseecom { > + compatible = "qcom,qseecom-sc8280xp", "qcom,qseecom"; > + qcom,scm = <&scm>; Why do you need this in DT? If you already know you have a firmware interface (via "qcom,scm"), then query the firmware to see if the SEE is there. Rob ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2 3/4] dt-bindings: firmware: Add Qualcomm QSEECOM interface 2023-01-30 21:05 ` Rob Herring @ 2023-01-30 21:46 ` Maximilian Luz 0 siblings, 0 replies; 3+ messages in thread From: Maximilian Luz @ 2023-01-30 21:46 UTC (permalink / raw) To: Rob Herring Cc: Bjorn Andersson, Andy Gross, Konrad Dybcio, Ard Biesheuvel, Krzysztof Kozlowski, Johan Hovold, Sudeep Holla, Ilias Apalodimas, Srinivas Kandagatla, Sumit Garg, Steev Klimaszewski, linux-arm-msm, linux-kernel, devicetree On 1/30/23 22:05, Rob Herring wrote: > On Fri, Jan 27, 2023 at 07:46:49PM +0100, Maximilian Luz wrote: >> Add bindings for the Qualcomm Secure Execution Environment interface >> (QSEECOM). >> >> Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com> >> --- >> >> Changes in v2: >> - Replaces uefisecapp bindings. >> - Fix various dt-checker complaints. >> >> --- >> .../bindings/firmware/qcom,qseecom.yaml | 49 +++++++++++++++++++ >> MAINTAINERS | 1 + >> 2 files changed, 50 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml >> >> diff --git a/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml >> new file mode 100644 >> index 000000000000..540a604f81bc >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/firmware/qcom,qseecom.yaml >> @@ -0,0 +1,49 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/firmware/qcom,qseecom.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Qualcomm Secure Execution Environment Communication Interface >> + >> +maintainers: >> + - Maximilian Luz <luzmaximilian@gmail.com> >> + >> +description: | >> + QSEECOM provides an interface to Qualcomm's Secure Execution Environment >> + (SEE) running in the Trust Zone via SCM calls. In particular, it allows > > SCM is SMCCC or something else? It's whatever qcom-scm.c uses. I'm not too familiar with the specifics, so maybe someone else can answer this better. >> + communication with secure applications running therein. >> + >> + Applications running in this environment can, for example, include >> + 'uefisecapp', which is required for accessing UEFI variables on certain >> + systems as these cannot be accessed directly. >> + >> +properties: >> + compatible: >> + items: >> + - enum: >> + - qcom,qseecom-sc8280xp >> + - const: qcom,qseecom >> + >> + qcom,scm: >> + $ref: '/schemas/types.yaml#/definitions/phandle' >> + description: >> + A phandle pointing to the QCOM SCM device (see ./qcom,scm.yaml). >> + >> +required: >> + - compatible >> + - qcom,scm >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + firmware { >> + scm { >> + compatible = "qcom,scm-sc8280xp", "qcom,scm"; >> + }; >> + qseecom { >> + compatible = "qcom,qseecom-sc8280xp", "qcom,qseecom"; >> + qcom,scm = <&scm>; > > Why do you need this in DT? If you already know you have a firmware > interface (via "qcom,scm"), then query the firmware to see if the SEE is > there. Unfortunately I don't know of any way to query this, but please let me know if you do. As I've briefly mentioned in the cover letter: There are two interfaces to manage secure apps. QSEECOM (on older and current-gen laptop devices) and scminvoke (on newer and some current-gen mobile devices if I understood right). ACPI also uses a separate device for this (QCOM0476), so it seemed like the best option to follow that. Ideally, scminvoke would be preferred since that can be integrated as TEE driver, but I've been told that on platforms where apps (like uefisecapp) are loaded via QSEECOM by firmware, we can only use QSEECOM to communicate with those. Regards, Max ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-01-30 21:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20230127184650.756795-1-luzmaximilian@gmail.com>
2023-01-27 18:46 ` [PATCH v2 3/4] dt-bindings: firmware: Add Qualcomm QSEECOM interface Maximilian Luz
2023-01-30 21:05 ` Rob Herring
2023-01-30 21:46 ` Maximilian Luz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).