[U-Boot] [PATCH v2] usb: xhci-mem: Fix scratchpad array issue

Ye Li ye.li at nxp.com
Tue Jan 8 04:00:14 UTC 2019


On 1/7/2019 5:54 PM, Bin Meng wrote:
> On Mon, Jan 7, 2019 at 10:45 AM Ye Li <ye.li at nxp.com> wrote:
>>
>> After updating the value of dev_context_ptrs[0], we should flush this
>> from cache to memory. Otherwise the xhci controller won't use it.
>>
>> Signed-off-by: Ye Li <ye.li at nxp.com>
>> Reviewed-by: Marek Vasut <marex at denx.de>
>> ---
>> Changes for v2:
>>    - Change to use sizeof(ctrl->dcbaa->dev_context_ptrs[0])
>>
>>  drivers/usb/host/xhci-mem.c | 3 +++
>>  1 file changed, 3 insertions(+)
>>
> 
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
> 
> But you need do the same in xhci_scratchpad_free() too. Please fix it in v3.
> 
> Regards,
> Bin
> 
The xhci controller has been stopped before calling xhci_scratchpad_free, is it necessary to 
add flush in this function? And I don't see other buffers are flushed when freeing. 

Best regards,
Ye Li


More information about the U-Boot mailing list