To: Tony Taylor Subject: Re: STIL.3 - Review of Draft 06 of STIL-TRC standard From: Gregg Wilder/TI Date: 1/8/03 Here are my comments: 1. p.1 Title does not match PAR - For 1450.2, IEEE required strict matching for title, scope, purpose. 2. p.3 first paragraph below figure 1 - define TRC and explain what TRC statements are. 3. p.3 (2) ...a report of resources required for the pattern may ->be<- created 4. p.5 3.1 for STIL1,3,6 - reword second sentence of each - eg, This STIL extension is commonly referred to as ... to differentiate it from all the other STIL extensions. 5. p.5 4.1 The following is a copy of the conventions as defined ->in<- STIL0 6. p.6 5.1 The regular vector memory has 64 million vectors and 2 thousand vectors of subroutine memory...There is also 2K of subroutine memory - duplicated? 7. p.7 Note 3 does not match example - Signals Data In Out should be SignalCharacteristics Data In Out - CompareStrobe not in example 8. p.8 49:,50:,51: MAX_VECTORS 65_536 - remove _ (or else define integer_expr to allow it) 9. p.13 Note 2,3,4 - examples do not have explicit timing listed in notes. Maybe show it as (for example, 200ns Down and 300ns)? 10. p.13 Note 8 - CompareStrobe not in example 11. p.14 Note 1 - ...even the label syntax is ->is<- arbitrary 12. p.14 5.4.2 - This example is different that ->from<- the previous 13. p.15 267: - ? 14. p.16 5.4.3 ...whereas the inheritance technique illustrated in this annex ->example<- 15. p.18 6. STIL Syntax Description" - remove " 16. p.19 7. A TRC block that is used contain a PatternReport - used to contain or that contains 17. p.19 7. Groups, Specs, and Variables. - connect with previous sentence 18. p.23 9.1(2) last sentence - (a) and ->an<- integer 19. p.24 11. In STIL.1 - STIL1 20. p.25 (1) Environment - STIL.1 - STIL1 21. p.26 (3) Category - as defied in STIL.1 - defined in STIL1 22. p.26 (4) NameChecks - clause 661 ? 23. p.26 (5) PatternCharacteristics - contains information relative the the - to the 24. p.26 (6) - relative the the - to the 25. p.26 (7) - relative the the - to the 26. p.26 (8) - The SystemCharacteristics block is used to contain statement with regard to - ? 27. p.26 (10) ...,the WavefromCharacteristics block - Waveform 28. p.26 (11) ...,the WavefromCharacteristics block - Waveform 29. p.25 12.1 and p.27 13.1 - define integer_expr 30. p.28 (3)(c) STIL.1 - STIL1 31. p.29 13.2 - 528: 65_536 531: 13_985 - remove _ (or else define integer_expr to allow it) 32. p.30 (3) MaxPeriods - This statement allows to specify - This statement specifies 33. p.30 (4) This statement allows to specify - This statement specifies 33. p.30 (4) This says Dynamic or Static shall be specified, but the syntax in 15.1 has parenthesis around - so parenthesis should be removed ? 34. p.30 (5) ...The integer value species - specifies 35. p.30 (6) This statement species - specifies 36. p.30 (6) Syntax in 15.1 has time_expr_min time_expr_max - should these just be time_expr ? 37. p.31,32 16.1 Dynamic - remove underscore 38. p.33 (5)(c) In 13.1, In and Out are used together. Here, InOut is used. Should these be consistent? 39. p.33 (6) If this statement is omitted, then the other statement shall be used - this is unclear - does this mean use the other form of FormatSelect with MaxTimeSets ? 40. p.34 (10) MaxIO - explain the two integer_expr - is it one for input and one for output? 41. p.34 Syntax in 16.1 shows parenthesis around and - are all of these supposed to be optional? 42. p.35 (17) Specify the minimul allowe drive - minimum allowed 43. p.35 (20) TimeLimits - should time_expr_min and _max in syntax be time_expr ? 44. p.36 17. The WaveformDescriptions block contains definitions of waveform shapes that are may be applied - may be (remove are) 45. p.37 (3)(4)(5) - it is not clear to me what these are - maybe add description of data values, IO controls, mask controls 46. p.37 (10) STIL.0 - STIL0 47. p.38 18.1 (also 15.1 and 16.1) - define time_expr 48. p.39 (2) By deafult - default This pattern data characteristic in not - is not 49. p.39 (5) This pattern data characteristic in not - is not 50. p.39 (7) This pattern data characteristic in not - is not 51. p.39 MinVectorsAfter - ...that must exist after to this - remove to 52. p.40 (11)(g) Syntax shows real_expr - should be time_expr ? 53. p.40 (16) This pattern data characteristic in not - is not 54. p.40 (17) This pattern data characteristic in not - is not 55. p.40 (18) This pattern data characteristic in not - is not 56. p.41 (20) ...procedures are typicall supported - typically 57. p.41 (22) This pattern data characteristic in not - is not 58. p.41 646: 65_000 653: 5_000_000 - remove _ (or else define integer_expr to allow it) 59. p.44 20. The definition of category namesand - names and ...(a) Tere are - There are -Gregg Tony Taylor wrote: > > Happy New Year to all, > > ... and a hopeful reminder that you have not forgotten about the TRC review > that we are conducting. I have received one response so far which has > identified some very good points about the document. The working group > plans to go over this input and all others received at the next phone > meeting on 1/17. > > Any input that you can provide will be appreciated. Even if you have not > reviewed the complete document, please send any comments on the parts that > you have reviewed. > > Of course, we will accept input after this identified review period. Our > intent is to clean up the document and present it to IEEE for the formal > ballot, which all of you are invited to participate in (assuming you are > IEEE members and have paid you $10 standards fee). > > - the STIL.3 working group