dmm: vc870: render POWER_FACTOR as such and not as frequency

[libsigrok.git] / src / dmm / vc870.c

diff --git a/src/dmm/vc870.c b/src/dmm/vc870.c
index d14b84152c63240f9ab29ddf7f91b43db458889c..125f4712d5fb23179bc09d363292c4940ece5b47 100644 (file)
--- a/src/dmm/vc870.c
+++ b/src/dmm/vc870.c
@@ -46,14 +46,20 @@ static const float factors[][8] = {
        {1e-4,  0,     0,     0,    0,    0,    0,    0},    /* Diode */
        {1e-3,  1e-2,  1e-1,  1,    1e1,  1e2,  1e3,  1e4},  /* Frequency */
        {1e-2,  0,     0,     0,    0,    0,    0,    0},    /* Loop current */
+       /*
+        * Note: Measurements showed that AC and DC differ
+        * in the factors used, although docs say they should
+        * be the same.
+        */
        {1e-8,  1e-7,  0,     0,    0,    0,    0,    0},    /* DCµA */
-       {1e-8,  1e-7,  0,     0,    0,    0,    0,    0},    /* ACµA */
+       {1e-7,  1e-6,  0,     0,    0,    0,    0,    0},    /* ACµA */
        {1e-6,  1e-5,  0,     0,    0,    0,    0,    0},    /* DCmA */
-       {1e-6,  1e-5,  0,     0,    0,    0,    0,    0},    /* ACmA */
+       {1e-5,  1e-4,  0,     0,    0,    0,    0,    0},    /* ACmA */
        {1e-3,  0,     0,     0,    0,    0,    0,    0},    /* DCA */
+       /* TODO: Verify factor for ACA */
        {1e-3,  0,     0,     0,    0,    0,    0,    0},    /* ACA */
        {1e-1,  0,     0,     0,    0,    0,    0,    0},    /* Act+apparent power */
-       {1e-1,  0,     0,     0,    0,    0,    0,    0},    /* Power factor / freq */
+       {1e-3,  0,     0,     0,    0,    0,    0,    0},    /* Power factor / freq */
        {1e-1,  0,     0,     0,    0,    0,    0,    0},    /* V eff + A eff */
 };
 
@@ -331,13 +337,6 @@ static void handle_flags(struct sr_datafeed_analog_old *analog,
                analog->mq = SR_MQ_POWER;
                analog->unit = SR_UNIT_WATT;
        }
-       if (info->is_power_factor_freq) {
-               /* TODO: Handle power factor. */
-               // analog->mq = SR_MQ_POWER_FACTOR;
-               // analog->unit = SR_UNIT_UNITLESS;
-               analog->mq = SR_MQ_FREQUENCY;
-               analog->unit = SR_UNIT_HERTZ;
-       }
        if (info->is_power_apparent_power) {
                analog->mq = SR_MQ_POWER;
                analog->unit = SR_UNIT_WATT;
@@ -345,6 +344,13 @@ static void handle_flags(struct sr_datafeed_analog_old *analog,
                // analog->mq = SR_MQ_APPARENT_POWER;
                // analog->unit = SR_UNIT_VOLT_AMPERE;
        }
+       if (info->is_power_factor_freq) {
+               analog->mq = SR_MQ_POWER_FACTOR;
+               analog->unit = SR_UNIT_UNITLESS;
+               /* TODO: Handle frequency. */
+               // analog->mq = SR_MQ_FREQUENCY;
+               // analog->unit = SR_UNIT_HERTZ;
+       }
 
        /* Measurement related flags */
        if (info->is_ac)