Re: [PATCH v2] spi: bcm63xx: fix premature CS deassertion on RX-only transactions

Mark Brown
2025-11-17T17:29:34Z

[-- Attachment #1: Type: text/plain, Size: 437 bytes --]

On Tue, Nov 18, 2025 at 03:53:38AM +1100, Hang Zhou wrote:

> I initially picked 0xFF because in most case, the SPI bus is high when it's idle, but in practice 0x00 works just as well here.

> If you prefer, I'm happy to switch the dummy value to 0x00 for consistency and respin this as v3.

No, it's OK - if this hardware defaults to keeing the signal high then
0xff is a better choice. Most hardware defaults to keeping the line
low.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]