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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250304-crafty-beaver-of-teaching-b58efb@krzk-bin>
Date: Tue, 4 Mar 2025 09:10:06 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Viken Dadhaniya <quic_vdadhani@...cinc.com>
Cc: andi.shyti@...nel.org, robh@...nel.org, krzk+dt@...nel.org, 
	conor+dt@...nel.org, gregkh@...uxfoundation.org, jirislaby@...nel.org, 
	broonie@...nel.or, andersson@...nel.org, konradybcio@...nel.org, 
	johan+linaro@...nel.org, dianders@...omium.org, agross@...nel.org, 
	linux-arm-msm@...r.kernel.org, linux-i2c@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org, linux-spi@...r.kernel.org, 
	quic_msavaliy@...cinc.com, quic_anupkulk@...cinc.com
Subject: Re: [PATCH v3 2/9] dt-bindings: qcom: se-common: Add QUP
 Peripheral-specific properties for I2C, SPI, and SERIAL bus

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@...cinc.com>
> Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
> Signed-off-by: Viken Dadhaniya <quic_vdadhani@...cinc.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@...cinc.com>
> +  - Viken Dadhaniya <quic_vdadhani@...cinc.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