Re: [PATCH 03/11] mmc: sdhci-msm: use sdhci_pltfm_init for private allocation

From: Jisheng Zhang <jszhang@marvell.com>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, ulf.hansson@linaro.org, swarren@wwwdotorg.org,
	lee@kernel.org, eric@anholt.net, michal.simek@xilinx.com,
	soren.brinkmann@xilinx.com, ludovic.desroches@atmel.com,
	srinivas.kandagatla@gmail.com, maxime.coquelin@st.com,
	patrice.chotard@st.com, thierry.reding@gmail.com,
	sebastian.hesselbarth@gmail.com, linux-mmc@vger.kernel.org,
	linux-rpi-kernel@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@stlinux.com,
	linux-tegra@vger.kernel.org
Subject: Re: [PATCH 03/11] mmc: sdhci-msm: use sdhci_pltfm_init for private allocation
Date: Wed, 6 Jan 2016 10:36:28 +0800	[thread overview]
Message-ID: <20160106103628.05a8f9c4@xhacker> (raw)
In-Reply-To: <201601052054.hzEzw4pe%fengguang.wu@intel.com>

On Tue, 5 Jan 2016 20:47:37 +0800 kbuild test robot wrote:

> Hi Jisheng,
> 
> [auto build test WARNING on v4.4-rc8]
> [also build test WARNING on next-20160105]
> [cannot apply to ulf.hansson-mmc/next]
> [if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
> 
> url:    https://github.com/0day-ci/linux/commits/Jisheng-Zhang/mmc-sdhci-bcm2835-use-sdhci_pltfm_init-for-private-allocation/20160105-190232
> config: arm64-allmodconfig (attached as .config)
> reproduce:
>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=arm64 
> 
> All warnings (new ones prefixed by >>):
> 
>    drivers/mmc/host/sdhci-msm.c: In function 'sdhci_msm_probe':
> >> drivers/mmc/host/sdhci-msm.c:440:32: warning: 'msm_host' is used uninitialized in this function [-Wuninitialized]  
>      host = sdhci_pltfm_init(pdev, &msm_host->sdhci_msm_pdata,

OOPS, I'll fix it.

Thanks a lot,
Jisheng

>                                    ^
> 
> vim +/msm_host +440 drivers/mmc/host/sdhci-msm.c
> 
>    424		.set_clock = sdhci_set_clock,
>    425		.set_bus_width = sdhci_set_bus_width,
>    426		.set_uhs_signaling = sdhci_set_uhs_signaling,
>    427	};
>    428	
>    429	static int sdhci_msm_probe(struct platform_device *pdev)
>    430	{
>    431		struct sdhci_host *host;
>    432		struct sdhci_pltfm_host *pltfm_host;
>    433		struct sdhci_msm_host *msm_host;
>    434		struct resource *core_memres;
>    435		int ret;
>    436		u16 host_version, core_minor;
>    437		u32 core_version, caps;
>    438		u8 core_major;
>    439	
>  > 440		host = sdhci_pltfm_init(pdev, &msm_host->sdhci_msm_pdata,  
>    441					sizeof(*msm_host));
>    442		if (IS_ERR(host))
>    443			return PTR_ERR(host);
>    444	
>    445		pltfm_host = sdhci_priv(host);
>    446		msm_host = sdhci_pltfm_priv(pltfm_host);
>    447		msm_host->sdhci_msm_pdata.ops = &sdhci_msm_ops;
>    448		msm_host->mmc = host->mmc;
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation