Linux Media Controller development
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: unlisted-recipients:; (no To-header on input)@canuck.infradead.org
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: [PATCH v4 08/47] [media] mt2266: use DVBv5 parameters for set_params()
Date: Sat, 24 Dec 2011 13:50:13 -0200	[thread overview]
Message-ID: <1324741852-26138-9-git-send-email-mchehab@redhat.com> (raw)
In-Reply-To: <1324741852-26138-8-git-send-email-mchehab@redhat.com>

Instead of using DVBv3 parameters, rely on DVBv5 parameters to
set the tuner.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
---
 drivers/media/common/tuners/mt2266.c |   25 ++++++++++++++-----------
 1 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/drivers/media/common/tuners/mt2266.c b/drivers/media/common/tuners/mt2266.c
index 25a8ea3..dd883d7 100644
--- a/drivers/media/common/tuners/mt2266.c
+++ b/drivers/media/common/tuners/mt2266.c
@@ -124,6 +124,7 @@ static u8 mt2266_vhf[] = { 0x1d, 0xfe, 0x00, 0x00, 0xb4, 0x03, 0xa5, 0xa5,
 
 static int mt2266_set_params(struct dvb_frontend *fe, struct dvb_frontend_parameters *params)
 {
+	struct dtv_frontend_properties *c = &fe->dtv_property_cache;
 	struct mt2266_priv *priv;
 	int ret=0;
 	u32 freq;
@@ -135,30 +136,32 @@ static int mt2266_set_params(struct dvb_frontend *fe, struct dvb_frontend_parame
 
 	priv = fe->tuner_priv;
 
-	freq = params->frequency / 1000; // Hz -> kHz
+	freq = priv->frequency / 1000; /* Hz -> kHz */
 	if (freq < 470000 && freq > 230000)
 		return -EINVAL; /* Gap between VHF and UHF bands */
-	priv->bandwidth = (fe->ops.info.type == FE_OFDM) ? params->u.ofdm.bandwidth : 0;
-	priv->frequency = freq * 1000;
 
+	priv->frequency = c->frequency;
 	tune = 2 * freq * (8192/16) / (FREF/16);
 	band = (freq < 300000) ? MT2266_VHF : MT2266_UHF;
 	if (band == MT2266_VHF)
 		tune *= 2;
 
-	switch (params->u.ofdm.bandwidth) {
-	case BANDWIDTH_6_MHZ:
+	switch (c->bandwidth_hz) {
+	case 6000000:
 		mt2266_writeregs(priv, mt2266_init_6mhz,
 				 sizeof(mt2266_init_6mhz));
+		priv->bandwidth = BANDWIDTH_6_MHZ;
 		break;
-	case BANDWIDTH_7_MHZ:
-		mt2266_writeregs(priv, mt2266_init_7mhz,
-				 sizeof(mt2266_init_7mhz));
-		break;
-	case BANDWIDTH_8_MHZ:
-	default:
+	case 8000000:
 		mt2266_writeregs(priv, mt2266_init_8mhz,
 				 sizeof(mt2266_init_8mhz));
+		priv->bandwidth = BANDWIDTH_8_MHZ;
+		break;
+	case 7000000:
+	default:
+		mt2266_writeregs(priv, mt2266_init_7mhz,
+				 sizeof(mt2266_init_7mhz));
+		priv->bandwidth = BANDWIDTH_7_MHZ;
 		break;
 	}
 
-- 
1.7.8.352.g876a6


  reply	other threads:[~2011-12-24 15:51 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-24 15:50 [PATCH v4 00/47] DVB tuners: remove dvb_frontend_parameters from set_params() Mauro Carvalho Chehab
2011-12-24 15:50 ` [PATCH v4 01/47] [media] dvb: replace SYS_DVBC_ANNEX_AC by the right delsys Mauro Carvalho Chehab
2011-12-24 15:50   ` [PATCH v4 02/47] [media] dvb_core: estimate bw for all non-terrestial systems Mauro Carvalho Chehab
2011-12-24 15:50     ` [PATCH v4 03/47] [media] qt1010: remove fake implementaion of get_bandwidth() Mauro Carvalho Chehab
2011-12-24 15:50       ` [PATCH v4 04/47] [media] mt2060: " Mauro Carvalho Chehab
2011-12-24 15:50         ` [PATCH v4 05/47] [media] mt2031: " Mauro Carvalho Chehab
2011-12-24 15:50           ` [PATCH v4 06/47] [media] mc44s803: use DVBv5 parameters on set_params() Mauro Carvalho Chehab
2011-12-24 15:50             ` [PATCH v4 07/47] [media] max2165: " Mauro Carvalho Chehab
2011-12-24 15:50               ` Mauro Carvalho Chehab [this message]
2011-12-24 15:50                 ` [PATCH v4 09/47] [media] mxl5005s: " Mauro Carvalho Chehab
2011-12-24 15:50                   ` [PATCH v4 10/47] [media] mxl5005s: fix: don't discard bandwidth changes Mauro Carvalho Chehab
2011-12-24 15:50                     ` [PATCH v4 11/47] [media] mxl5007t: use DVBv5 parameters on set_params() Mauro Carvalho Chehab
2011-12-24 15:50                       ` [PATCH v4 12/47] [media] tda18218: " Mauro Carvalho Chehab
2011-12-24 15:50                         ` [PATCH v4 13/47] [media] tda18271: add support for QAM 7 MHz map Mauro Carvalho Chehab
2011-12-24 15:50                           ` [PATCH v4 14/47] [media] tda18271-fe: use DVBv5 parameters on set_params() Mauro Carvalho Chehab
2011-12-24 15:50                             ` [PATCH v4 15/47] [media] tda827x: " Mauro Carvalho Chehab
2011-12-24 15:50                               ` [PATCH v4 16/47] [media] tuner-xc2028: " Mauro Carvalho Chehab
2011-12-24 15:50                                 ` [PATCH v4 17/47] [media] xc4000: " Mauro Carvalho Chehab
2011-12-24 15:50                                   ` [PATCH v4 18/47] [media] cx24113: " Mauro Carvalho Chehab
2011-12-24 15:50                                     ` [PATCH v4 19/47] [media] zl10039: " Mauro Carvalho Chehab
2011-12-24 15:50                                       ` [PATCH v4 20/47] [media] av7110: " Mauro Carvalho Chehab
2011-12-24 15:50                                         ` [PATCH v4 21/47] [media] budget-ci: " Mauro Carvalho Chehab
2011-12-24 15:50                                           ` [PATCH v4 22/47] [media] budget-patch: " Mauro Carvalho Chehab
2011-12-24 15:50                                             ` [PATCH v4 23/47] [media] saa7134: " Mauro Carvalho Chehab
2011-12-24 15:50                                               ` [PATCH v4 24/47] [media] cx88: " Mauro Carvalho Chehab
2011-12-24 15:50                                                 ` [PATCH v4 25/47] [media] tua6100: " Mauro Carvalho Chehab
2011-12-24 15:50                                                   ` [PATCH v4 26/47] [media] itd1000: " Mauro Carvalho Chehab
2011-12-24 15:50                                                     ` [PATCH v4 27/47] [media] bsbe1, bsru6, tdh1: " Mauro Carvalho Chehab
2011-12-24 15:50                                                       ` [PATCH v4 28/47] [media] ix2505v: " Mauro Carvalho Chehab
2011-12-24 15:50                                                         ` [PATCH v4 29/47] [media] stb6000: " Mauro Carvalho Chehab
2011-12-24 15:50                                                           ` [PATCH v4 30/47] [media] tda826x: " Mauro Carvalho Chehab
2011-12-24 15:50                                                             ` [PATCH v4 31/47] [media] mxl111sf-tuner: " Mauro Carvalho Chehab
2011-12-24 15:50                                                               ` [PATCH v4 32/47] [media] mantis_vp1033: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                 ` [PATCH v4 33/47] [media] mantis_vp2033: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                   ` [PATCH v4 34/47] [media] mantis_vp2040: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                     ` [PATCH v4 35/47] [media] pluto2: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                       ` [PATCH v4 36/47] [media] dvb-ttusb-budget: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                         ` [PATCH v4 37/47] [media] tuner-simple: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                           ` [PATCH v4 38/47] [media] dvb-bt8xx: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                             ` [PATCH v4 39/47] [media] dvb-pll: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                               ` [PATCH v4 40/47] [media] zl10036: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                                 ` [PATCH v4 41/47] [media] dib0070: Remove unused dvb_frontend_parameters Mauro Carvalho Chehab
2011-12-24 15:50                                                                                   ` [PATCH v4 42/47] [media] cxusb: use DVBv5 parameters on set_params() Mauro Carvalho Chehab
2011-12-24 15:50                                                                                     ` [PATCH v4 43/47] [media] dib0700_devices: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                                       ` [PATCH v4 44/47] [media] budget-av: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                                         ` [PATCH v4 45/47] [media] budget: " Mauro Carvalho Chehab
2011-12-24 15:50                                                                                           ` [PATCH v4 46/47] [media] dvb: remove dvb_frontend_parameters from calc_regs() Mauro Carvalho Chehab
2011-12-24 15:50                                                                                             ` [PATCH v4 47/47] [media] tuners: remove dvb_frontend_parameters from set_params() Mauro Carvalho Chehab
2012-01-02 19:32                                 ` [PATCH v4 16/47] [media] tuner-xc2028: use DVBv5 parameters on set_params() Gianluca Gennari
2012-01-04 18:50                                   ` Mauro Carvalho Chehab
2012-01-04 19:10                                     ` Gianluca Gennari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1324741852-26138-9-git-send-email-mchehab@redhat.com \
    --to=mchehab@redhat.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox