Peter Maydell <peter.mayd...@linaro.org> writes: > On Thu, 9 Nov 2023 at 20:59, Philippe Mathieu-Daudé <phi...@linaro.org> wrote: >> >> Hi Peter, >> >> On 9/11/23 20:29, Peter Maydell wrote: >> > On Thu, 9 Nov 2023 at 19:28, Philippe Mathieu-Daudé <phi...@linaro.org> >> > wrote: >> >> >> >> Missing review: #10 >> >> >> >> Hi, >> >> >> >> This series add support for (async) FIFO on the transmit path >> >> of the PL011 UART. >> > >> > Hi; what's the rationale for the "for-8.2" targeting here? >> > What bug are we fixing? >> >> The bug is on Trusted Substrate when the ZynqMP machine is used: >> https://linaro.atlassian.net/browse/TRS-149?focusedCommentId=149574 > > And have we confirmed that the async FIFO support fixes that problem? > That bug report seems to have mostly just speculation in it that > maybe this XXX comment is why...
I've been fighting with numerous issues with the TRS build over the last week so I can confirm I have seen a) a lock up with pl011_write blocking everything under the BQL because data wasn't read fast enough and b) the problem goes away with Philippe's patches. So have a: Tested-by: Alex Bennée <alex.ben...@linaro.org> for the series. > > -- PMM -- Alex Bennée Virtualisation Tech Lead @ Linaro
