Re: [PATCH] spi: tegra114: initialize native chip selects inactive
From: Mark Brown
Date: Wed Jul 29 2026 - 06:27:54 EST
On Tue, Jul 28, 2026 at 10:09:43PM +0100, Jon Hunter wrote:
> On 28/07/2026 21:03, Abraham Zukor wrote:
> > I think that defaulting to active-low is correct. The Tegra234 SPI
> > controller hardware defaults to active-low (all bits in CS_POL_INACTIVE
> > are set) so an active-high device is already asserted before this driver
> > initializes. I checked the controller drivers under drivers/spi/; they
> > either default to active-low or preserve the hardware default.
> I am a bit reluctant to make this change after 10+ years of this driver
> being around.
Yeah, I'd not be surprised to run into existing systems that somehow
manage to rely on the current behaviour.
Attachment:
signature.asc
Description: PGP signature