[lkml]   [2017]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 00/10] spi-nor: intel-spi: Various fixes and enhancements
On Fri, Sep 01, 2017 at 01:00:31AM -0700, Bin Meng wrote:
> This series does several bug fixes and clean ups against the intel-spi
> spi-nor driver, as well as enhancements to make the driver independent
> on the underlying BIOS/bootloader.
> At present the driver uses the HW sequencer for the read/write/erase on
> all supported platforms, read_reg/write_reg for BXT, and the SW sequencer
> for read_reg/write_reg for BYT/LPT. The way the driver uses the HW and SW
> sequencer relies on some programmed register settings and hence creates
> unneeded dependencies with the underlying BIOS/bootloader. For example,
> the driver unfortunately does not work as expected when booting from
> Intel Baytrail FSP based bootloaders like U-Boot, as the Baytrail FSP
> does not set up some SPI controller settings to make the driver happy.
> Now such limitation has been removed with this series.
> Bin Meng (10):
> spi-nor: intel-spi: Fix number of protected range registers for
> spi-nor: intel-spi: Remove useless 'buf' parameter in the HW/SW cycle
> spi-nor: intel-spi: Fix broken software sequencing codes
> spi-nor: intel-spi: Check transfer length in the HW/SW cycle
> spi-nor: intel-spi: Use SW sequencer for BYT/LPT
> spi-nor: intel-spi: Remove 'Atomic Cycle Sequence' in
> intel_spi_write()
> spi-nor: intel-spi: Don't assume OPMENU0/1 to be programmed by BIOS
> spi-nor: intel-spi: Remove the unnecessary HSFSTS register RW
> spi-nor: intel-spi: Rename swseq to swseq_reg in 'struct intel_spi'
> spi-nor: intel-spi: Fall back to use SW sequencer to erase
> drivers/mtd/spi-nor/intel-spi.c | 209 +++++++++++++++++++++++++++++-----------
> 1 file changed, 151 insertions(+), 58 deletions(-)

Nice work!

For the whole series,

Acked-by: Mika Westerberg <>

 \ /
  Last update: 2017-09-01 12:09    [W:0.183 / U:5.772 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site