Bug #118891: 【BSP】【EVT3】【ST】【eth】设备过压保护后，恢复正常电压开机，出现eth与中控连接失败的情况 - Figure - 中科创达

Bug #118891

【BSP】【EVT3】【ST】【eth】设备过压保护后，恢复正常电压开机，出现eth与中控连接失败的情况

Added by 移动测试一组_CDTS 刘强 about 2 years ago. Updated almost 2 years ago.

Status:

CLOSED

Start date:

2023-06-25

Priority:

High

Due date:

Assignee:

移动测试一组_CDTS 刘强

% Done:

Category:

BSP

Target version:

Need_Info:

Found Version:

0112

Resolution:

FIXED

Degrated:

Severity:

Major

Verified Version:

Reproducibility:

Every time

Fixed Version:

Test Type:

Root cause:

错过了mcu的linkup消息导致无法初始化phy

Description

前置条件
None

操作步骤：
1.设备烧写0112 user版本
2.与中控正常连接
3.副屏通过稳压电源-触发过压保护关机
4.恢复正常电压开机

预期结果：
设备正常启动

实际结果：
副屏与中控eth连接断开

tlog_iov0201017500006162025411a2305070000003671_000076_0625153824.tar.gz (2.29 MB) 移动测试一组_CDTS 刘强, 2023-06-25 16:07

1.txt (3.42 MB) 移动测试一组_CDTS 刘强, 2023-06-25 16:07

History

#1 Updated by 移动测试一组_CDTS 刘强 about 2 years ago

补充信息：重新断电-上电后恢复正常
补充信息：触发欠压保护后，恢复正常电压上电-不复现问题
补充信息：目前该问题必现

#2 Updated by 移动测试一组_CDTS 刘强 about 2 years ago

补充信息：
过压之后等几秒电流降下去关机后再回复正常开机就不会复现，如果过压后马上回复正常就会复现
低压的话，过几秒和马上回复都不会出现

#3 Updated by CD TPM-王祥林 about 2 years ago

Assignee changed from CD TPM-王祥林 to CD_BSP 彭文亮

#4 Updated by CD_BSP 彭文亮 about 2 years ago

结论： 应该是mcu发送的iop_link现象过早，或者是由于操作上引起了内核的启动延迟。phy芯片的初始化发生在了 phy状态准备好之前。正常的顺序应该是，phy状态准备好->phy芯片的初始化。@王祥林
01-01 08:00:09.266 0 0 I : [ 6.324645] mv2112_setup_forced
01-01 08:00:09.368 0 0 E [ 6.421155] mv2112_read_status: autoneg not complete;time=6420 ms
01-01 08:00:09.822 0 0 E [ 6.880107] link_delay_work_func: DTC_ETH: SQI=0, link=0
...
06-25 15:56:24.817 0 0 E [ 2290.400219] link_delay_work_func: DTC_ETH: SQI=0, link=0

#5 Updated by CD_BSP 彭文亮 about 2 years ago

正在看能否通过修改phy link的逻辑，来规避此问题。

#6 Updated by IoT scm about 2 years ago

Gerrit Merge Information：

ID	Project	Branch	Uploader
210928	general/kernel/msm-4.19	Pre_figure_turbox-c2130c-la1.1-vendor-dev	wenliang.peng@thundersoft.com
BSP:Ethernet: Optimized the initialization time of phy. Added 2 variables phyisready and phyisnoinit. Add a check of the physiisready variable before the initialization of the phy. TC-RID: 0301-0400601 IssueID: TS-R-DF-118891 Change-Id: Ic574dd1ea33ac8f2a1f44c2d1a7bf3b92cb5f9ae

#7 Updated by IoT scm about 2 years ago

Gerrit Merge Information：

ID	Project	Branch	Uploader
211573	general/kernel/msm-4.19	Pre_figure_turbox-c2130c-la1.1-vendor-release_sorp	wangxl0510@thundersoft.com
BSP:Ethernet: Optimized the initialization time of phy. Added 2 variables phyisready and phyisnoinit. Add a check of the physiisready variable before the initialization of the phy. TC-RID: 0301-0400601 IssueID: TS-R-DF-118891 Change-Id: Ic574dd1ea33ac8f2a1f44c2d1a7bf3b92cb5f9ae (cherry picked from commit 00a6fcb7889b9c2caea6b80c1a3298d1665764b8)