From patchwork Fri Jan 16 11:24:37 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Akihiro TSUKADA X-Patchwork-Id: 27923 Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from ) id 1YC51P-0006jb-5c; Fri, 16 Jan 2015 12:24:59 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.72/mailfrontend-6) with esmtp id 1YC51M-0007mG-5U; Fri, 16 Jan 2015 12:24:58 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753982AbbAPLYy (ORCPT + 1 other); Fri, 16 Jan 2015 06:24:54 -0500 Received: from mail-pa0-f44.google.com ([209.85.220.44]:47890 "EHLO mail-pa0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752558AbbAPLYx (ORCPT ); Fri, 16 Jan 2015 06:24:53 -0500 Received: by mail-pa0-f44.google.com with SMTP id et14so23759508pad.3 for ; Fri, 16 Jan 2015 03:24:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=czyEXLMrZqMdJoa0RJZrjf9Im5pxnQnjykCk1ISGZVU=; b=VqtyUZ5DTGrh9UwTPPBbC5/O4K8xQTNZYfaex1TQuk+OqR/HWMXun41PpeMW/fNSvF CYv7b9IEr2bycsbMGDVXL/qc5X8FEi/ExV/91/YXbIVlcxcA41nRbNc7oBXn8F+fcE4r lgU+lIVUQrx+wZTg9cF8ArRFUeC4J19P8ouV+HSOmN55qfpXXuQMn02Q2VxbPbxP3y5A uLqRxupAtCNqeQ/OHyQFyypVPNLf0ms8VPLlRx6BxWQtn0/0c9VTiVHVS2ATLSd4AwoP NMniangIcw+x/gAjNk0lOnK4Pe5WZRJCgJ8N9AR7HTDCK+axs4POLV2mFmr/MZj6/GoR CATQ== X-Received: by 10.68.131.39 with SMTP id oj7mr21623612pbb.145.1421407493516; Fri, 16 Jan 2015 03:24:53 -0800 (PST) Received: from seabird.localdomain.localdomain (softbank219203027033.bbtec.net. [219.203.27.33]) by mx.google.com with ESMTPSA id c9sm1060415pdm.51.2015.01.16.03.24.51 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 16 Jan 2015 03:24:52 -0800 (PST) From: tskd08@gmail.com To: linux-media@vger.kernel.org Cc: m.chehab@samsung.com, Akihiro Tsukada Subject: [PATCH v3 1/4] dvb: qm1d1c0042: use dvb-core i2c binding model template Date: Fri, 16 Jan 2015 20:24:37 +0900 Message-Id: <1421407480-9122-2-git-send-email-tskd08@gmail.com> X-Mailer: git-send-email 2.2.2 In-Reply-To: <1421407480-9122-1-git-send-email-tskd08@gmail.com> References: <1421407480-9122-1-git-send-email-tskd08@gmail.com> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2015.1.16.50023 X-PMX-Spam: Gauge=IIIIIIIII, Probability=9%, Report=' FORGED_FROM_GMAIL 0.1, MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODY_SIZE_4000_4999 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, DKIM_SIGNATURE 0, NO_REAL_NAME 0, REFERENCES 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_URI_IN_BODY 0, __FRAUD_BODY_WEBMAIL 0, __FRAUD_WEBMAIL 0, __FRAUD_WEBMAIL_FROM 0, __FROM_GMAIL 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MULTIPLE_RCPTS_CC_X2 0, __PHISH_SPEAR_STRUCTURE_1 0, __REFERENCES 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' From: Akihiro Tsukada Signed-off-by: Akihiro Tsukada --- drivers/media/tuners/qm1d1c0042.c | 60 +++++++++++++-------------------------- drivers/media/tuners/qm1d1c0042.h | 2 -- 2 files changed, 19 insertions(+), 43 deletions(-) diff --git a/drivers/media/tuners/qm1d1c0042.c b/drivers/media/tuners/qm1d1c0042.c index 18bc745..b6d637d 100644 --- a/drivers/media/tuners/qm1d1c0042.c +++ b/drivers/media/tuners/qm1d1c0042.c @@ -29,6 +29,7 @@ #include #include +#include "dvb_i2c.h" #include "qm1d1c0042.h" #define QM1D1C0042_NUM_REGS 0x20 @@ -55,11 +56,6 @@ struct qm1d1c0042_state { u8 regs[QM1D1C0042_NUM_REGS]; }; -static struct qm1d1c0042_state *cfg_to_state(struct qm1d1c0042_config *c) -{ - return container_of(c, struct qm1d1c0042_state, cfg); -} - static int reg_write(struct qm1d1c0042_state *state, u8 reg, u8 val) { u8 wbuf[2] = { reg, val }; @@ -106,10 +102,12 @@ static int qm1d1c0042_set_srch_mode(struct qm1d1c0042_state *state, bool fast) return reg_write(state, 0x03, state->regs[0x03]); } -static int qm1d1c0042_wakeup(struct qm1d1c0042_state *state) +static int qm1d1c0042_wakeup(struct dvb_frontend *fe) { + struct qm1d1c0042_state *state; int ret; + state = fe->tuner_priv; state->regs[0x01] |= 1 << 3; /* BB_Reg_enable */ state->regs[0x01] &= (~(1 << 0)) & 0xff; /* NORMAL (wake-up) */ state->regs[0x05] &= (~(1 << 3)) & 0xff; /* pfd_rst NORMAL */ @@ -119,7 +117,7 @@ static int qm1d1c0042_wakeup(struct qm1d1c0042_state *state) if (ret < 0) dev_warn(&state->i2c->dev, "(%s) failed. [adap%d-fe%d]\n", - __func__, state->cfg.fe->dvb->num, state->cfg.fe->id); + __func__, fe->dvb->num, fe->id); return ret; } @@ -133,9 +131,6 @@ static int qm1d1c0042_set_config(struct dvb_frontend *fe, void *priv_cfg) state = fe->tuner_priv; cfg = priv_cfg; - if (cfg->fe) - state->cfg.fe = cfg->fe; - if (cfg->xtal_freq != QM1D1C0042_CFG_XTAL_DFLT) dev_warn(&state->i2c->dev, "(%s) changing xtal_freq not supported. ", __func__); @@ -359,7 +354,7 @@ static int qm1d1c0042_init(struct dvb_frontend *fe) goto failed; } - ret = qm1d1c0042_wakeup(state); + ret = qm1d1c0042_wakeup(fe); if (ret < 0) goto failed; @@ -395,33 +390,18 @@ static const struct dvb_tuner_ops qm1d1c0042_ops = { static int qm1d1c0042_probe(struct i2c_client *client, const struct i2c_device_id *id) { - struct qm1d1c0042_state *state; - struct qm1d1c0042_config *cfg; + struct dvb_i2c_tuner_config *cfg; struct dvb_frontend *fe; - - state = kzalloc(sizeof(*state), GFP_KERNEL); - if (!state) - return -ENOMEM; - state->i2c = client; + struct qm1d1c0042_state *state; cfg = client->dev.platform_data; fe = cfg->fe; - fe->tuner_priv = state; - qm1d1c0042_set_config(fe, cfg); - memcpy(&fe->ops.tuner_ops, &qm1d1c0042_ops, sizeof(qm1d1c0042_ops)); + state = fe->tuner_priv; + state->i2c = client; - i2c_set_clientdata(client, &state->cfg); - dev_info(&client->dev, "Sharp QM1D1C0042 attached.\n"); - return 0; -} + qm1d1c0042_set_config(fe, (void *)cfg->devcfg.priv_cfg); -static int qm1d1c0042_remove(struct i2c_client *client) -{ - struct qm1d1c0042_state *state; - - state = cfg_to_state(i2c_get_clientdata(client)); - state->cfg.fe->tuner_priv = NULL; - kfree(state); + dev_info(&client->dev, "Sharp QM1D1C0042 attached.\n"); return 0; } @@ -430,18 +410,16 @@ static const struct i2c_device_id qm1d1c0042_id[] = { {"qm1d1c0042", 0}, {} }; -MODULE_DEVICE_TABLE(i2c, qm1d1c0042_id); -static struct i2c_driver qm1d1c0042_driver = { - .driver = { - .name = "qm1d1c0042", - }, - .probe = qm1d1c0042_probe, - .remove = qm1d1c0042_remove, - .id_table = qm1d1c0042_id, +static const struct dvb_i2c_module_param qm1d1c0042_param = { + .ops.tuner_ops = &qm1d1c0042_ops, + .priv_probe = qm1d1c0042_probe, + + .priv_size = sizeof(struct qm1d1c0042_state), + .is_tuner = true, }; -module_i2c_driver(qm1d1c0042_driver); +DEFINE_DVB_I2C_MODULE(qm1d1c0042, qm1d1c0042_id, qm1d1c0042_param); MODULE_DESCRIPTION("Sharp QM1D1C0042 tuner"); MODULE_AUTHOR("Akihiro TSUKADA"); diff --git a/drivers/media/tuners/qm1d1c0042.h b/drivers/media/tuners/qm1d1c0042.h index 4f5c188..043787e 100644 --- a/drivers/media/tuners/qm1d1c0042.h +++ b/drivers/media/tuners/qm1d1c0042.h @@ -21,8 +21,6 @@ struct qm1d1c0042_config { - struct dvb_frontend *fe; - u32 xtal_freq; /* [kHz] */ /* currently ignored */ bool lpf; /* enable LPF */ bool fast_srch; /* enable fast search mode, no LPF */