From patchwork Tue Mar 13 22:18:05 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Scheller X-Patchwork-Id: 10281049 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 0ED7B6038F for ; Tue, 13 Mar 2018 22:18:16 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id E80F52855D for ; Tue, 13 Mar 2018 22:18:15 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id DCE7A28563; Tue, 13 Mar 2018 22:18:15 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 55CA42855D for ; Tue, 13 Mar 2018 22:18:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752636AbeCMWSO (ORCPT ); Tue, 13 Mar 2018 18:18:14 -0400 Received: from mail-wr0-f195.google.com ([209.85.128.195]:39447 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752100AbeCMWSM (ORCPT ); Tue, 13 Mar 2018 18:18:12 -0400 Received: by mail-wr0-f195.google.com with SMTP id k3so2476301wrg.6 for ; Tue, 13 Mar 2018 15:18:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=/u/OCfL4L1vlIfnhI2OW9ZN4Zn1IyJgepLV+9RWdEus=; b=mchyoPdMGqa9c8SE6cYhbmV2fOR+Mn8Tjb6dCD/MS+RowWlKsIavNlOAeVflnl5JpC jVj9VrbMHiM5oMcglYVhuEYUFlFCKHLqqja+acpeHakcN+PFkBOqghXEQRPwJPuM7ySu kaWHUAGLr6OBoSS8IG/DEeWmn1wRVrOCCH4Jiv97TsCDua9ORFVj2QO0QpgZyfSfppmo u6j31+PnPLtHqCCVgOodLv8yidcJn/Z2I/GR3asS8lUs8LYNWTURdnEjxooaDQ1NHRlq /rW4LWIur0knoCBVA2L0tHxVC4/VYyEUUfnatyHsdCzH1eVsKhidswDCHx95iCzALW0n lB3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=/u/OCfL4L1vlIfnhI2OW9ZN4Zn1IyJgepLV+9RWdEus=; b=g8IDMrTZFfNupHP6g0Luv/7/syW5kATn+Wb49eX17BRr1zT3OoU0173svknmdw3aHR 3wDu9HsroTJkWTD80Lal4+mePBc+zg8NH2XSWJADIpx0Hk4xt+Qn78/eSzhoyMstxTFn 4UaG8vCugF3WBq4FjgHuLaTIJvCiMEHnBPphEIs8MEAy/PHiJM6/+uQOIP6tgBvQqT+n ote9Yu4/e9TN908kGr3fR5ul99OsYcpPu7ho02HWC06yjaTCo3/7ePtUgp55MN7b4kbb LaHm+BaPBHpUYyiHparjYhMS1zghXInOFxV7yMGHNdPljYnMaZf4Ekoc5EcFLwZLHRlX /UDg== X-Gm-Message-State: AElRT7FX4RibiiI2xwg64p2AEduX7R+5k/EkP2e5rX7F7jRRlHhN0hoS F2Vu4YS9wjeNESpOuf0ibXsvUA== X-Google-Smtp-Source: AG47ELubHFN5sZKxEVVjGyMuI4uG0OG3AYoUc7+n9aK+XV2nBw/ZIKiCtosFUmv+Jm9ls88jSJf3XQ== X-Received: by 10.223.166.135 with SMTP id t7mr1873138wrc.260.1520979490843; Tue, 13 Mar 2018 15:18:10 -0700 (PDT) Received: from dvbdev.wuest.de (ip-84-118-193-200.unity-media.net. [84.118.193.200]) by smtp.gmail.com with ESMTPSA id k11sm1205732wre.81.2018.03.13.15.18.09 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 13 Mar 2018 15:18:10 -0700 (PDT) From: Daniel Scheller To: linux-media@vger.kernel.org, mchehab@kernel.org, mchehab@s-opensource.com Cc: rascobie@slingshot.co.nz Subject: [PATCH v3 3/3] [media] dvb-frontends/stv0910: more detailed reporting in get_frontend() Date: Tue, 13 Mar 2018 23:18:05 +0100 Message-Id: <20180313221805.26818-4-d.scheller.oss@gmail.com> X-Mailer: git-send-email 2.16.1 In-Reply-To: <20180313221805.26818-1-d.scheller.oss@gmail.com> References: <20180313221805.26818-1-d.scheller.oss@gmail.com> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Daniel Scheller The first two missing FECs in the modcod2fec fe_code_rate table were 1/4 and 1/3. Add them as they're now defined by the API. Also, report the currently used S2 rolloff factor in get_frontend(). For cosmetic reasons, also change all feroll_off occurences to fe_rolloff. In addition, Richard Scobie suggested to report the currently active delivery system aswell, so add this while at it. Signed-off-by: Daniel Scheller --- v2 to v3: - Squashed all stv0910 get_frontend() reporting into a single commit Please take note of some additional things in the cover letter. drivers/media/dvb-frontends/stv0910.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/drivers/media/dvb-frontends/stv0910.c b/drivers/media/dvb-frontends/stv0910.c index 52355c14fd64..e12bc87de87b 100644 --- a/drivers/media/dvb-frontends/stv0910.c +++ b/drivers/media/dvb-frontends/stv0910.c @@ -113,7 +113,7 @@ struct stv { enum fe_stv0910_mod_cod mod_cod; enum dvbs2_fectype fectype; u32 pilots; - enum fe_stv0910_roll_off feroll_off; + enum fe_stv0910_roll_off fe_rolloff; int is_standard_broadcast; int is_vcm; @@ -541,7 +541,7 @@ static int get_signal_parameters(struct stv *state) } state->is_vcm = 0; state->is_standard_broadcast = 1; - state->feroll_off = FE_SAT_35; + state->fe_rolloff = FE_SAT_35; } return 0; } @@ -1300,14 +1300,14 @@ static int manage_matype_info(struct stv *state) read_regs(state, RSTV0910_P2_MATSTR1 + state->regoff, bbheader, 2); - state->feroll_off = + state->fe_rolloff = (enum fe_stv0910_roll_off)(bbheader[0] & 0x03); state->is_vcm = (bbheader[0] & 0x10) == 0; state->is_standard_broadcast = (bbheader[0] & 0xFC) == 0xF0; } else if (state->receive_mode == RCVMODE_DVBS) { state->is_vcm = 0; state->is_standard_broadcast = 1; - state->feroll_off = FE_SAT_35; + state->fe_rolloff = FE_SAT_35; } return 0; } @@ -1562,7 +1562,7 @@ static int get_frontend(struct dvb_frontend *fe, APSK_32, }; const enum fe_code_rate modcod2fec[0x20] = { - FEC_NONE, FEC_NONE, FEC_NONE, FEC_2_5, + FEC_NONE, FEC_1_4, FEC_1_3, FEC_2_5, FEC_1_2, FEC_3_5, FEC_2_3, FEC_3_4, FEC_4_5, FEC_5_6, FEC_8_9, FEC_9_10, FEC_3_5, FEC_2_3, FEC_3_4, FEC_5_6, @@ -1571,11 +1571,16 @@ static int get_frontend(struct dvb_frontend *fe, FEC_3_4, FEC_4_5, FEC_5_6, FEC_8_9, FEC_9_10 }; + const enum fe_rolloff ro2ro[4] = { + ROLLOFF_35, ROLLOFF_25, ROLLOFF_20, ROLLOFF_15, + }; read_reg(state, RSTV0910_P2_DMDMODCOD + state->regoff, &tmp); mc = ((tmp & 0x7c) >> 2); p->pilot = (tmp & 0x01) ? PILOT_ON : PILOT_OFF; p->modulation = modcod2mod[mc]; p->fec_inner = modcod2fec[mc]; + p->rolloff = ro2ro[state->fe_rolloff]; + p->delivery_system = SYS_DVBS2; } else if (state->receive_mode == RCVMODE_DVBS) { read_reg(state, RSTV0910_P2_VITCURPUN + state->regoff, &tmp); switch (tmp & 0x1F) { @@ -1599,6 +1604,7 @@ static int get_frontend(struct dvb_frontend *fe, break; } p->rolloff = ROLLOFF_35; + p->delivery_system = SYS_DVBS; } if (state->receive_mode != RCVMODE_NONE) {