Em 16-08-2012 22:35, Antti Palosaari escreveu:
> Signed-off-by: Antti Palosaari <cr...@iki.fi>
> ---
>  Documentation/DocBook/media/dvb/dvbproperty.xml | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/Documentation/DocBook/media/dvb/dvbproperty.xml 
> b/Documentation/DocBook/media/dvb/dvbproperty.xml
> index d188be9..2dfa6a0 100644
> --- a/Documentation/DocBook/media/dvb/dvbproperty.xml
> +++ b/Documentation/DocBook/media/dvb/dvbproperty.xml
> @@ -827,6 +827,17 @@ enum fe_interleaving {
>  };
>       </programlisting>
>       </section>
> +     <section id="DTV-LNA">
> +     <title><constant>DTV_LNA</constant></title>
> +     <para>Low-noise amplifier.</para>
> +     <para>Hardware might offer controllable LNA which can be set manually
> +             using that parameter. Usually LNA could be found only from
> +             terrestrial devices if at all.</para>
> +     <para>Possible values: 0, 1, INT_MIN</para>

Hmm... INT_MIN... are you sure it is portable on all Linux compilers?

I don't like the idea on trusting on whatever C/C++/Java/... compiler (or some 
interpreter)
would define as "INT_MIN".

The better is to define a value for that, or, instead, to define something
at the API header file that won't cause troubles with 32 bits or 64 bits
userspace, like defining it as:

#define DVB_AUTO_LNA ((u32)~0)

> +     <para>0, LNA off</para>
> +     <para>1, LNA on</para>
> +     <para>INT_MIN, LNA auto</para>
> +     </section>
>  </section>
>       <section id="frontend-property-terrestrial-systems">
>       <title>Properties used on terrestrial delivery systems</title>
> @@ -847,6 +858,7 @@ enum fe_interleaving {
>                               <listitem><para><link 
> linkend="DTV-GUARD-INTERVAL"><constant>DTV_GUARD_INTERVAL</constant></link></para></listitem>
>                               <listitem><para><link 
> linkend="DTV-TRANSMISSION-MODE"><constant>DTV_TRANSMISSION_MODE</constant></link></para></listitem>
>                               <listitem><para><link 
> linkend="DTV-HIERARCHY"><constant>DTV_HIERARCHY</constant></link></para></listitem>
> +                             <listitem><para><link 
> linkend="DTV-LNA"><constant>DTV_LNA</constant></link></para></listitem>
>                       </itemizedlist>
>               </section>
>               <section id="dvbt2-params">
> @@ -870,6 +882,7 @@ enum fe_interleaving {
>                       <listitem><para><link 
> linkend="DTV-TRANSMISSION-MODE"><constant>DTV_TRANSMISSION_MODE</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-HIERARCHY"><constant>DTV_HIERARCHY</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-DVBT2-PLP-ID"><constant>DTV_DVBT2_PLP_ID</constant></link></para></listitem>
> +                     <listitem><para><link 
> linkend="DTV-LNA"><constant>DTV_LNA</constant></link></para></listitem>
>               </itemizedlist>
>               </section>
>               <section id="isdbt">
> @@ -981,6 +994,7 @@ enum fe_interleaving {
>                               <listitem><para><link 
> linkend="DTV-GUARD-INTERVAL"><constant>DTV_GUARD_INTERVAL</constant></link></para></listitem>
>                               <listitem><para><link 
> linkend="DTV-TRANSMISSION-MODE"><constant>DTV_TRANSMISSION_MODE</constant></link></para></listitem>
>                               <listitem><para><link 
> linkend="DTV-INTERLEAVING"><constant>DTV_INTERLEAVING</constant></link></para></listitem>
> +                             <listitem><para><link 
> linkend="DTV-LNA"><constant>DTV_LNA</constant></link></para></listitem>
>                       </itemizedlist>
>               </section>
>       </section>
> @@ -1001,6 +1015,7 @@ enum fe_interleaving {
>                       <listitem><para><link 
> linkend="DTV-INVERSION"><constant>DTV_INVERSION</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-SYMBOL-RATE"><constant>DTV_SYMBOL_RATE</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-INNER-FEC"><constant>DTV_INNER_FEC</constant></link></para></listitem>
> +                     <listitem><para><link 
> linkend="DTV-LNA"><constant>DTV_LNA</constant></link></para></listitem>
>               </itemizedlist>
>       </section>
>       <section id="dvbc-annex-b-params">
> @@ -1015,6 +1030,7 @@ enum fe_interleaving {
>                       <listitem><para><link 
> linkend="DTV-FREQUENCY"><constant>DTV_FREQUENCY</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-MODULATION"><constant>DTV_MODULATION</constant></link></para></listitem>
>                       <listitem><para><link 
> linkend="DTV-INVERSION"><constant>DTV_INVERSION</constant></link></para></listitem>
> +                     <listitem><para><link 
> linkend="DTV-LNA"><constant>DTV_LNA</constant></link></para></listitem>
>               </itemizedlist>
>       </section>
>       </section>
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to