Re: [802.3_4PPOE] Issue with the state machine

[Yair Darshan <YDarshan@xxxxxxxxxxxxx>]

Hi Chris,

Thanks Chris.

Yes, I saw it, and actually the normal flow is when we start with ENTRY_PRI which from there we can go to detection of the primary.

Going to detection_pri from IDLE_PRI (due to errors that brings us back to IDLE_PRI) which in this case is pending on if secondary is already powered due to the fact that before primary gone to IDLE_PRI, all 4 pairs where powered etc. So, all is good now.

Yair

 

 

From: Chris Bullock (bullock) [mailto:bullock@xxxxxxxxx]
Sent: Monday, October 30, 2017 7:01 PM
To: Yair Darshan <YDarshan@xxxxxxxxxxxxx>; Picard, Jean <jean_picard@xxxxxx>; Dylan Walker (dylanwal) <dylanwal@xxxxxxxxx>
Cc: STDS-802-3-4PPOE@xxxxxxxxxxxxxxxxx
Subject: RE: Issue with the state machine

 

EXTERNAL EMAIL

Hi Yair,

 

The existing logic is intentional.  The idea is that you should only end up in IDLE_PRI if there is an error condition or power is intentionally removed from the primary alternative…..in which case, you should exit back to Main IDLE unless power is applied to the secondary alternative.

 

Thanks,

Chris

 

From: Yair Darshan [mailto:YDarshan@xxxxxxxxxxxxx]
Sent: Sunday, October 29, 2017 5:44 AM
To: Picard, Jean <jean_picard@xxxxxx>; Chris Bullock (bullock) <bullock@xxxxxxxxx>; Dylan Walker (dylanwal) <dylanwal@xxxxxxxxx>
Cc: Yair Darshan <YDarshan@xxxxxxxxxxxxx>; STDS-802-3-4PPOE@xxxxxxxxxxxxxxxxx
Subject: Issue with the state machine

 

Hi guys,

I find issue here, please verify.

Thanks

Yair

---------

From IDLE_PRI to START_DETECT_PRI (145.2.5.7, Page 141 line 10)

Comment

The condition:           !pwr_app_pri * pwr_app_sec           doesn’t make sense that only if the secondary is powered we can  go to START_DET_PRI.

 

pwr_app_pri	pwr_app_sec	START_DET_PRI
0	0	0
0	1	1
1	0	0
1	1	0

The conditions should allow to move to START_DET_PRI in the following conditions:

1. !pwr_app_pri OR
2. !pwr_app_sec

i.e. the condition should be: !pwr_app_pri + !pwr_app_sec          

pwr_app_pri	pwr_app_sec	START_DET_PRI
0	0	1
0	1	1
1	0	1
1	1	0

Proposed remedy:

!pwr_app_pri * pwr_app_sec          

To: !pwr_app_pri + !pwr_app_sec          

 

 

Darshan Yair

Chief R&D Engineer

Analog Mixed Signal Group

Microsemi Corporation

 

1 Hanagar St., P.O. Box 7220
Neve Ne'eman Industrial Zone
Hod Hasharon 45421, Israel
Tel:  +972-9-775-5100, EXT 210.

Cell: +972-54-4893019
Fax: +972-9-775-5111

 

E-mail: <mailto:ydarshan@xxxxxxxxxxxxx>.