From patchwork Tue Jan 8 00:25:48 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mauro Carvalho Chehab X-Patchwork-Id: 1943151 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 8C0A0DF230 for ; Tue, 8 Jan 2013 00:26:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756183Ab3AHA0Y (ORCPT ); Mon, 7 Jan 2013 19:26:24 -0500 Received: from mx1.redhat.com ([209.132.183.28]:63192 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754816Ab3AHA0X (ORCPT ); Mon, 7 Jan 2013 19:26:23 -0500 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r080QN5l015109 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 7 Jan 2013 19:26:23 -0500 Received: from pedra (vpn1-7-20.gru2.redhat.com [10.97.7.20]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r080QLNI014450 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Mon, 7 Jan 2013 19:26:23 -0500 Received: from v4l by pedra with local (Exim 4.76) (envelope-from ) id 1TsN0q-0001HU-NV; Mon, 07 Jan 2013 22:25:52 -0200 From: Mauro Carvalho Chehab Cc: Mauro Carvalho Chehab , Linux Media Mailing List Subject: [PATCH RFCv9 2/4] dvb: the core logic to handle the DVBv5 QoS properties Date: Mon, 7 Jan 2013 22:25:48 -0200 Message-Id: <1357604750-772-3-git-send-email-mchehab@redhat.com> In-Reply-To: <1357604750-772-1-git-send-email-mchehab@redhat.com> References: <1357604750-772-1-git-send-email-mchehab@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.23 To: unlisted-recipients:; (no To-header on input) Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Add the logic to poll, reset counters and report the QoS stats to the end user. The idea is that the core will periodically poll the frontend for the stats. The frontend may return -EBUSY, if the previous collect didn't finish, or it may fill the cached data. The value returned to the end user is always the cached data. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-core/dvb_frontend.c | 56 +++++++++++++++++++++++++++++++++++ drivers/media/dvb-core/dvb_frontend.h | 12 ++++++++ 2 files changed, 68 insertions(+) diff --git a/drivers/media/dvb-core/dvb_frontend.c b/drivers/media/dvb-core/dvb_frontend.c index dd35fa9..66e066b 100644 --- a/drivers/media/dvb-core/dvb_frontend.c +++ b/drivers/media/dvb-core/dvb_frontend.c @@ -260,6 +260,9 @@ static int dvb_frontend_get_event(struct dvb_frontend *fe, return ret; } + if (fe->ops.get_qos_stats) + fe->ops.get_qos_stats(fe); + mutex_lock(&events->mtx); *event = events->events[events->eventr]; events->eventr = (events->eventr + 1) % MAX_EVENT; @@ -1053,6 +1056,15 @@ static struct dtv_cmds_h dtv_cmds[DTV_MAX_COMMAND + 1] = { _DTV_CMD(DTV_ATSCMH_SCCC_CODE_MODE_B, 0, 0), _DTV_CMD(DTV_ATSCMH_SCCC_CODE_MODE_C, 0, 0), _DTV_CMD(DTV_ATSCMH_SCCC_CODE_MODE_D, 0, 0), + + /* Statistics API */ + _DTV_CMD(DTV_QOS_ENUM, 0, 0), + _DTV_CMD(DTV_QOS_SIGNAL_STRENGTH, 0, 0), + _DTV_CMD(DTV_QOS_CNR, 0, 0), + _DTV_CMD(DTV_QOS_BIT_ERROR_COUNT, 0, 0), + _DTV_CMD(DTV_QOS_TOTAL_BITS_COUNT, 0, 0), + _DTV_CMD(DTV_QOS_ERROR_BLOCK_COUNT, 0, 0), + _DTV_CMD(DTV_QOS_TOTAL_BLOCKS_COUNT, 0, 0), }; static void dtv_property_dump(struct dvb_frontend *fe, struct dtv_property *tvp) @@ -1443,6 +1455,25 @@ static int dtv_property_process_get(struct dvb_frontend *fe, tvp->u.data = c->lna; break; + /* Fill quality measures */ + case DTV_QOS_SIGNAL_STRENGTH: + tvp->u.st = c->strength; + break; + case DTV_QOS_CNR: + tvp->u.st = c->cnr; + break; + case DTV_QOS_BIT_ERROR_COUNT: + tvp->u.st = c->bit_error; + break; + case DTV_QOS_TOTAL_BITS_COUNT: + tvp->u.st = c->bit_count; + break; + case DTV_QOS_ERROR_BLOCK_COUNT: + tvp->u.st = c->block_error; + break; + case DTV_QOS_TOTAL_BLOCKS_COUNT: + tvp->u.st = c->block_count; + break; default: return -EINVAL; } @@ -1646,6 +1677,26 @@ static int set_delivery_system(struct dvb_frontend *fe, u32 desired_system) return 0; } +static int reset_qos_counters(struct dvb_frontend *fe) +{ + struct dtv_frontend_properties *c = &fe->dtv_property_cache; + + /* Reset QoS cache */ + + memset (&c->strength, 0, sizeof(c->strength)); + memset (&c->cnr, 0, sizeof(c->cnr)); + memset (&c->bit_error, 0, sizeof(c->bit_error)); + memset (&c->bit_count, 0, sizeof(c->bit_count)); + memset (&c->block_error, 0, sizeof(c->block_error)); + memset (&c->block_count, 0, sizeof(c->block_count)); + + /* Call frontend reset counter method, if available */ + if (fe->ops.reset_qos_counters) + return fe->ops.reset_qos_counters(fe); + + return 0; +} + static int dtv_property_process_set(struct dvb_frontend *fe, struct dtv_property *tvp, struct file *file) @@ -1705,6 +1756,8 @@ static int dtv_property_process_set(struct dvb_frontend *fe, break; case DTV_DELIVERY_SYSTEM: r = set_delivery_system(fe, tvp->u.data); + if (r >= 0) + reset_qos_counters(fe); break; case DTV_VOLTAGE: c->voltage = tvp->u.data; @@ -2305,6 +2358,9 @@ static int dvb_frontend_ioctl_legacy(struct file *file, if (err) break; err = dtv_set_frontend(fe); + if (err >= 0) + reset_qos_counters(fe); + break; case FE_GET_EVENT: err = dvb_frontend_get_event (fe, parg, file->f_flags); diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index 97112cd..7bb49f1 100644 --- a/drivers/media/dvb-core/dvb_frontend.h +++ b/drivers/media/dvb-core/dvb_frontend.h @@ -315,6 +315,10 @@ struct dvb_frontend_ops { int (*set_property)(struct dvb_frontend* fe, struct dtv_property* tvp); int (*get_property)(struct dvb_frontend* fe, struct dtv_property* tvp); + + /* QoS statistics callbacks */ + int (*get_qos_stats)(struct dvb_frontend *fe); + int (*reset_qos_counters)(struct dvb_frontend *fe); }; #ifdef __DVB_CORE__ @@ -393,6 +397,14 @@ struct dtv_frontend_properties { u8 atscmh_sccc_code_mode_d; u32 lna; + + /* QoS statistics data */ + struct dtv_fe_stats strength; + struct dtv_fe_stats cnr; + struct dtv_fe_stats bit_error; + struct dtv_fe_stats bit_count; + struct dtv_fe_stats block_error; + struct dtv_fe_stats block_count; }; struct dvb_frontend {