On Tue, Jan 23, 2024 at 9:34 AM Tudor Ambarus <tudor.ambarus@xxxxxxxxxx> wrote:
>
> Hi,
>
> The patch set cleans a bit the driver and adds support for gs101 SPI.
>
It might be more convenient (for review purposes) to extract all the
cleanup patches into a separate series, and base it on top of the
gs101 SPI enablement series.
> Apart of the SPI patches, I added support for iowrite{8,16}_32 accessors
> in asm-generic/io.h. This will allow devices that require 32 bits
> register accesses to write data in chunks of 8 or 16 bits (a typical use
> case is SPI, where clients can request transfers in words of 8 bits for
> example). GS101 only allows 32bit register accesses otherwise it raisses
> a Serror Interrupt and hangs the system, thus the accessors are needed
> here. If the accessors are fine, I expect they'll be queued either to
> the SPI tree or to the ASM header files tree, but by providing an
> immutable tag, so that the other tree can merge them too.
>
> The SPI patches were tested with the spi-loopback-test on the gs101
> controller.
>
> Thanks!
> ta
>
> Tudor Ambarus (21):
> spi: dt-bindings: samsung: add google,gs101-spi compatible
> spi: s3c64xx: sort headers alphabetically
> spi: s3c64xx: remove extra blank line
> spi: s3c64xx: remove unneeded (void *) casts in of_match_table
> spi: s3c64xx: explicitly include <linux/bits.h>
> spi: s3c64xx: remove else after return
> spi: s3c64xx: use bitfield access macros
> spi: s3c64xx: move error check up to avoid rechecking
> spi: s3c64xx: use full mask for {RX, TX}_FIFO_LVL
> spi: s3c64xx: move common code outside if else
> spi: s3c64xx: check return code of dmaengine_slave_config()
> spi: s3c64xx: propagate the dma_submit_error() error code
> spi: s3c64xx: rename prepare_dma() to s3c64xx_prepare_dma()
> spi: s3c64xx: return ETIMEDOUT for wait_for_completion_timeout()
> spi: s3c64xx: simplify s3c64xx_wait_for_pio()
> spi: s3c64xx: add missing blank line after declaration
> spi: s3c64xx: downgrade dev_warn to dev_dbg for optional dt props
> asm-generic/io.h: add iowrite{8,16}_32 accessors
> spi: s3c64xx: add support for google,gs101-spi
> spi: s3c64xx: make the SPI alias optional for newer SoCs
> MAINTAINERS: add Tudor Ambarus as R for the samsung SPI driver
>
> .../devicetree/bindings/spi/samsung,spi.yaml | 1 +
> MAINTAINERS | 1 +
> drivers/spi/spi-s3c64xx.c | 447 +++++++++---------
> include/asm-generic/io.h | 50 ++
> 4 files changed, 276 insertions(+), 223 deletions(-)
>
> --
> 2.43.0.429.g432eaa2c6b-goog
>
