diff mbox series

[v3,2/9] dt-bindings: qcom: se-common: Add QUP Peripheral-specific properties for I2C, SPI, and SERIAL bus

Message ID 20250303124349.3474185-3-quic_vdadhani@quicinc.com
State New
Headers show
Series Add support to load QUP SE firmware from | expand

Commit Message

Viken Dadhaniya March 3, 2025, 12:43 p.m. UTC
Introduce a new YAML schema for QUP-supported peripherals. Define common
properties used across QUP-supported peripherals.

Add property `qcom,gsi-dma-allowed` to configure the Serial Engine (SE) for
QCOM GPI DMA mode.

Co-developed-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
Signed-off-by: Viken Dadhaniya <quic_vdadhani@quicinc.com>
---
 .../soc/qcom/qcom,se-common-props.yaml        | 26 +++++++++++++++++++
 1 file changed, 26 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml

Comments

Krzysztof Kozlowski March 4, 2025, 8:10 a.m. UTC | #1
On Mon, Mar 03, 2025 at 06:13:42PM +0530, Viken Dadhaniya wrote:
> Introduce a new YAML schema for QUP-supported peripherals. Define common
> properties used across QUP-supported peripherals.
> 
> Add property `qcom,gsi-dma-allowed` to configure the Serial Engine (SE) for
> QCOM GPI DMA mode.
> 
> Co-developed-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
> Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
> Signed-off-by: Viken Dadhaniya <quic_vdadhani@quicinc.com>
> ---
>  .../soc/qcom/qcom,se-common-props.yaml        | 26 +++++++++++++++++++
>  1 file changed, 26 insertions(+)

You miss here any user of this.

>  create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml
> 
> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml
> new file mode 100644
> index 000000000000..a111e51bb1c4
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml
> @@ -0,0 +1,26 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/soc/qcom/qcom,se-common-props.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: QUP Peripheral-specific properties for I2C, SPI and SERIAL bus.

Drop full stop. Titles never have them.

> +
> +description:
> +  The Generic Interface (GENI) based Qualcomm Universal Peripheral (QUP) is
> +  a programmable module that supports a wide range of serial interfaces
> +  such as UART, SPI, I2C, I3C, etc. This defines the common properties used
> +  across QUP-supported peripherals.
> +
> +maintainers:
> +  - Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
> +  - Viken Dadhaniya <quic_vdadhani@quicinc.com>
> +
> +properties:
> +  qcom,gsi-dma-allowed:

Based on description: qcom,enable-gsi-dma
(or description is not accurate)

> +    $ref: /schemas/types.yaml#/definitions/flag
> +    description:
> +      Configure the Serial Engine (SE) to transfer data in QCOM GPI DMA mode.
> +      By default, FIFO mode (PIO/CPU DMA) will be selected.

Best regards,
Krzysztof
Krzysztof Kozlowski March 4, 2025, 8:22 a.m. UTC | #2
On 04/03/2025 09:10, Krzysztof Kozlowski wrote:
> On Mon, Mar 03, 2025 at 06:13:42PM +0530, Viken Dadhaniya wrote:
>> Introduce a new YAML schema for QUP-supported peripherals. Define common
>> properties used across QUP-supported peripherals.
>>
>> Add property `qcom,gsi-dma-allowed` to configure the Serial Engine (SE) for
>> QCOM GPI DMA mode.
>>
>> Co-developed-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
>> Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
>> Signed-off-by: Viken Dadhaniya <quic_vdadhani@quicinc.com>
>> ---
>>  .../soc/qcom/qcom,se-common-props.yaml        | 26 +++++++++++++++++++
>>  1 file changed, 26 insertions(+)
> 
> You miss here any user of this.

I see now you have users in other patches, so it is fine.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml
new file mode 100644
index 000000000000..a111e51bb1c4
--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/qcom/qcom,se-common-props.yaml
@@ -0,0 +1,26 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/soc/qcom/qcom,se-common-props.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: QUP Peripheral-specific properties for I2C, SPI and SERIAL bus.
+
+description:
+  The Generic Interface (GENI) based Qualcomm Universal Peripheral (QUP) is
+  a programmable module that supports a wide range of serial interfaces
+  such as UART, SPI, I2C, I3C, etc. This defines the common properties used
+  across QUP-supported peripherals.
+
+maintainers:
+  - Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
+  - Viken Dadhaniya <quic_vdadhani@quicinc.com>
+
+properties:
+  qcom,gsi-dma-allowed:
+    $ref: /schemas/types.yaml#/definitions/flag
+    description:
+      Configure the Serial Engine (SE) to transfer data in QCOM GPI DMA mode.
+      By default, FIFO mode (PIO/CPU DMA) will be selected.
+
+additionalProperties: true