Fangyi,
I have a few comments:
"The output value of DC_Offset returned by Rx AMI_Init is to be added to the Rx
AMI_GetWave output waveform"
by whom (EDA tool, model maker, etc.)? I would also add the highlighted word
at the
end of this:
"to form the complete waveform at the Rx equalizer output."
I would add the highlighted word:
"If the impulse response was generated by differentiating the channel step
response"
I would change the wording of this a little:
"In the case of Usage In,"
to
"If the DC_Offset parameter is defined as Usage In,"
I would add the highlighted words in these sentences too:
"The complete waveform at the Rx equalizer output is"
"EDA tools may obtain the complete waveform at the Rx equalizer output by"
You only mention at the end of the Usage Rules section that "returned by Rx
AMI_Init"
and this doesn't sound very "definition like". I think we should have words
earlier in
this section stating that the output value is returned by the Init function,
and the
GetWave function should not change (return) it (or whatever you have in mind).
I am sorry, but I don't understand the definition statement of the
NRZ_Threshold parameter
and I think it should be reworded:
"The NRZ threshold voltage applied to the complete waveform at the Rx output,
i.e. the Rx AMI_GetWave output waveform plus the output value of DC_Offset
returned by Rx AMI_Init."
What does "applied to" mean, and who does that and when?...
"Voltages that do not fall into one of these regions are considered a symbol
error."
I am not sure about this condition being a symbol error, it could be a "no
change"
situation...
I would change
"The algorithmic model is responsible for determining any output values from
the AMI_Init and AMI_GetWave calls for the EDA tool to use during waveform and
eye processing"
to:
"The algorithmic model is responsible for determining the output value of this
parameter in the AMI_Init and AMI_GetWave functions for the EDA tool to use
during waveform and eye processing"
but again, the usage rules should spell out when the DLL should return a value
(Init, GetWave,
or both).
That's it for now...
Thanks,
Arpad
=========================================================================
From: ibis-macro-bounce@xxxxxxxxxxxxx [mailto:ibis-macro-bounce@xxxxxxxxxxxxx] ;
On Behalf Of Fangyi Rao
Sent: Tuesday, July 16, 2019 1:56 AM
To: ibis-macro@xxxxxxxxxxxxx
Subject: [ibis-macro] Re: new BIRD for single-ended signal in AMI
Correcting a typo.
"... from InOut to In or InOut"
From: ibis-macro-bounce@xxxxxxxxxxxxx<mailto:ibis-macro-bounce@xxxxxxxxxxxxx>
<ibis-macro-bounce@xxxxxxxxxxxxx<mailto:ibis-macro-bounce@xxxxxxxxxxxxx>> On
Behalf Of Fangyi Rao
Sent: Monday, July 15, 2019 11:53 PM
To: ibis-macro@xxxxxxxxxxxxx<mailto:ibis-macro@xxxxxxxxxxxxx>
Subject: [ibis-macro] Re: new BIRD for single-ended signal in AMI
[EXTERNAL]
Based on Walter's suggestion I changed the Usage of DC_Offset from In to In or
InOut. For Usage In, the output value of DC_Offset is assumed to be 0V
according to the default value.
Fangyi
From: ibis-macro-bounce@xxxxxxxxxxxxx<mailto:ibis-macro-bounce@xxxxxxxxxxxxx>
<ibis-macro-bounce@xxxxxxxxxxxxx<mailto:ibis-macro-bounce@xxxxxxxxxxxxx>> On
Behalf Of Fangyi Rao
Sent: Sunday, July 14, 2019 12:40 PM
To: ibis-macro@xxxxxxxxxxxxx<mailto:ibis-macro@xxxxxxxxxxxxx>
Subject: [ibis-macro] new BIRD for single-ended signal in AMI
[EXTERNAL]
Hi All,
Per last week's ATM discussion I drafted the attached new BIRD titled "Enable
Single-ended Signal in AMI Modeling". The basic idea is simple.
1. DC_Offset usage is InOut. Defaults for both input and output values are 0
V.
2. Physical Rx input waveform = input waveform to Rx AMI_GetWave + input
value of DC_Offset
3. Complete waveform at Rx output = output waveform of Rx AMI_GetWave +
output value of DC_Offset returned by Rx AMI_Init
4. NRZ_Threshold is to be applied to the complete waveform at Rx output.
Default is 0 V.
I'd like to introduce the BIRD at this week's ATM.
Regards,
Fangyi