Minutes of Draft2 review (Chap 5, 6) - 17 Jan 2007

Date: 17th Jan 2008 (9:00pm Pacific)

Attendees:
* Patrick O'Halloran - Tiburon
* Martin O'leary, Cadence
* Marq Kole, NXP
* Dave Miller, Freescale
* Marek Mierzwinski, Tiburon
* Boris Troyanovsky, Tiburon
* David Cronauer, Synopsys
* Stu Sutherland, Sutherland HDL
* Sri Chandra, Freescale

The following chapters were reviewed:
   - Chapter 5, Analog behavior
   - Chapter 6, Hierarchical structures

Review notes: (page number references are to the actual document page 
numbers and not the pdf page numbers)

* How to track the changes
   - Any editorial changes will be tracked through the minutes, mantis 
items are not required (typos, cross-references, english, consistency etc)
   - Any technical changes will be tracked through mantis - changes to 
modulus operator (suggested by Paul/Dave) and changes to chapter 2 
(removal of clauses 2.7.1 and 2.7.2). Sri will raise mantis ticket on 
behalf of dave.


* Chapter 4, Expressions
   - (pg 85): Array literals using the concatenation operators have been 
changed to follow the system verilog conventions with a ` in front of 
it. Stu will look for this globally in the document.


* Chapter 5, Analog behavior
   - (pg 99) Clause 5.3.2, Block Names
     -- are localparameters declaration allowed inside named blocks. 
analog_block_item_declaration does not seem to take in local parameters 
as per A.2.8 (need to follow up with Graham)
     -- remove reference to defparam in the example block as it is being 
phased out.
     -- remove paranthesis "()" for modules top & example with no arguments
   - (pg 112) The new proposal clause 6.6.1 Hierarchical references 
being discussed should also include that branch_lvalue for indirect 
branch assignment cannot be an hierarchical reference. This is currently 
missing in the proposal.
   - (pg 113) Clause heading for "Multiple indirect contributions" to be 
fixed.
   - (pg 116) The reference to case expression should be changed to 
analog_expression
   - (pg 119) Make additional note to clause 5.10.1 that the "," 
operator is equivalent to the "or" operator.
   - (pg 121) remove reference to Table5.1 in footnote a.
   - (pg 122) Change the general form given for cross/above/timer 
operators to an actual BNF reference, as specified in A.6.5
   - (pg 122) remove ";" at the end of syntax. This will automatically 
vanish with the change above
   - (pg 123) remove "blank space" before the ";" terminating the analog 
statements.
   - (pg 127) Typo in branch_declaration in the BNF specified as part of 
module_or_generate_item_declaration. This will be a global fix, as this 
referenced in couple of places in the document coming from the BNF.
   - (pg 127) parameter_declaration for non_port_module_item should be 
bold red
   - (pg 128) cross should be in bold in the comparator example

* Chapter 6, Hierarchical structures
   - (pg 129) The word statement specified as part of "defparam 
statement" should be in italics in section 6.3 instead of 6.3.1
   - (pg 133) The arguments specified for the modules badres and parares 
should have blank space after ",". This might be in other places of the 
document also.
   - (pg 154). The correct cross-reference is 7.8 as chapter after 4 
have been moved one ahead.


Next review (24 Jan 2008):
   - Clause 6.6.1 hierarchical references
   - Updates to analog initial block proposal
   (We will continue from Chapter 7 onwards on the 31st of Jan 2008 
committee meeting)


cheers,
Sri
-- 
Srikanth Chandrasekaran
Design Technology (Tools Development)
Freescale Semiconductor Inc.
T:+91-120-439 5000 p:x3824 f: x5199

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.