Re: Comments on Version 2.3 draft 3

From: Geoffrey.Coram <geoffrey.coram_at_.....>
Date: Fri Mar 28 2008 - 08:37:04 PDT
Ken Kundert wrote:
> Turn on generation of hyperlinks, so it is easy to navigate PDF file.
> 
> Page v:
> Ken Kundert, Designers Guide LLC --> Ken Kundert, Designer's Guide 
> Consulting
> Xpedion Design Systems --> Agilent
> (c) 2007 --> (c) 2008


There are a lot of 2007's in the document:
page 1, Copyright 1999-2007
footer on all pages

On page v, affiliations:
There's an extra ) after Waterloo.
I think Kevin Cameron recently moved from Sonics to ?
There's no affiliation listed for Paul Floyd.


The roman-numeral numbering restarts on page 7 of the PDF file
with the table of contents.

On the second iii, 9 of the PDF, "analog initial block"
should start with a capital A.


Page 30 (47 of the PDF) has a cross-ref to 3.2.1, but this
section has now been eviscerated; the cross-ref should
probably be to 4.2.1.1 or 4.2.1.2

Page 30 (47 of the PDF) has an editor's note asking if ' is
necessary for the list of values for a string parameter:
   parameter string gatetype = "nmos" from {"nmos", "pmos"};

I don't have an answer to that, but I do note that the braces {}
need to be red (required syntax item) in Syntax 3-2 and in
A.2.5 for value_range.


I'm OK with "gatetype" on page 31 (48 of the PDF) for mos,
but that doesn't work for the ebersmoll (a BJT model) on
page 32 (49 of PDF).  Please use "bjttype" (though type is
in common use in many existing Verilog-A compact models --
I guess we'll need some sort of `begin_keywords to prevent
trouble with the SV keyword "type").


Page 38 (55 of PDF): I have no idea what "multi-disciplinaries"
are; I'd suggest rewording to "multi-disciplinary systems."


I'm a little puzzled about 4.2.1.2:
   Individual bits that are x or z in the net or the variable
   shall be treated as zero upon conversion.

Is this applicable for AMS?  I'm not sure we deal with bits.


Page 67 (84 of the PDF): several of the items in Syntax 4-2
are not "analog_filter_functions": ddx, ac_stim, white_noise,
flicker_noise, noise_table -- and they should not be subject
to the same restrictions as the true analog operators that
require state information.

For the true analog operators, idt, idtmod, etc.: why don't
sections 4.5.4 and 4.5.5 (etc.) refer to Syntax 4-2?  Since
ddx() is not an analog filter, it's not a good idea to put
this syntax in 4.5.6.

Page 88 (105 of the PDF): arrayinit should probably be
arrayadd, which is defined in 4.7.1 -- this wasn't in 2.2,
who added it?


I'll start a new e-mail for Section 5 and following.

-Geoffrey

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
Received on Fri Mar 28 08:37:59 2008

This archive was generated by hypermail 2.1.8 : Fri Mar 28 2008 - 08:38:09 PDT