This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/v4l-utils.git tree:

Subject: libdvbv5: don't override a parameter when setting lna
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Wed Apr 23 16:00:58 2014 -0300

We should not fill the parameters before setting LNA, as, otherwise,
the first property (typically frequency) would be overriden.

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 lib/libdvbv5/dvb-fe.c |   16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)

---

http://git.linuxtv.org/v4l-utils.git?a=commitdiff;h=ae8d14f7df5cc9a96b89b31af43ff5115bd17fdf

diff --git a/lib/libdvbv5/dvb-fe.c b/lib/libdvbv5/dvb-fe.c
index 0841546..013edd6 100644
--- a/lib/libdvbv5/dvb-fe.c
+++ b/lib/libdvbv5/dvb-fe.c
@@ -629,15 +629,11 @@ int dvb_fe_set_parms(struct dvb_v5_fe_parms *parms)
        struct dvb_frontend_parameters v3_parms;
        uint32_t bw;
 
-       if (dvb_fe_is_satellite(tmp_parms.current_sys))
-               dvb_sat_set_parms(&tmp_parms);
-
-       /* Filter out any user DTV_foo property such as DTV_POLARIZATION */
-       tmp_parms.n_props = dvb_copy_fe_props(tmp_parms.dvb_prop, 
tmp_parms.n_props, tmp_parms.dvb_prop);
-
        if (parms->lna != LNA_AUTO && !parms->legacy_fe) {
+               struct dvb_v5_fe_parms tmp_lna_parms;
+
                memset(&prop, 0, sizeof(prop));
-               prop.props = tmp_parms.dvb_prop;
+               prop.props = tmp_lna_parms.dvb_prop;
 
                prop.props[0].cmd = DTV_LNA;
                prop.props[0].u.data = parms->lna;
@@ -649,6 +645,12 @@ int dvb_fe_set_parms(struct dvb_v5_fe_parms *parms)
                        dvb_logdbg("LNA is %s", parms->lna ? "ON" : "OFF");
        }
 
+       if (dvb_fe_is_satellite(tmp_parms.current_sys))
+               dvb_sat_set_parms(&tmp_parms);
+
+       /* Filter out any user DTV_foo property such as DTV_POLARIZATION */
+       tmp_parms.n_props = dvb_copy_fe_props(tmp_parms.dvb_prop, 
tmp_parms.n_props, tmp_parms.dvb_prop);
+
        memset(&prop, 0, sizeof(prop));
        prop.props = tmp_parms.dvb_prop;
        prop.num = tmp_parms.n_props;

_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to