[U-Boot] [PATCH v3] drivers: usb: xhci-fsl: Implement Erratum A-010151 for FSL USB3 controller

Sriram Dash sriram.dash at nxp.com
Fri Sep 16 12:24:48 CEST 2016


>From: Marek Vasut [mailto:marex at denx.de]
>On 09/16/2016 11:35 AM, Sriram Dash wrote:
>
>[...]
>
>>>> I agree to your point. We can set the bit from fsl specific file
>>>> with the function setbits_le32(fsl_xhci->dwc3_reg->g_usb3pipectl[0],
>>>> DWC3_GUSB3PIPECTL_DISRXDETP3);
>>>> If any other Soc, other than fsl/nxp wants the functionality, they
>>>> will be using the same in their respective code. What do you say?
>>>
>>> Why do you use setbits_le32() instead of writel() ?
>>>
>>
>> We will be modifying a single bit. So, better to use setbit_le32 and
>> leave other bits unchanged.
>
>In that case, you should use clrsetbits_le32() to clear the bit first in case someone
>decided to clear it instead.
>

OK. Will modify in v4.

>[...]
>
>
>--
>Best regards,
>Marek Vasut


More information about the U-Boot mailing list