Re: [PATCH 0/8] spi: dw-dma: Add max SG entries burst capability support
On 04-08-20, 22:14, Mark Brown wrote: > On Fri, Jul 31, 2020 at 03:59:54PM +0300, Serge Semin wrote: > > On Fri, Jul 31, 2020 at 12:26:12PM +0300, Andy Shevchenko wrote: > > > On Fri, Jul 31, 2020 at 10:59:45AM +0300, Serge Semin wrote: > > > > > Note since the DMA-engine subsystem in kernel 5.8-rcX doesn't have the > > > > max_sg_burst capability supported, this series is intended to be applied > > > > only after the "next"-branch of the DMA-engine repository is merged into > > > > the mainline repo. Alternatively the series could be merged in through > > > > the > > > > DMA-engine repo. > > > > This needs to be thought through... > > > There is nothing to think about: either Mark completes review/accepts the > > series > > and Vinod agrees to merge it in through the DMA-engine repo, or we'll have > > to > > wait until the next merge window is closed and then merge the series in > > traditionally through the SPI repo. > > Well, the merge window is open now so this won't get applied till -rc1 > anyway. Yes and the max_sg_burst capability is in my next and should be in PR to Linus this week, so rc1 should have this. -- ~Vinod
Re: [PATCH 0/8] spi: dw-dma: Add max SG entries burst capability support
On Fri, Jul 31, 2020 at 03:59:54PM +0300, Serge Semin wrote: > On Fri, Jul 31, 2020 at 12:26:12PM +0300, Andy Shevchenko wrote: > > On Fri, Jul 31, 2020 at 10:59:45AM +0300, Serge Semin wrote: > > > Note since the DMA-engine subsystem in kernel 5.8-rcX doesn't have the > > > max_sg_burst capability supported, this series is intended to be applied > > > only after the "next"-branch of the DMA-engine repository is merged into > > > the mainline repo. Alternatively the series could be merged in through the > > > DMA-engine repo. > > This needs to be thought through... > There is nothing to think about: either Mark completes review/accepts the > series > and Vinod agrees to merge it in through the DMA-engine repo, or we'll have to > wait until the next merge window is closed and then merge the series in > traditionally through the SPI repo. Well, the merge window is open now so this won't get applied till -rc1 anyway. signature.asc Description: PGP signature
Re: [PATCH 0/8] spi: dw-dma: Add max SG entries burst capability support
On Fri, Jul 31, 2020 at 12:26:12PM +0300, Andy Shevchenko wrote: > On Fri, Jul 31, 2020 at 10:59:45AM +0300, Serge Semin wrote: > > ... > > > Note since the DMA-engine subsystem in kernel 5.8-rcX doesn't have the > > max_sg_burst capability supported, this series is intended to be applied > > only after the "next"-branch of the DMA-engine repository is merged into > > the mainline repo. Alternatively the series could be merged in through the > > DMA-engine repo. > > This needs to be thought through... There is nothing to think about: either Mark completes review/accepts the series and Vinod agrees to merge it in through the DMA-engine repo, or we'll have to wait until the next merge window is closed and then merge the series in traditionally through the SPI repo. -Sergey > > I gave some simple comments (and on top just try not to modify the same lines > inside one series two or more times, e.g. ret = func() -> return func() -> ret > = func() in one case). > > -- > With Best Regards, > Andy Shevchenko > >
Re: [PATCH 0/8] spi: dw-dma: Add max SG entries burst capability support
On Fri, Jul 31, 2020 at 10:59:45AM +0300, Serge Semin wrote: ... > Note since the DMA-engine subsystem in kernel 5.8-rcX doesn't have the > max_sg_burst capability supported, this series is intended to be applied > only after the "next"-branch of the DMA-engine repository is merged into > the mainline repo. Alternatively the series could be merged in through the > DMA-engine repo. This needs to be thought through... I gave some simple comments (and on top just try not to modify the same lines inside one series two or more times, e.g. ret = func() -> return func() -> ret = func() in one case). -- With Best Regards, Andy Shevchenko