Re: Verilog-AMS v2.3/draft4a

Shalom,

Thanks for your comments and the review points on the LRM v2.3/draft4a. 
I will collate the 3 emails that you have sent with the errata and 
record them on Mantis which the committee can take up and fix in the 
next revision of the LRM. The current standard has been sent to the 
board for consideration at this point.

I will send you the Mantis reference to you once i have done the same, 
so that you can ensure all points are covered. At this point i plan to 
put these items under one ticket.

Regards,
Sri

Bresticker, Shalom wrote:
> Hi,
> 
> Clause 3 starts,
> 
> "Verilog-AMS HDL supports integer, genvar, real, and parameter data
> types as found in IEEE std 1364-2005 Verilog HDL. It includes the string
> data type defined by IEEE std 1800-2005 SystemVerilog. It also modifies
> the parameter data types and introduces array of real as an extension of
> the real data type."
> 
> Comments:
> 
> 1. 1364-2005's genvar is not a data type.
> Neither are parameters.
> 
> 
> 2. Arrays of real already exist in 1364-2005.
> Perhaps the intent was arrays of parameters.
> 
> 
> 3. 3.3 has the example, two lines before Table 3-3,
> "r = {"H",""}; // yields "H\0". "" is converted to 8'b0l".
> 
> That should be 8'b0, not 8'b01.
> 
> 
> 4. The BNF non-terminal constant_concatenation is used for array
> initialization, as in 
> real_type and variable_type in Syntax 3-1, param_assignment in Syntax
> 3-2, net_assignment in Syntax 3-6, for example. 
> 
> That non-terminal is missing the apostrophe preceding the curly brackets
> that 3.4 says is needed: "For parameters defined as arrays, the
> initializer shall be a constant_param_arrayinit expression which is a
> list of constant expressions containing only constant numbers and
> previously defined parameters within '{ and } delimiters."
> 
> And the "constant_param_arrayinit" mentioned there does not appear
> anywhere.
> 
> 
> 5. In 3.4.2, "The use of brackets, [ and ], indicate inclusion of the
> end points in the valid range. The use of parenthesis, ( and ), indicate
> exclusion of the end points from the valid range. It is possible to
> include one end point and not the other using [ ) and ( ]."
> 
> "parenthesis" should be plural, "parentheses".
> "indicate" should be "indicates", twice.
> 
> Regards,
> Shalom
> ---------------------------------------------------------------------
> Intel Israel (74) Limited
> 
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
> 
> 

-- 
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.