From patchwork Tue Apr 11 19:15:18 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Stultz X-Patchwork-Id: 97278 Delivered-To: patches@linaro.org Received: by 10.140.109.52 with SMTP id k49csp101799qgf; Tue, 11 Apr 2017 12:15:28 -0700 (PDT) X-Received: by 10.84.215.218 with SMTP id g26mr20319238plj.22.1491938128700; Tue, 11 Apr 2017 12:15:28 -0700 (PDT) Return-Path: Received: from mail-pf0-x235.google.com (mail-pf0-x235.google.com. [2607:f8b0:400e:c00::235]) by mx.google.com with ESMTPS id e4si5717947pfl.271.2017.04.11.12.15.28 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 11 Apr 2017 12:15:28 -0700 (PDT) Received-SPF: pass (google.com: domain of john.stultz@linaro.org designates 2607:f8b0:400e:c00::235 as permitted sender) client-ip=2607:f8b0:400e:c00::235; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: domain of john.stultz@linaro.org designates 2607:f8b0:400e:c00::235 as permitted sender) smtp.mailfrom=john.stultz@linaro.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: by mail-pf0-x235.google.com with SMTP id c198so2804528pfc.1 for ; Tue, 11 Apr 2017 12:15:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=1cUU8kW3Ju4B2Bkg1mGM4fYbERu1+M3jJxg7oitcL3s=; b=jmaplgQuUkXLmIgMQynt7LIJ8C9U8kuHfUs/0qgdGW7GjsltdDlwb30s9jv64j+lyY hVURRGr0tfS5HmACkoaGcQ185Bv1cjFaPzf3DitM6Ea60t590o6NHTiqjPH0rHplerm7 eV3/YH0yaj9r6yqFvWLfSVh7+AeupsX13asFw= 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=1cUU8kW3Ju4B2Bkg1mGM4fYbERu1+M3jJxg7oitcL3s=; b=J7m5MPmOsSviPoGicUikHH3SN4TawxqqbA0ajq0xDN/ceaJR89AgX9a4vQPYdcGx77 /QAMgCECWoCKoosMSmoEyfStzfW9C8yN++0xu/emnzdGUKHsCkr0ctxVlQOadGvdMCB5 GwTFBivt2u1Kzjhqx+JtiRagY622cTbff8JDuhwgnKAK4zB6z/N3epGs4upAYrN4oN5G i15T4PrrJJqyuGvRRwLxv+GsjlTOoVzRoYm9GY4lNEr3RByxDU4+1m13t7saumuRGMIf dxwLnSaBqnYYDaCRyqER7LrBg8pdffspi5ks1TH4XLtprlhsT7opO8hhAlSZmL8IXxkS k5kg== X-Gm-Message-State: AN3rC/4F6wluL+NQCD4jHB0s7SDj5xnynIOwuHq91d2PCUyCBrRG4mXCEgMqlp/l/wJB13memgc= X-Received: by 10.84.193.3 with SMTP id e3mr32836093pld.71.1491938128437; Tue, 11 Apr 2017 12:15:28 -0700 (PDT) Return-Path: Received: from localhost.localdomain ([2601:1c2:1002:83f0:4e72:b9ff:fe99:466a]) by smtp.gmail.com with ESMTPSA id y184sm32033726pgd.41.2017.04.11.12.15.26 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 11 Apr 2017 12:15:27 -0700 (PDT) From: John Stultz To: lkml Cc: John Stultz , Zhangfei Gao , Liam Girdwood , Mark Brown , Jaroslav Kysela , Wei Xu , Rob Herring , Andy Green , Dave Long , Guodong Xu Subject: [PATCH 3/3] ASoC: hisilicon: Use devm_snd_soc_register_component Date: Tue, 11 Apr 2017 12:15:18 -0700 Message-Id: <1491938118-19108-3-git-send-email-john.stultz@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1491938118-19108-1-git-send-email-john.stultz@linaro.org> References: <1491938118-19108-1-git-send-email-john.stultz@linaro.org> Per feedback from Mark Brown, this patch updates the hi6210-i2s driver to use devm_snd_soc_register_component which simplifies the logic a bit. Cc: Zhangfei Gao Cc: Liam Girdwood Cc: Mark Brown Cc: Jaroslav Kysela Cc: Wei Xu Cc: Rob Herring Cc: Andy Green Cc: Dave Long Cc: Guodong Xu Signed-off-by: John Stultz --- sound/soc/hisilicon/hi6210-i2s.c | 18 ++---------------- 1 file changed, 2 insertions(+), 16 deletions(-) -- 2.7.4 diff --git a/sound/soc/hisilicon/hi6210-i2s.c b/sound/soc/hisilicon/hi6210-i2s.c index 27553dc..45163e5 100644 --- a/sound/soc/hisilicon/hi6210-i2s.c +++ b/sound/soc/hisilicon/hi6210-i2s.c @@ -591,22 +591,9 @@ static int hi6210_i2s_probe(struct platform_device *pdev) if (ret) return ret; - ret = snd_soc_register_component(&pdev->dev, &hi6210_i2s_i2s_comp, + ret = devm_snd_soc_register_component(&pdev->dev, &hi6210_i2s_i2s_comp, &i2s->dai, 1); - if (ret) { - dev_err(&pdev->dev, "Failed to register dai\n"); - return ret; - } - - return 0; -} - -static int hi6210_i2s_remove(struct platform_device *pdev) -{ - snd_soc_unregister_component(&pdev->dev); - dev_set_drvdata(&pdev->dev, NULL); - - return 0; + return ret; } static const struct of_device_id hi6210_i2s_dt_ids[] = { @@ -618,7 +605,6 @@ MODULE_DEVICE_TABLE(of, hi6210_i2s_dt_ids); static struct platform_driver hi6210_i2s_driver = { .probe = hi6210_i2s_probe, - .remove = hi6210_i2s_remove, .driver = { .name = "hi6210_i2s", .of_match_table = hi6210_i2s_dt_ids,