[U-Boot] [PATCHv2 1/5] fsl: serdes: ensure accessing the initialized maps of serdes protocol

york sun york.sun at nxp.com
Tue Nov 29 22:58:28 CET 2016


On 08/02/2016 04:14 AM, Zhiqiang Hou wrote:
> From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
>
> Up to now, the function is_serdes_configed() doesn't check if the map
> of serdes protocol is initialized before accessing it. The function
> is_serdes_configed() will get wrong result when it was called before
> the serdes protocol maps initialized. As the first element of the map
> isn't used for any device, so use it as the flag to indicate if the
> map has been initialized.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> ---
> V2:
>  - Check initialization-state of the serdes protocol map before serdes init.
>  - Comment the element 'NONE' used as a initialization-state flag.

Zhiqiang,

This patch has been merged but I found an issue with it today. Please 
try to boot P4080DS from NOR flash. It causes recursive calls
fsl_serdes_init()->p4080_erratum_serdes_a005()->is_serdes_configured()->fsl_serdes_init(). 
Please check if you have any similar recursive case for other platforms. 
Please send a fix ASAP.

York


More information about the U-Boot mailing list