From patchwork Wed Mar 14 17:28:48 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Akihiro TSUKADA X-Patchwork-Id: 47939 Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ewADI-000882-Lh; Wed, 14 Mar 2018 17:29:21 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751302AbeCNR3S (ORCPT + 1 other); Wed, 14 Mar 2018 13:29:18 -0400 Received: from mail-pf0-f196.google.com ([209.85.192.196]:39461 "EHLO mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750779AbeCNR3R (ORCPT ); Wed, 14 Mar 2018 13:29:17 -0400 Received: by mail-pf0-f196.google.com with SMTP id u5so1698618pfh.6 for ; Wed, 14 Mar 2018 10:29:16 -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; bh=xcsOPdHLFlQuti2cF+AqjWXbGjjD5bd44B/0TL2MgEc=; b=uMcNpma2InaDbIXqIXHSjJNcr8rS4McOWTRJdmtMcXxHVjjYhq4azLQfpMVWYmZF/g 5v8FKcokmDjnxs4LBmWqgwEsdRh8S0nXWNBNWrmEryyXWS1Y8gGNAW1KbbXX6KnhOZ85 46Gxt8gbEbDt/xVcCQCWgrGEm5ktjwe9TOBugH89B8BL2rE0+CJhsAFg40b14eEuKDVa CUvld2X5AsJE29iYdTeohnUGVGRPmzAt/9A2gM2XBtGGyQRu2Fp9BciDd/7JfU+K+Qjx FUmJbREGA5MxN3+UyyiEOAgKXypXUIycm/trALNkiTiSkJfIou69oLkDh5V+nKhju+Gi 93sQ== 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; bh=xcsOPdHLFlQuti2cF+AqjWXbGjjD5bd44B/0TL2MgEc=; b=V0eVtW+8i8I8n7d40diGOVB/4gw0yyHAxlLwusKhiqDlGyC0hYAm4RnuayjtDeO4Wi BcDOYGm6AM2m0UABhLZ8LV1EvnT6RTBnbCwSkj57jo/2Wx/6SnLS5u6niN5OdauAQZyt rOZfSFvyya2xiwuL+kVddfT83gSx4LJApPVevB4Mf0uM3LFYbV/coQQm1mH913Cixj8n W8VHQrc32wtkYmHm8kiSdHj0lSQH8Y23nOrVegHLmhiHs0Hvo/MOy8PwtLipy1+ZebpY KcFrM4zGGGbhZyOp5QDxwwBJ0bcQABQv2+TCrdLXZWIfXcAQhka6UKnU3QIkYVbZBH26 Oucw== X-Gm-Message-State: AElRT7GASjvoW6FeoJN21ly8MyL/tqumG+daufnmJrlpveWspjUtFy6E ujZcv/vgpm8O4eCePKDle4p6Jw== X-Google-Smtp-Source: AG47ELvbdF/aq/POMZnoK2fvQjUKW/40sK95qOvQ+rCF0wR/8bMcydMA7QtKE/TKpnyCsgIxZidmMQ== X-Received: by 10.99.105.7 with SMTP id e7mr4332601pgc.193.1521048556105; Wed, 14 Mar 2018 10:29:16 -0700 (PDT) Received: from localhost.localdomain (softbank219203027033.bbtec.net. [219.203.27.33]) by smtp.googlemail.com with ESMTPSA id s78sm6390482pfa.161.2018.03.14.10.29.13 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 14 Mar 2018 10:29:15 -0700 (PDT) From: tskd08@gmail.com To: linux-media@vger.kernel.org Cc: mchehab@s-opensource.com, Akihiro Tsukada Subject: [PATCH v4] media/dvb: earth-pt3: use the new i2c binding helper Date: Thu, 15 Mar 2018 02:28:48 +0900 Message-Id: <20180314172848.2317-1-tskd08@gmail.com> X-Mailer: git-send-email 2.16.2 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org From: Akihiro Tsukada This patch slightly simplifies the binding code by introducing commit 8f569c0b4e6b ("media: dvb-core: add helper functions for I2C binding"). Signed-off-by: Akihiro Tsukada --- Changes since v3 - earth-pt3: replaced the old helper functions (proposed by patch #27922) with the new one (commit:8f569c0b). - withdrew the following v3 patches: #27923 (tuners/qm1d1c0042), #27924 (tuners/mxl301rf), #27925 (dvb-frontends/tc90522). The new helper functions do not require changes to / apply to those demod/tuner i2c drivers. drivers/media/pci/pt3/pt3.c | 58 +++++++++++++++++---------------------------- 1 file changed, 22 insertions(+), 36 deletions(-) diff --git a/drivers/media/pci/pt3/pt3.c b/drivers/media/pci/pt3/pt3.c index da74828805b..eef4105919b 100644 --- a/drivers/media/pci/pt3/pt3.c +++ b/drivers/media/pci/pt3/pt3.c @@ -376,25 +376,23 @@ static int pt3_fe_init(struct pt3_board *pt3) static int pt3_attach_fe(struct pt3_board *pt3, int i) { - struct i2c_board_info info; + const struct i2c_board_info *info; struct tc90522_config cfg; struct i2c_client *cl; struct dvb_adapter *dvb_adap; int ret; - info = adap_conf[i].demod_info; + info = &adap_conf[i].demod_info; cfg = adap_conf[i].demod_cfg; cfg.tuner_i2c = NULL; - info.platform_data = &cfg; ret = -ENODEV; request_module("tc90522"); - cl = i2c_new_device(&pt3->i2c_adap, &info); - if (!cl || !cl->dev.driver) + cl = dvb_module_probe("tc90522", info->type, &pt3->i2c_adap, + info->addr, &cfg); + if (!cl) return -ENODEV; pt3->adaps[i]->i2c_demod = cl; - if (!try_module_get(cl->dev.driver->owner)) - goto err_demod_i2c_unregister_device; if (!strncmp(cl->name, TC90522_I2C_DEV_SAT, strlen(TC90522_I2C_DEV_SAT))) { @@ -402,41 +400,33 @@ static int pt3_attach_fe(struct pt3_board *pt3, int i) tcfg = adap_conf[i].tuner_cfg.qm1d1c0042; tcfg.fe = cfg.fe; - info = adap_conf[i].tuner_info; - info.platform_data = &tcfg; - request_module("qm1d1c0042"); - cl = i2c_new_device(cfg.tuner_i2c, &info); + info = &adap_conf[i].tuner_info; + cl = dvb_module_probe("qm1d1c0042", info->type, cfg.tuner_i2c, + info->addr, &tcfg); } else { struct mxl301rf_config tcfg; tcfg = adap_conf[i].tuner_cfg.mxl301rf; tcfg.fe = cfg.fe; - info = adap_conf[i].tuner_info; - info.platform_data = &tcfg; - request_module("mxl301rf"); - cl = i2c_new_device(cfg.tuner_i2c, &info); + info = &adap_conf[i].tuner_info; + cl = dvb_module_probe("mxl301rf", info->type, cfg.tuner_i2c, + info->addr, &tcfg); } - if (!cl || !cl->dev.driver) - goto err_demod_module_put; + if (!cl) + goto err_demod_module_release; pt3->adaps[i]->i2c_tuner = cl; - if (!try_module_get(cl->dev.driver->owner)) - goto err_tuner_i2c_unregister_device; dvb_adap = &pt3->adaps[one_adapter ? 0 : i]->dvb_adap; ret = dvb_register_frontend(dvb_adap, cfg.fe); if (ret < 0) - goto err_tuner_module_put; + goto err_tuner_module_release; pt3->adaps[i]->fe = cfg.fe; return 0; -err_tuner_module_put: - module_put(pt3->adaps[i]->i2c_tuner->dev.driver->owner); -err_tuner_i2c_unregister_device: - i2c_unregister_device(pt3->adaps[i]->i2c_tuner); -err_demod_module_put: - module_put(pt3->adaps[i]->i2c_demod->dev.driver->owner); -err_demod_i2c_unregister_device: - i2c_unregister_device(pt3->adaps[i]->i2c_demod); +err_tuner_module_release: + dvb_module_release(pt3->adaps[i]->i2c_tuner); +err_demod_module_release: + dvb_module_release(pt3->adaps[i]->i2c_demod); return ret; } @@ -630,14 +620,10 @@ static void pt3_cleanup_adapter(struct pt3_board *pt3, int index) adap->fe->callback = NULL; if (adap->fe->frontend_priv) dvb_unregister_frontend(adap->fe); - if (adap->i2c_tuner) { - module_put(adap->i2c_tuner->dev.driver->owner); - i2c_unregister_device(adap->i2c_tuner); - } - if (adap->i2c_demod) { - module_put(adap->i2c_demod->dev.driver->owner); - i2c_unregister_device(adap->i2c_demod); - } + if (adap->i2c_tuner) + dvb_module_release(adap->i2c_tuner); + if (adap->i2c_demod) + dvb_module_release(adap->i2c_demod); } pt3_free_dmabuf(adap); dvb_dmxdev_release(&adap->dmxdev);