Messages in this thread | | | Date | Thu, 6 Jul 2023 10:37:00 +0530 | Subject | Re: [PATCH v3 2/3] spi: dt-bindings: qcom,spi-geni-qcom: Add SPI device mode support for GENI based QuPv3 | From | Praveen Talari <> |
| |
Hi
On 6/23/2023 12:35 AM, Conor Dooley wrote: > On Thu, Jun 22, 2023 at 07:29:54PM +0530, Praveen Talari wrote: >> Add a property to configure QUPv3 SE as SPI Device mode. >> >> Signed-off-by: Praveen Talari <quic_ptalari@quicinc.com> >> --- >> v2 -> v3: >> - modified commit message to use device mode instead of slave mode > Suitability or w/e of the property aside, I don't understand this. > Why not change the *property*, which has far more visibility than the > commit message, to use device rather than slave? We are going use existing property spi-slave > > Chers, > Conor. > >> --- >> Documentation/devicetree/bindings/spi/qcom,spi-geni-qcom.yaml | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-geni-qcom.yaml b/Documentation/devicetree/bindings/spi/qcom,spi-geni-qcom.yaml >> index 2e20ca313ec1..5c7d0293bbf7 100644 >> --- a/Documentation/devicetree/bindings/spi/qcom,spi-geni-qcom.yaml >> +++ b/Documentation/devicetree/bindings/spi/qcom,spi-geni-qcom.yaml >> @@ -66,6 +66,10 @@ properties: >> reg: >> maxItems: 1 >> >> + qcom,slv-ctrl: >> + description: configure QUPv3 SE as Device mode >> + type: boolean >> + >> required: >> - compatible >> - clocks >> -- >> 2.17.1 >>
| |