RE: Verilog-AMS v2.3/draft4a

Thanks.

Please understand that I will recommend to my employer entity to make
its vote of approval conditional on correction of certain of these
errors.

Regards,
Shalom 

> -----Original Message-----
> From: Sri Chandra [mailto:sri.chandra@freescale.com] 
> Sent: Sunday, June 01, 2008 3:03 PM
> To: Bresticker, Shalom
> Cc: Verilog-AMS LRM Committee; Stuart Sutherland; Wilmore, Jim
> Subject: 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
> 
---------------------------------------------------------------------
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.


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