Hi Anderson 2012/2/15 Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx>: > Hi Santiago, > > On Wed, Feb 15, 2012 at 4:50 AM, Santiago Carot <sancane@xxxxxxxxx> wrote: >> Hi Anderson, >> >> 2012/2/14 Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx>: >>> According to HTS 1.0, it is not mandatory to have Temperature Type >>> information (the Characteristic and the indication field are both >>> unconditionally optional). >>> >> >> I'm afraid there is an slight conceptual error in this patch. >> According to HTS 1.0, there are two *exclusive* methods to provide the >> temperature type to a Colector: either by using a characteristic type >> or providing it as a field within the measurement. This doesn't mean >> that the temperature characteristic or field type are both >> unconditionally optional because one of them *MUST* be used, but not >> both of them at the same time. So there will always be a method to get >> the temperature type either if it's a static value (characteristic >> type) or a non-static value (temperature field). Health Thermometer >> Service[3.2] ,[3.1] & Health Thermometer Profile [4.7] > > I've seen these sections. And they are exclusive, but none of them are > mandatory. The reason for this patch is that we have seen an LE > thermometer without any method to get its type (and the current > upstream code simply ignores its measurements). Looking at both > HTS/HTP and developer.bluetooth.org does not mention them as > "conditionally optional" (usually shown by a C<n> note on the service > table on service specification). > In any case, I don't think the best solution is to provide a value for Type for wich we don't have enough information to determine what it is, I think that in the case in wich Type is really optional, the value provided in the measurement should be optional too. Under this case it would be better not to provide anything in the MeasurementReceived callback instead of providing a invented (Body) value. Regards -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
