On 01/06/2018 07:46 PM, Jason Rush wrote: > On 1/6/2018 9:42 AM, Marek Vasut wrote: >> On 01/06/2018 02:39 PM, Goldschmidt Simon wrote: >>> On Fri, 05/01/2018, Marek Vasut wrote: >>>> On 01/05/2018 08:31 PM, Goldschmidt Simon wrote: >>>>> On Fri, 05/01/2018 Marek Vasut wrote: >>>>>> On 01/05/2018 04:49 PM, Goldschmidt Simon wrote: >>>>> <snip> >>>>> >>>>>>>>> OK, so I need these patches to get qspi work on socfpga: >>>>>>>>> >>>>>>>>> - Series "spi: cadence_spi: Adopt Linux DT bindings" (v4) from Jason >>>>>>>>> Rush: >>>>>>>>> https://patchwork.ozlabs.org/project/uboot/list/?series=13864 >>>>>>>>> - Patch "Revert "spi: cadence_qspi_apb: Use 32 bit indirect read >>>>>>>>> transaction when possible" (v2) >>>>>>>>> https://patchwork.ozlabs.org/patch/838871/ >>>>>>>> I've waited for ack/tested-by from marek or someone who usually worked >>>>>>>> on these cadence. >>>>>>> Vignesh acked, who already did some of the last changes. But Ok, I've >>>>>>> added Marek to the loop. >>>>>>> >>>>>>> Marek, do you see any problems here? Are you running QSPI on the >>>>>>> socfpga platform anywhere? >>>>>> I am not entirely sure what this partial thread is all about, do you >>>>>> need some patches Acked ? Repost them including the Acks collected >>>>>> already and CC me, I want to review them. PW link is not enough. >>>>> Marek, you were one of the people addressed in "to:" when Jason Rush >>>>> sent "[PATCH v4 1/5] spi: cadence_spi: Sync DT bindings with Linux" >>>>> and successors on Nov. 16th 2017. >>>>> >>>>> You were also in the "to:" field when I sent "[PATCH v4 1/5] spi: >>>>> cadence_spi: Sync DT bindings with Linux" on Nov. 17th 2017. >>>>> >>>>> Should I resend them anyway? >>>> I really dunno what to make of this sparse thread, sorry. Also, I don't >>>> quite remember what happened in this thread on November 17th, sorry. >>>> >>>> So maybe you should elaborate what you expect me to do ... or just >>>> resend the patches, yes. >>> OK, I'll resend the patches then. I'll try to combine the 2 series >>> into one but have to check with Jason first. >> Super, thanks. We really need to sort this QSPI stuff out, it's been >> going on for way too long. >> >> Just make sure to submit it soon, MW opens on the 8th. >> > > There was a minor upstream change to one of the files since I submitted v4 of > my > cadence device-tree patchset, so I rebased and resent the patchset as a v5. I > included everyone that was originally involved with the patches and added a CC > for Marek. > > This is only the patchset for the device tree changes for the cadence qspi > driver, > Simon will still have to add the patch that fixes the cache invalidation bug > in the > cadence driver.
Sigh, can we get a single patchset out which fixes the problem ? I mean, if I understand this correctly, you're all addressing one single problem, but with two patchsets, yes ? -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
