Re: [PATCH 1/3] spi: tegra210-quad: use device_reset_optional() instead of device_reset()

From: Mark Brown
Date: Tue Mar 18 2025 - 15:13:39 EST


On Tue, Mar 18, 2025 at 08:00:05PM +0100, Arnd Bergmann wrote:

> That does sound like the easiest answer: if the spi controller driver
> knows that it needs a reset but there is no reset controller, shutting
> itself down and removing its child devices seems like the least
> offensive action.

In that case it's probably more just refuse to probe in the first case
without the reset controller. Given that the device isn't working at
all it seems like the hardware description is broken anyway...

> No idea if there are other spi controllers that do something like this.

I'm really not thrilled about adding runtime error handling at that
level in the controller - it'll start to get into policy stuff if anyone
does something clever and realistically it's all broken hardware
description or very severe physical failure type stuff.

Attachment: signature.asc
Description: PGP signature