]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
[media] DocBook: fix FE_READ_STATUS argument description
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Thu, 28 May 2015 17:43:06 +0000 (14:43 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Sat, 30 May 2015 12:43:09 +0000 (09:43 -0300)
What's written there about the arguments for this ioctl
is bogus: it doesn't return an enum (or a typedef)
for enum fe_status. Instead, it returns a bitmask with the
values defined by enum fe_status.

Also, the size of the integer returned is not 16 bits, but,
instead, sizeof(fe_status_t), e. g. sizeof(enum), with is
arch-dependent.

This should of course be fixed, but this should be done on
a separate patch.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Documentation/DocBook/media/dvb/fe-read-status.xml

index 9c1810ae920dd94f9427d25d8d2f06d5ee31006e..bbd0b5bb6b127367d60558e28c8f1f8db337a3e0 100644 (file)
@@ -16,7 +16,7 @@
        <funcdef>int <function>ioctl</function></funcdef>
        <paramdef>int <parameter>fd</parameter></paramdef>
        <paramdef>int <parameter>request</parameter></paramdef>
-       <paramdef>&fe-status; *<parameter>argp</parameter></paramdef>
+       <paramdef>unsigned int *<parameter>status</parameter></paramdef>
       </funcprototype>
     </funcsynopsis>
   </refsynopsisdiv>
        </listitem>
       </varlistentry>
       <varlistentry>
-       <term><parameter>argp</parameter></term>
+       <term><parameter>status</parameter></term>
        <listitem>
-           <para>pointer to &fe-status;</para>
+           <para>pointer to a bitmask integer filled with the values defined by
+               &fe-status;.</para>
        </listitem>
       </varlistentry>
     </variablelist>
     <para>All DVB frontend devices support the
 <constant>FE_READ_STATUS</constant> ioctl. It is used to check about the
 locking status of the frontend after being tuned. The ioctl takes a
-pointer to a 16-bits number where the status will be written.
-&return-value-dvb;.
+pointer to an integer where the status will be written.
 </para>
+<para>NOTE: the size of status is actually sizeof(enum fe_status), with varies
+    according with the architecture. This needs to be fixed in the future.</para>
+&return-value-dvb;
 </refsect1>
 
 <section id="fe-status-t">
-<title>enum fe_status</title>
+<title>int fe_status</title>
 
-<para>The enum fe_status is used to indicate the current state
-    and/or state changes of the frontend hardware.</para>
+<para>The fe_status parameter is used to indicate the current state
+    and/or state changes of the frontend hardware. It is produced using
+    the &fe-status; values on a bitmask</para>
 
 <table pgwide="1" frame="none" id="fe-status">
     <title>enum fe_status</title>