I will add the inline comment in the patch, send it to you later.
From: York Sun
Sent: Wednesday, August 09, 2017 12:14 AM
To: Xiaowei Bao <xiaowei....@nxp.com>; email@example.com; Priyanka Jain
<priyanka.j...@nxp.com>; Z.q. Hou <zhiqiang....@nxp.com>; M.h. Lian
Subject: Re: [PATCH] armv8: layerscape platform pcie link up state judgment
On 08/07/2017 11:56 PM, Xiaowei Bao wrote:
> Hi York,
> I will pay attention to the case of the case in commit message.
> This patch is for some special reset times for longer pcie devices, in this
> case, the pcie device may on polling compliance state, the RC considers the
> pcie device is link up, but the pcie device is not link up, only the L0 state
> is link up state. So add the link up status judgement mechanisms.
> About 100ms timeout, the pcie spec does not specify the link up timeout time,
> and the link up state is determined by a state machine. The state machine
> implementation is relatively complex, refer to uboot of other platform pcie
> link up state to determine the realization of the mechanism, we evaluated a
> timeout, in detect state consider the pcie device is link down, in L0 state
> consider the pcie device is link up, within 100ms in other states can be
> restored to the L0 state considers the pcie device is link up .
Can you put this information to inline comment? It will help us when we read
the code later.
U-Boot mailing list