On Sun, 4 Sep 2016 18:53:13 +0200
Karsten Merker <mer...@debian.org> wrote:
> Hello everybody,
> during the mainlining discussion in the Olimex blog at
> the topic of A33 SPI support came up. Chen-Yu and Siarhei have
> pointed out that the chances are good that the A33 can work with
> the already existing SPI drivers and that simply nobody has taken
> a look at those in the context of the A33 yet - so I have taken
> that as an inspiration to take a look :-).
Yes, probably nobody just had an A33 board with SPI pins on an
expansion header. The A33 SoC is primarily used in tablets.
> From the description in the docs published by Allwinner, the
> A23/A33 SPI controller is clearly derived from the one in the
> A31, but it is not completely identical. The biggest difference
> is that the RX and TX FIFOs on the A23/A33 are only 64 Bytes deep
> while they are 128 Bytes deep on the A31. Documentation source:
> Page 463 of the A33 manual:
> Page 889 of the A31 manual:
> The current spi-sun6i driver hardcodes the FIFO depth to 128
> Bytes, so that needs to be changed depending on the SoC type.
> Another difference is an additional configuration option
> influencing the signal sampling on the A23/A33. This is handled
> by bit 13 of the SPI_INTCTL register. The description states:
> "Master Sample Data Mode
> 1-Normal Sample Mode
> 0-Delay Sample Mode
> In Normal Sample Mode,SPI Master samples the data at
> the correct edge for each SPI mode.
> In Delay Sample Mode,SPI master samples data at the
> edge that is half cycle delayed by the correct edge defined
> in respective SPI mode."
> The manual states that "Delay Sample Mode" is the default, which
> sounds a bit strange to me. Can somebody with a deeper knowledge
> of SPI comment on that?
> There are some additional test modes on the A23/33 compared to the A31,
> but those should be irrelevant for normal operations.
> Following are two RFC patches based on Maxime Ripard's sunxi/for-next
> branch at
> that modify the spi-sun6i driver to handle different FIFO depths and add the
> relevant nodes (clock, pinctrl, spi0 controller) to the A23/A33 dtsi files.
> Please note that this is largely untested as I currently don't have any
> A23/A33 hardware (an A33 board is on the way to me but will probably take
> some time to arrive). All I can say at the moment is that the code
> compiles. I would apprechiate very much if people could take a look at it
> and provide feedback, and if somebody has appropriate hardware, I would
> welcome very much if you could subject the code to some practical tests.
Well, basically it is really difficult to get people interested in
spending their time to look at this code at this point :-)
Please first test it on real hardware when the hardware arrives. And
then submit patches to the appropriate mailing list.
> Btw, while checking the docs I have stumbled over one thing that strikes me
> as rather strange: the A23/A33 has two SPI controllers (SPI0 and SPI1), but
> there is no documented pinmux that makes SPI1 available on any pins, so SPI1
> appears to be completely useless?
> Karsten Merker (2):
> spi: sunxi: Add Allwinner A23/A33 support to spi-sun6i
> ARM: dts: sun8i: Add SPI support to the Allwinner A23/A33 dtsi.
> .../devicetree/bindings/spi/spi-sun6i.txt | 6 ++-
> arch/arm/boot/dts/sun8i-a23-a33.dtsi | 38 +++++++++++++++++++
> arch/arm/boot/dts/sun8i-a23.dtsi | 12 ++++++
> arch/arm/boot/dts/sun8i-a33.dtsi | 12 ++++++
> drivers/spi/spi-sun6i.c | 43
> 5 files changed, 103 insertions(+), 8 deletions(-)
You received this message because you are subscribed to the Google Groups
To unsubscribe from this group and stop receiving emails from it, send an email
For more options, visit https://groups.google.com/d/optout.