On Sun, Jan 12, 2014 at 10:32:27PM +0100, Laurent Pinchart wrote: > Hi Geert, > > Thank you for the patches. > > On Sunday 12 January 2014 11:48:38 Geert Uytterhoeven wrote: > > This patch series integrates Renesas RSPI on the RZ/A1H aka R7S72100 SoC, > > and the Genmai development board. > > > > The series is based on renesas-devel-v3.13-rc7-20140109v2, with Magnus' > > pinctrl, Wolfram's riic, and Simon's sh_eth work applied on top. > > [Simon: I don't know in which order you will merge these. If you want me > > to rebase my patches, just ask! Thanks!] > > > > It was tested on the Genmai development board (genmai-reference), using > > an in-kernel version of Documentation/spi/spidev_test.c, exercising > > loopback mode on rspi1, with the same outcome as the SDK reference code. > > > > It will not yet work with the non-reference version of genmai, due to > > missing pinmux configuration in setup-r7s72100.c. > > The whole series looks good to me. The first patch depends on r7s72100 pinmux > support, for which I'm still waiting a new version from Magnus. The rest > could > already be applied as far as I'm concerned.
Thanks, that sounds reasonable to me. But I would like to hear from Magnus. > > > [1/5] pinctrl: sh-pfc: r7s72100: Add RSPI support > > [2/5] ARM: shmobile: r7s72100: Add RSPI clocks > > [3/5] ARM: shmobile: genmai: Add RSPI support > > [4/5] [RFC] ARM: shmobile: genmai: Add preliminary RSPI pinmux > > [5/5] ARM: shmobile: genmai: Enable RSPI in defconfig > > > > Changes in V2: > > - Add "ARM: shmobile: genmai: Add RSPI children" > > - Add "[RFC] ARM: shmobile: genmai: Add preliminary RSPI pinmux" > > > > Changes in V3: > > - Implement review comments > > - Add defconfig update > > - Remove DT support patches, they will be sent in a separate series > > - Merge "ARM: shmobile: r7s72100: Add RSPI platform devices" and > > "ARM: shmobile: genmai: Add RSPI children" > > -- > Regards, > > Laurent Pinchart > -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
