lkml.org 
[lkml]   [2017]   [Feb]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.2 015/126] [media] mb86a20s: fix the locking logic
    3.2.85-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

    commit dafb65fb98d85d8e78405e82c83e81975e5d5480 upstream.

    On this frontend, it takes a while to start output normal
    TS data. That only happens on state S9. On S8, the TS output
    is enabled, but it is not reliable enough.

    However, the zigzag loop is too fast to let it sync.

    As, on practical tests, the zigzag software loop doesn't
    seem to be helping, but just slowing down the tuning, let's
    switch to hardware algorithm, as the tuners used on such
    devices are capable of work with frequency drifts without
    any help from software.

    Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
    [bwh: Backported to 3.2: adjust filename]
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    drivers/media/dvb/frontends/mb86a20s.c | 12 +++++++++++-
    1 file changed, 11 insertions(+), 1 deletion(-)

    --- a/drivers/media/dvb/frontends/mb86a20s.c
    +++ b/drivers/media/dvb/frontends/mb86a20s.c
    @@ -477,7 +477,11 @@ static int mb86a20s_read_status(struct d
    if (val >= 7)
    *status |= FE_HAS_SYNC;

    - if (val >= 8) /* Maybe 9? */
    + /*
    + * Actually, on state S8, it starts receiving TS, but the TS
    + * output is only on normal state after the transition to S9.
    + */
    + if (val >= 9)
    *status |= FE_HAS_LOCK;

    dprintk("val = %d, status = 0x%02x\n", val, *status);
    @@ -562,6 +566,11 @@ static void mb86a20s_release(struct dvb_
    kfree(state);
    }

    +static int mb86a20s_get_frontend_algo(struct dvb_frontend *fe)
    +{
    + return DVBFE_ALGO_HW;
    +}
    +
    static struct dvb_frontend_ops mb86a20s_ops;

    struct dvb_frontend *mb86a20s_attach(const struct mb86a20s_config *config,
    @@ -632,6 +641,7 @@ static struct dvb_frontend_ops mb86a20s_
    .read_status = mb86a20s_read_status,
    .read_signal_strength = mb86a20s_read_signal_strength,
    .tune = mb86a20s_tune,
    + .get_frontend_algo = mb86a20s_get_frontend_algo,
    };

    MODULE_DESCRIPTION("DVB Frontend module for Fujitsu mb86A20s hardware");
    \
     
     \ /
      Last update: 2017-02-16 01:40    [W:2.146 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site