Re: [payload] WGLC on draft-ietf-payload-tetra-00

"Dave Satterlee (Vocal)" <Dave.Satterlee@vocal.com> Tue, 10 July 2018 18:04 UTC

Return-Path: <Dave.Satterlee@vocal.com>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1B047130FB7 for <payload@ietfa.amsl.com>; Tue, 10 Jul 2018 11:04:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id NoJv8slUI7x2 for <payload@ietfa.amsl.com>; Tue, 10 Jul 2018 11:04:28 -0700 (PDT)
Received: from cuda.olm1.com (cuda.olm1.com [72.236.255.32]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0CC2F130DCE for <payload@ietf.org>; Tue, 10 Jul 2018 11:04:27 -0700 (PDT)
X-ASG-Debug-ID: 1531245831-092fd34fcb94d20001-U2jSCT
Received: from host105.olm1.com (host105.olm1.com [72.236.255.15]) by cuda.olm1.com with ESMTP id pvetjzmSpkPLJIQT (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for <payload@ietf.org>; Tue, 10 Jul 2018 14:03:51 -0400 (EDT)
X-Barracuda-Envelope-From: Dave.Satterlee@vocal.com
X-Barracuda-Apparent-Source-IP: 72.236.255.15
Received: from [192.168.11.222] (rrcs-72-43-202-98.nys.biz.rr.com [72.43.202.98]) by host105.olm1.com (Postfix) with ESMTPSA id B8DB8B46D01; Tue, 10 Jul 2018 14:03:51 -0400 (EDT)
From: "Dave Satterlee (Vocal)" <Dave.Satterlee@vocal.com>
Openpgp: preference=signencrypt
Autocrypt: addr=Dave.Satterlee@vocal.com; prefer-encrypt=mutual; keydata= xsFNBFefUJQBEACplue4WNnPVdkcwqiUNnmdmvwADKWk7Leil/8zu7HcFDldPe5OBR7PEutO sLZy679YVSGGxrQhpLQCdnQHXchMxxwCluCcKFDb2G9DDR60ADVW0cPrbk1YQ8/S/qHwRG3F DXQBOQhM+Yp/RMjvT1vXXWlyoEFYpciMO9Zz71SUG/arZddXHE6+npWGhIqOQYcW5KNWLvq5 6FyZC/+Ak5CnS1V5XwRP1wKjSxS+ReghT3ykp5Pwtsh472ix22WvHO3MOktrFUr88bbVMjKP M9wFDkAHuWRNEHHDycN5byRE1IYwEED1YnjTkxRPVaHx5j8rtov3QZz52dBukn+ln4Kip4GV p9MgsUnnV85IDVUzGDymD/rKoQ030GBIXG8jbrtCd4E/qTLQF/kv8LNhuq+kPXFHj8s3ODjF NzhD8tNUN2l6poAG8Qpy114ybgy8L7Lco9ts2Ham4Q1je5pKNQGkxrgFyaKXGE6euVP2pqOC UzdqgsW8dRb6/SrR5dDEUd8WOjByS0lhH11J2RsFkF2gS5aCxwlDVCBL/EdtzD5J/qT5hDDB gNd7EemB7k7myWaEYWa0hDMvRoQ9B0QpASTqTfmtUDC0QK99qWYXzzuP+sViL1cU++L58SBM A9hxnQdHqbAIpJu/vxVdsmELyrGg0estmZ2MOFDZqkbz7MOpmQARAQABzTFEYXZlIFNhdHRl cmxlZSAoVm9jYWwpIDxEYXZlLlNhdHRlcmxlZUB2b2NhbC5jb20+wsF/BBMBCAApBQJXn1CU AhsjBQkJZgGABwsJCAcDAgEGFQgCCQoLBBYCAwECHgECF4AACgkQX+7cGIMiwvmcwRAAipLB 7qq+QL3e93wRPG8EAZRkeyID/O7FbQwcAi9WtCmi4eL3yrASRjifHb8o0XZ7J9I0a+hQ6QnJ Lin2tehfYeD41TOSXfqTGGbmMQ5zpwQr+cbrASZbgoUYfvlV1ad9eL9pZPKjiJdtD4ZQtw9N DVa3/f53PRUHJJRi4hADpJK3xuYNd6UAN5SG4fGV1nQy7+/25OTpE0Yc3+avv9AMU46LlcgD cdC9tvUXlUoY9fvu+NWSLuSWh/g4z5Ye/bdlN/2J9TMpizxuhJBEy2M02DUI+FxwY7Vvys6D tkMqrMXfD2XV0gIlg1hyW+WwKypkvPjRcveZ0vukzNZBWVyU9ORsdkfD26vGSbfBT+oiZCTb afjbhZzwDKK3KzTeITbjhSTNnWmOKeQXqlV9nXSVYZxJObzIzzY7iUujmWFgAK4BklfIA0SG Z7TxDU52oIZhF9mKUESA12AELhXx8ZVM4CPO2xNEhbFVLTr6YCr1+uDamyWl0vqzz4Q9NPGI xU2yFHvdzE6fu547hU7BCZ+7aEx59HA9HIBowHFUk5JjmmNU5vv+VMu2/iqfGJdZQnmC2Gtn q32Mtq2PFmCVEKpWHC9WAehihD5hE0qPj+uW6MtNuJhySMU8wJZZVGK4AtcALh24ZBFF+zon mk2yFOsy9YIjqTU6+yUiR9Qv85LtKiDOwU0EV59QlAEQAMaYaxnHmdvoDR8DOyJJSMCDFfQf d2J3oh2mkB1jkY7PKrD2w94yklvFaeczfD4MiAzCy6maV/yJ4fS+UbDaLaHstHXqWjupvf9z l73edz/e5nEuf2BGf8lzs9y5N6tIL+SQmjjaO7zisQyxTLedRBTJSKrZwjPJISN3Xlo67Dh9 6RYwhObjFCAN+Q8F5q8fZvdn+w4nzkvlowihkHYuv70t4v2Kl85TBwWmVn+f238UcgKXLR9f wJCd++ZPhjttYkcAjYR+5Yh18IQv6wqfdI+LQbH0Wuh2QhcD50Cv+QdWl+0ERkQReiH10Pej plv3FEO/heMqRGDjm4qcZvgUHDT1xP2B39HJO8pGOmEgVdDm71fTrbxL4+eo09NJymccMvz8 Lki0u9JulwNM4v90AySMZCTTzwvNVQkEFxAPgSLzNpnYcXhX5pm4pm9Zmrm2GMeh+8Lc+V4b rkPiTf+Unnbj4ZPZpbESzxbxlLlrJLai2IEw+W4siBPawQ9USfDaoUbjM2NZ/B7m/f9hddfT OrHOm9ex0KPNuKLXclQsnJd1yvc+UCH3Msb7zWApkH9R3eFCRiYAchrswEla8PO6UpiQO7KY SeQxPIGA+e+XSUpbnnnSe7lWtUL5kLneUXI67I+SCKST5yRZ1n9dIfxFxO6kP69sh4nxfcHK s10FIeBjABEBAAHCwWUEGAEIAA8FAlefUJQCGwwFCQlmAYAACgkQX+7cGIMiwvk41g//SXIa p+Uvj2Wswi5tVi4EcnA0gw06iiTeT4lQX8RlrZc9wrxrZu9L9nm6eZnmEUJxO4+4yzbuhQ3u 8dkDjLIcUcoJsEI/Pr9bFjuNniCPM+AHUSG5jXxSNrLaqsoJfDmGQCh2I8goyCB60K14v4cX 9kGZBMxlPkZZ+C9trGf+CGiKBZ6NEU6wUvr1Wpf8N/q8eFN84LrFEcllYmllClWNkqrEVtlC ePnyxXde/rO13kxEG6x4EgqIbEzzUDMYOdXlvFWsBH51VE0IpUIP+Wo6aE+mHacVGhf3NN7F 0/KNZjChmjtFkWcGl9JZzLi9JVeNi+r7c1iDXewPMQehF/b4ooivym5Q6zQDXLztDFm1mORT Nh9U34mk2Q3OjGYAYWJ6cXjGtNHRRSECDYSKBOGUAENydntaBn8zRfC5wTtrx222oQFthPXI ebGob9Xomu8Mpjlny/rMHZcN99cexECdKA0+vmKIGuKifLWpIpRWLZvNWTNOoR9Pt5otz2IA WY1UIUKN7K71MzvJ8RFKyszoe9cd+RVA9siudAa2AZiTuMeVQZBnnKzZHsz5qKH6U9RQB1nm 2VVBxLtNCQvqnLclX1PA0/6ri2NUUmXHrnu099CF1TGxKDcLghfDrWOWaDFwWqBabIhKPOU3 9HJXyFltQhHUyjLK107ffvm02XrJ77M=
Organization: Vocal Technologies
To: "payload@ietf.org" <payload@ietf.org>
Message-ID: <ea97ab6a-861b-6e30-c595-8e5ade5c9308@vocal.com>
X-ASG-Orig-Subj: Re: [payload] WGLC on draft-ietf-payload-tetra-00
Date: Tue, 10 Jul 2018 14:03:51 -0400
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.9.0
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="------------F743E1CD0B6779527F497BA2"
Content-Language: en-US
X-Barracuda-Connect: host105.olm1.com[72.236.255.15]
X-Barracuda-Start-Time: 1531245831
X-Barracuda-Encrypted: ECDHE-RSA-AES256-GCM-SHA384
X-Barracuda-URL: https://72.236.255.32:443/cgi-mod/mark.cgi
X-Virus-Scanned: by bsmtpd at olm1.com
X-Barracuda-BRTS-Status: 1
X-Barracuda-Spam-Score: 0.00
X-Barracuda-Spam-Status: No, SCORE=0.00 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=5.0 tests=HTML_MESSAGE
X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.53363 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- 0.00 HTML_MESSAGE BODY: HTML included in message
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/XJ1FXZ5Tz9KzS4GDw-Kx1pYGGGw>
Subject: Re: [payload] WGLC on draft-ietf-payload-tetra-00
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/payload/>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Jul 2018 18:04:31 -0000

Review Comments for:
 
 RTP Payload Format for the TETRA Audio Codec
 draft-ietf-payload-tetra-00
 
July 10, 2018 by
David Satterlee
David.Satterlee@vocal.com
 
The draft appears to be generally well specified and useful.
 
Review Comments:


1.  Introduction

It would seem to me that:
TErrestial Trunked Radio (TETRA)

should be have a capital A (in multiple places in the document):
TErrestial Trunked RAdio (TETRA)



4.3.7.  S: Spare (7 bits)

The use of 'those' appears awkward. There are a number of these cases throughout the document:

<< Those bits are reserved for future use and set to "0" currently.
>> The S bits...



5.  Payload example

<< The following example shows how a first and a consecutive 30 ms frame
   is combined into a single 60ms RTP packet. 

>> The use of consecutive is awkward - maybe subsequent, or second consecutive?



8.1.  Offer/Answer Considerations

<< o  Integer multiples of 30ms SHALL be used for ptime.  It is
      recommended to use packet size of 60ms.  Even if there is no good
      reason why not doing so, there is no need that ptime and maxptime
      parameters are negotiated symmetrically.

>> o  Integer multiples of 30ms SHALL be used for ptime.  It is
      recommended to use packet size of 60ms.  There is no need that
      the ptime and maxptime parameters be negotiated symmetrically.



8.2.  Declarative SDP Considerations

<< For declarative media, the "ptime" and "maxptime" parameter specifies
   the possible variants used by the sender.

>> For declarative media, the "ptime" and "maxptime" parameter specify
   the possible variants used by the sender.