Re: [PATCH net-next 3/5] dt-bindings: net: adi,adin1110: Add adi,spi-oa-tc6

[Alvin Šipraga]

On Tue, Nov 03, 2026 at 06:09:16PM +0200, Iustin Stolniceanu wrote:
> From: Ciprian Regus <ciprian.regus@xxxxxxxxxx>
> 
> The ADIN1110 and ADIN2111 support two SPI host interface protocols:
> the generic SPI protocol and the OPEN Alliance TC6 protocol. The
> protocol is selected by the state of the SPI_CFG1 pin, which the device
> latches when it comes out of reset, together with SPI_CFG0, which
> selects CRC (generic SPI) or protection (OPEN Alliance) mode.
> 
> The protocol is set by the board hardware, so the driver must know it
> in advance. Add a boolean property to describe a device configured for
> the OPEN Alliance TC6 protocol. Without it, the generic SPI protocol is
> assumed, as before.
> 
> Signed-off-by: Ciprian Regus <ciprian.regus@xxxxxxxxxx>
> Signed-off-by: Iustin Stolniceanu <iustin.stolniceanu@xxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/net/adi,adin1110.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/adi,adin1110.yaml b/Documentation/devicetree/bindings/net/adi,adin1110.yaml
> index 0a73e01d7f97..e19b6c94708d 100644
> --- a/Documentation/devicetree/bindings/net/adi,adin1110.yaml
> +++ b/Documentation/devicetree/bindings/net/adi,adin1110.yaml
> @@ -43,6 +43,12 @@ properties:
>        Enable CRC8 checks on SPI read/writes.
>      type: boolean
>  
> +  adi,spi-oa-tc6:
> +    description: |
> +      The device is configured, through the SPI_CFG1 pin, for the
> +      Open Alliance TC6 protocol instead of the generic SPI one.
> +    type: boolean
> +

Should this disallow the adi,spi-crc property if set?

I also see that there's a similar protection concept in OA TC6, but that
seems not to be supported here? Is that intrinsically the same as adi,spi-crc?

>    interrupts:
>      maxItems: 1
>  
> -- 
> 2.43.0
>