From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 85CD8FD21E1 for ; Mon, 30 Jul 2018 06:11:58 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2E960208AD for ; Mon, 30 Jul 2018 06:11:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2E960208AD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726589AbeG3HpR (ORCPT ); Mon, 30 Jul 2018 03:45:17 -0400 Received: from mga03.intel.com ([134.134.136.65]:61327 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726087AbeG3HpR (ORCPT ); Mon, 30 Jul 2018 03:45:17 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 Jul 2018 23:11:54 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,421,1526367600"; d="scan'208";a="78923549" Received: from ahunter-desktop.fi.intel.com (HELO [10.237.72.168]) ([10.237.72.168]) by orsmga002.jf.intel.com with ESMTP; 29 Jul 2018 23:11:52 -0700 Subject: Re: [PATCH v2] mmc: host: iproc: Add ACPI support to IPROC SDHCI To: Srinath Mannam Cc: Ulf Hansson , Ray Jui , Scott Branden , Vikram Prakash , BCM Kernel Feedback , linux-mmc@vger.kernel.org, Linux Kernel Mailing List References: <1532794529-30137-1-git-send-email-srinath.mannam@broadcom.com> <201807290127.DcHKtABy%fengguang.wu@intel.com> From: Adrian Hunter Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki, Business Identity Code: 0357606 - 4, Domiciled in Helsinki Message-ID: <0c1215a2-b8b3-e315-71e1-04501d02ba42@intel.com> Date: Mon, 30 Jul 2018 09:10:11 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 30/07/18 09:02, Srinath Mannam wrote: > Hi Adrian Hunter, > > Auto build error came to this patch, because it is depend on the > patch "mmc: sdhci-pltfm: Convert DT properties to generic device properties" > you have given to me. > > Can I send my patch along with your patch in a single patch series? Yes, that is normally how it is done. > Please advice. > > Regards, > Srinath. > > > On Sat, Jul 28, 2018 at 11:02 PM, kbuild test robot wrote: >> Hi Srinath, >> >> Thank you for the patch! Yet something to improve: >> >> [auto build test ERROR on ulf.hansson-mmc/next] >> [also build test ERROR on v4.18-rc6 next-20180727] >> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] >> >> url: https://github.com/0day-ci/linux/commits/Srinath-Mannam/mmc-host-iproc-Add-ACPI-support-to-IPROC-SDHCI/20180729-003026 >> base: git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git next >> config: ia64-allmodconfig (attached as .config) >> compiler: ia64-linux-gcc (GCC) 8.1.0 >> reproduce: >> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross >> chmod +x ~/bin/make.cross >> # save the attached .config to linux build tree >> GCC_VERSION=8.1.0 make.cross ARCH=ia64 >> >> All errors (new ones prefixed by >>): >> >> drivers/mmc/host/sdhci-iproc.c: In function 'sdhci_iproc_probe': >>>> drivers/mmc/host/sdhci-iproc.c:304:2: error: implicit declaration of function 'sdhci_get_property'; did you mean 'sdhci_get_of_property'? [-Werror=implicit-function-declaration] >> sdhci_get_property(pdev); >> ^~~~~~~~~~~~~~~~~~ >> sdhci_get_of_property >> cc1: some warnings being treated as errors >> >> vim +304 drivers/mmc/host/sdhci-iproc.c >> >> 276 >> 277 static int sdhci_iproc_probe(struct platform_device *pdev) >> 278 { >> 279 struct device *dev = &pdev->dev; >> 280 const struct sdhci_iproc_data *iproc_data = NULL; >> 281 struct sdhci_host *host; >> 282 struct sdhci_iproc_host *iproc_host; >> 283 struct sdhci_pltfm_host *pltfm_host; >> 284 int ret; >> 285 >> 286 if (dev->of_node) >> 287 iproc_data = of_device_get_match_data(dev); >> 288 else if (has_acpi_companion(dev)) >> 289 iproc_data = acpi_device_get_match_data(dev); >> 290 >> 291 if (!iproc_data) >> 292 return -ENODEV; >> 293 >> 294 host = sdhci_pltfm_init(pdev, iproc_data->pdata, sizeof(*iproc_host)); >> 295 if (IS_ERR(host)) >> 296 return PTR_ERR(host); >> 297 >> 298 pltfm_host = sdhci_priv(host); >> 299 iproc_host = sdhci_pltfm_priv(pltfm_host); >> 300 >> 301 iproc_host->data = iproc_data; >> 302 >> 303 mmc_of_parse(host->mmc); >> > 304 sdhci_get_property(pdev); >> 305 >> 306 host->mmc->caps |= iproc_host->data->mmc_caps; >> 307 >> 308 if (dev->of_node) { >> 309 pltfm_host->clk = devm_clk_get(dev, NULL); >> 310 if (IS_ERR(pltfm_host->clk)) { >> 311 ret = PTR_ERR(pltfm_host->clk); >> 312 goto err; >> 313 } >> 314 ret = clk_prepare_enable(pltfm_host->clk); >> 315 if (ret) { >> 316 dev_err(dev, "failed to enable host clk\n"); >> 317 goto err; >> 318 } >> 319 } >> 320 >> 321 if (iproc_host->data->pdata->quirks & SDHCI_QUIRK_MISSING_CAPS) { >> 322 host->caps = iproc_host->data->caps; >> 323 host->caps1 = iproc_host->data->caps1; >> 324 } >> 325 >> 326 ret = sdhci_add_host(host); >> 327 if (ret) >> 328 goto err_clk; >> 329 >> 330 return 0; >> 331 >> 332 err_clk: >> 333 if (dev->of_node) >> 334 clk_disable_unprepare(pltfm_host->clk); >> 335 err: >> 336 sdhci_pltfm_free(pdev); >> 337 return ret; >> 338 } >> 339 >> >> --- >> 0-DAY kernel test infrastructure Open Source Technology Center >> https://lists.01.org/pipermail/kbuild-all Intel Corporation >