On Wed, Feb 10, 2021 at 1:36 AM Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > On 2/9/21 3:32 PM, Bin Meng wrote: > > Hi Philippe, > > > > On Thu, Feb 4, 2021 at 2:02 PM Bin Meng <bmeng...@gmail.com> wrote: > >> > >> On Thu, Jan 28, 2021 at 2:30 PM Bin Meng <bmeng...@gmail.com> wrote: > >>> > >>> From: Bin Meng <bin.m...@windriver.com> > >>> > >>> This includes the previously v3 series [1], and one single patch [2]. > >>> > >>> Compared to v3, this fixed the following issue in patch [v3,6/6]: > >>> - Keep the card state to SSI_SD_CMD instead of SSI_SD_RESPONSE after > >>> receiving the STOP_TRAN token per the spec > >>> > >>> All software tested so far (U-Boot/Linux/VxWorks) do work without > >>> the fix, but it is better to comform with the spec. > >>> > >>> In addition to [2], one more issue was exposed when testing with > >>> VxWorks driver related to STOP_TRANSMISSION (CMD12) response. > >>> > >>> [1] http://patchwork.ozlabs.org/project/qemu-devel/list/?series=226136 > >>> [2] > >>> http://patchwork.ozlabs.org/project/qemu-devel/patch/1611636214-52427-1-git-send-email-bmeng...@gmail.com/ > >>> > >>> Changes in v4: > >>> - Keep the card state to SSI_SD_CMD instead of SSI_SD_RESPONSE after > >>> receiving the STOP_TRAN token per the spec > >>> - new patch: fix STOP_TRANSMISSION (CMD12) response > >>> - new patch: handle the rest commands with R1b response type > >>> > >> > >> Ping? > > > > Will a PR be sent soon to include this series so that the SiFive SPI > > series can follow? > > I had it planned for yesterday but had problems with the mails from > the list, + the CVE (you fixed) took priority. > > Missing review is patch #8 "Fix STOP_TRANSMISSION (CMD12) response" > for which I don't have test yet.
Ping? If there is any comment for patch #8? Regards, Bin