Skip to content

Commit 4c12ada

Browse files
dhowellsmchehab
authored andcommitted
[media] dvb: Document FE_SCALE_DECIBEL units consistently
In comments and in the documentation, the units of properties marked with the FE_SCALE_DECIBEL scale are specified in terms of 1/1000 dB or 0.0001 dB. This is inconsistent, however, as 1/1000 is 0.001, not 0.0001. Note that the v4l-utils divide the value by 1000 for the signal strength suggesting that the 1/1000 is correct. Settle on millidecibels, ie. 1/1000dB or 0.001dB. Signed-off-by: David Howells <dhowells@redhat.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
1 parent 24d795d commit 4c12ada

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

Documentation/DocBook/media/dvb/dvbproperty.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -953,7 +953,7 @@ enum fe_interleaving {
953953
<para>Possible scales for this metric are:</para>
954954
<itemizedlist mark='bullet'>
955955
<listitem><para><constant>FE_SCALE_NOT_AVAILABLE</constant> - it failed to measure it, or the measurement was not complete yet.</para></listitem>
956-
<listitem><para><constant>FE_SCALE_DECIBEL</constant> - signal strength is in 0.0001 dBm units, power measured in miliwatts. This value is generally negative.</para></listitem>
956+
<listitem><para><constant>FE_SCALE_DECIBEL</constant> - signal strength is in 0.001 dBm units, power measured in miliwatts. This value is generally negative.</para></listitem>
957957
<listitem><para><constant>FE_SCALE_RELATIVE</constant> - The frontend provides a 0% to 100% measurement for power (actually, 0 to 65535).</para></listitem>
958958
</itemizedlist>
959959
</section>
@@ -963,7 +963,7 @@ enum fe_interleaving {
963963
<para>Possible scales for this metric are:</para>
964964
<itemizedlist mark='bullet'>
965965
<listitem><para><constant>FE_SCALE_NOT_AVAILABLE</constant> - it failed to measure it, or the measurement was not complete yet.</para></listitem>
966-
<listitem><para><constant>FE_SCALE_DECIBEL</constant> - Signal/Noise ratio is in 0.0001 dB units.</para></listitem>
966+
<listitem><para><constant>FE_SCALE_DECIBEL</constant> - Signal/Noise ratio is in 0.001 dB units.</para></listitem>
967967
<listitem><para><constant>FE_SCALE_RELATIVE</constant> - The frontend provides a 0% to 100% measurement for Signal/Noise (actually, 0 to 65535).</para></listitem>
968968
</itemizedlist>
969969
</section>

include/uapi/linux/dvb/frontend.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ struct dtv_cmds_h {
467467
* @FE_SCALE_NOT_AVAILABLE: That QoS measure is not available. That
468468
* could indicate a temporary or a permanent
469469
* condition.
470-
* @FE_SCALE_DECIBEL: The scale is measured in 0.0001 dB steps, typically
470+
* @FE_SCALE_DECIBEL: The scale is measured in 0.001 dB steps, typically
471471
* used on signal measures.
472472
* @FE_SCALE_RELATIVE: The scale is a relative percentual measure,
473473
* ranging from 0 (0%) to 0xffff (100%).
@@ -516,7 +516,7 @@ struct dtv_stats {
516516
__u8 scale; /* enum fecap_scale_params type */
517517
union {
518518
__u64 uvalue; /* for counters and relative scales */
519-
__s64 svalue; /* for 0.0001 dB measures */
519+
__s64 svalue; /* for 0.001 dB measures */
520520
};
521521
} __attribute__ ((packed));
522522

0 commit comments

Comments
 (0)