Re: [PATCH] Remove attempt by intel-spi-pci to turn the SPI flash chip writeable

From: Arnd Bergmann
Date: Mon Aug 03 2020 - 09:55:16 EST


On Mon, Aug 3, 2020 at 3:45 PM Daniel Gutson
<daniel.gutson@xxxxxxxxxxxxx> wrote:

> However, this flag applies only for a number of devices, coming from the
> platform driver, whereas the devices detected through the PCI driver
> (intel-spi-pci) are not subject to this check since the configuration
> takes place in intel-spi-pci which doesn't have an argument.

This part of the description sounds wrong: the current behavior is that
the BIOS setting is ignored for PCI devices and it only uses the module
parameter, the same way as it does for the platform driver.

With your patch, both the BIOS setting and the module parameter
have to explicitly allow writing on PCI devices, while at least for Bay
Trail platform devices the BIOS write protection is still ignored.

It sounds like this is what you want, but you should update the description
accordingly.

Arnd