toggle quoted messageShow quoted text
After the first set of patches were merged I didn't start on the single
value representation, partly because I was waiting to see if there would be
any more feedback on the mailing list, and partly because of work related
changes and winter holiday.
Since there was no feedback against the change, I will start working on it
and hope to have the patch ready by next week.
Sorry if the delay on my part caused any inconveniences,
On Wed, Jan 4, 2017 at 8:09 PM, Marcus Shawcroft <marcus.shawcroft(a)gmail.com
Re-send with updated email address for Bogdan...
On 4 January 2017 at 18:05, Marcus Shawcroft <marcus.shawcroft(a)gmail.com>
On 29 November 2016 at 21:54, Davidoaia, Bogdan Mwe might as well have the same type for all the channels. As you also
If we want to define specific value types for each channel type, then
pointed out, using double is not really necessary for most drivers and the
ones that need it can just return the values converted to _INT_PLUS_MICRO.
implicit type _INT_PLUS_MICRO (although this should be done as a separate
As such, we could eliminate the type field altogether and have the
step, as it will affect the apps that use the type field).
then I can start working on it next Wednesday as I am currently away with
If you think this is ok and nobody has an objection to this change,
only access to email from time to time.
Following up on this old thread. Looking in the current tree we have
no drivers using SENSOR_VALUE_TYPE_DOUBLE, the sensor.h API does still
define SENSOR_VALUE_TYPE_DOUBLE and the type field used to distinguish
the last two remaining value types.
Is the intention that we continue along the path of moving to a single
If so, are you intending to present further patches, if not, then I am
happy to post patches to continue this work.