Re: [PATCH v2 1/4] spi: dt-bindings: amlogic: spisg: Document A9-specific properties

Mark Brown <[email protected]> Thu, 30 Jul 2026 14:22:28 +0100
Newsgroups org.infradead.lists.linux-amlogic,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-spi
Message-ID <[email protected]>
--===============1697848232373399293==
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="9CS4vkVm2ZtSBEMP"
Content-Disposition: inline


--9CS4vkVm2ZtSBEMP
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Jul 23, 2026 at 06:43:42AM +0000, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <[email protected]>

> Document the Amlogic A9 SPISG controller and its additional
> Device Tree properties.

> Add the "amlogic,a9-spisg" compatible string and define the
> A9-specific "amlogic,mo-idle-output" and "amlogic,word-gap"
> properties, which configure the MOSI idle output level and
> the inter-word gap, respectively.

> @@ -35,6 +34,23 @@ properties:
>    resets:
>      maxItems: 1
> =20
> +  amlogic,mo-idle-output:
> +    description: |
> +      Controls the MOSI output level when the controller is idle.
> +      0 - Drive MOSI low
> +      1 - Drive MOSI high
> +      2 - Follow the last transmitted bit
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    enum: [0, 1, 2]
> +    default: 0
> +
> +  amlogic,word-gap:
> +    description: |
> +      Number of idle clock cycles inserted between consecutive SPI words.
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    enum: [0, 1, 2, 3]
> +    default: 1

I'd expect these to be per-device properties, not per controller.  The
word_gap sounds a lot like the existing word_delay feature we have - if
it's different it'd be good to be clear why.

--9CS4vkVm2ZtSBEMP
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmprUBMACgkQJNaLcl1U
h9B6Igf/X+zzg9VfJoUN5T5BGj5eihf8ukNh7rLmr4zYQsfOkHIr/NcL/kZsXsxf
7KuCeVVRO/KVxZesl6kSr8Kf7h8SEjJChTauUDDy2Yy3up9zzI0seOO2Kpibc3Nh
Xq+6ZmSDie9G3zBKMHNB71//bhMI+K5duCbfk8A0F6dsG1wsJobxAAlhTcHZnsiu
LkmiQ+7p/rbBodJ6B+zk7tnpSvNFYaOkXVYO0I+kgDCF7uCyHok1/UKT/iYaaBbI
R6ryWEykKC5fRmxPgculPRoamYLQhC/uMIMtSLdQK1DiKxVc5tEzp0pVLkKi4xss
nQMdyTzCWVf4gThGtW1MS6J6xSXnRw==
=2hV1
-----END PGP SIGNATURE-----

--9CS4vkVm2ZtSBEMP--


--===============1697848232373399293==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
linux-amlogic mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-amlogic

--===============1697848232373399293==--