[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20200511083022.23678-1-chris.ruehl@gtsys.com.hk>
Date: Mon, 11 May 2020 16:30:19 +0800
From: Chris Ruehl <chris.ruehl@...ys.com.hk>
To: Chris Ruehl <chris.ruehl@...ys.com.hk>,
Jack Lo <jack.lo@...ys.com.hk>
Cc: Mark Brown <broonie@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
linux-spi@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: [PATCH v1 0/3] spi: spi-rockchip spi slave mode
The spi-rockchip driver does not implement spi slave mode, but the register map
have a corresponding flag. An example implementation found
here: https://dev.t-firefly.com/thread-101485-1-1.html
This patchset clean the compatiblity names and add support slave mode.
We need the slave mode for our project, but the PCBA is not yet available
in consequence the code in the patch isn't tested yet but need your review.
Patch 1/3
Cleanup, move from the compatibily layer struct spi_master over
to struct spi_controller, and rename the related function calls.
Patch 2/3
Add support for spi slave mode and support function
Patch 3/3
Use OF property num-cs and enable support for cs_gpiods
IMHO its wrong, that the num_chipselect is set fixed to the amount of
native chip-select lines. SPI0 which has two native lines, while the others
SPIs one native cs line only.
Set max_native_cs in the controller struct with ROCKCHIP_SPI_MAX_CS_NUM seems
the correct way to do and let num-cs the OF / dts config set the CS count for
the boards.
Patch against next-20200508
Thanks for review!
Happy hacking
Chris
Signed-off-by: Chris Ruehl <chris.ruehl@...ys.com.hk>
---
Powered by blists - more mailing lists