From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC PATCH v3.1 13/27] mmc: sdhci-uhs2: add set_power() to support vdd2
Date: Fri, 06 Nov 2020 16:11:21 +0800 [thread overview]
Message-ID: <202011061654.dGiuyglX-lkp@intel.com> (raw)
In-Reply-To: <20201106022726.19831-14-takahiro.akashi@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 5643 bytes --]
Hi AKASHI,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on linus/master]
[also build test WARNING on v5.10-rc2]
[cannot apply to v3.1 next-20201105]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/AKASHI-Takahiro/Add-support-UHS-II-for-GL9755/20201106-103058
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 521b619acdc8f1f5acdac15b84f81fd9515b2aff
config: x86_64-randconfig-a006-20201104 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 09ec07827b1128504457a93dee80b2ceee1af600)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/d95febed81c44315d5b757c9dca1b7dd006c0bd3
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review AKASHI-Takahiro/Add-support-UHS-II-for-GL9755/20201106-103058
git checkout d95febed81c44315d5b757c9dca1b7dd006c0bd3
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/mmc/host/sdhci-uhs2.c:113:6: warning: variable 'pwr' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
if (mode != MMC_POWER_OFF) {
^~~~~~~~~~~~~~~~~~~~~
drivers/mmc/host/sdhci-uhs2.c:122:19: note: uninitialized use occurs here
if (host->pwr == pwr)
^~~
drivers/mmc/host/sdhci-uhs2.c:113:2: note: remove the 'if' if its condition is always true
if (mode != MMC_POWER_OFF) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/host/sdhci-uhs2.c:105:8: note: initialize the variable 'pwr' to silence this warning
u8 pwr;
^
= '\0'
drivers/mmc/host/sdhci-uhs2.c:187:12: warning: unused function 'sdhci_uhs2_host_ops_init' [-Wunused-function]
static int sdhci_uhs2_host_ops_init(struct sdhci_host *host)
^
2 warnings generated.
vim +113 drivers/mmc/host/sdhci-uhs2.c
100
101 void sdhci_uhs2_set_power(struct sdhci_host *host, unsigned char mode,
102 unsigned short vdd)
103 {
104 struct mmc_host *mmc = host->mmc;
105 u8 pwr;
106
107 /* FIXME: check if flags & MMC_UHS2_SUPPORT? */
108 if (!(host->mmc->caps & MMC_CAP_UHS2)) {
109 sdhci_set_power(host, mode, vdd);
110 return;
111 }
112
> 113 if (mode != MMC_POWER_OFF) {
114 pwr = sdhci_get_vdd_value(vdd);
115 if (!pwr)
116 WARN(1, "%s: Invalid vdd %#x\n",
117 mmc_hostname(host->mmc), vdd);
118
119 pwr |= SDHCI_VDD2_POWER_180;
120 }
121
122 if (host->pwr == pwr)
123 return;
124 host ->pwr = pwr;
125
126 if (pwr == 0) {
127 sdhci_writeb(host, 0, SDHCI_POWER_CONTROL);
128
129 if (!IS_ERR(host->mmc->supply.vmmc))
130 mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, 0);
131 if (!IS_ERR_OR_NULL(host->mmc->supply.vmmc2))
132 mmc_regulator_set_ocr(mmc, mmc->supply.vmmc2, 0);
133
134 if (host->quirks2 & SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON)
135 sdhci_runtime_pm_bus_off(host);
136 } else {
137 if (!IS_ERR(host->mmc->supply.vmmc))
138 mmc_regulator_set_ocr(mmc, mmc->supply.vmmc, vdd);
139 if (!IS_ERR_OR_NULL(host->mmc->supply.vmmc2))
140 /* support 1.8v only for now */
141 mmc_regulator_set_ocr(mmc, mmc->supply.vmmc2,
142 fls(MMC_VDD2_165_195) - 1);
143
144 /*
145 * Spec says that we should clear the power reg before setting
146 * a new value. Some controllers don't seem to like this though.
147 */
148 if (!(host->quirks & SDHCI_QUIRK_SINGLE_POWER_WRITE))
149 sdhci_writeb(host, 0, SDHCI_POWER_CONTROL);
150
151 /*
152 * At least the Marvell CaFe chip gets confused if we set the
153 * voltage and set turn on power at the same time, so set the
154 * voltage first.
155 */
156 if (host->quirks & SDHCI_QUIRK_NO_SIMULT_VDD_AND_POWER)
157 sdhci_writeb(host, pwr, SDHCI_POWER_CONTROL);
158
159 /* vdd first */
160 pwr |= SDHCI_POWER_ON;
161 sdhci_writeb(host, pwr & 0xf, SDHCI_POWER_CONTROL);
162 mdelay(5);
163
164 pwr |= SDHCI_VDD2_POWER_ON;
165 sdhci_writeb(host, pwr, SDHCI_POWER_CONTROL);
166 mdelay(5);
167
168 if (host->quirks2 & SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON)
169 sdhci_runtime_pm_bus_on(host);
170
171 /*
172 * Some controllers need an extra 10ms delay of 10ms before
173 * they can apply clock after applying power
174 */
175 if (host->quirks & SDHCI_QUIRK_DELAY_AFTER_POWER)
176 mdelay(10);
177 }
178 }
179 EXPORT_SYMBOL_GPL(sdhci_uhs2_set_power);
180
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 34794 bytes --]
next prev parent reply other threads:[~2020-11-06 8:11 UTC|newest]
Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-06 2:26 [RFC PATCH v3.1 00/27] Add support UHS-II for GL9755 AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 01/27] mmc: add UHS-II related definitions in public headers AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 02/27] mmc: core: UHS-II support, modify power-up sequence AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 03/27] mmc: core: UHS-II support, skip set_chip_select() AKASHI Takahiro
2020-11-10 7:15 ` Bough Chen
2020-11-06 2:27 ` [RFC PATCH v3.1 04/27] mmc: core: UHS-II support, try to select UHS-II interface AKASHI Takahiro
2020-11-06 4:24 ` kernel test robot
2020-11-06 2:27 ` [RFC PATCH v3.1 05/27] mmc: core: UHS-II support, skip TMODE setup in some cases AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 06/27] mmc: core: UHS-II support, generate UHS-II SD command packet AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 07/27] mmc: core: UHS-II support, set APP_CMD bit if necessary AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 08/27] mmc: sdhci: add a kernel configuration for enabling UHS-II support AKASHI Takahiro
2020-11-26 8:14 ` Adrian Hunter
2020-11-30 5:17 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 09/27] mmc: sdhci: add UHS-II related definitions in headers AKASHI Takahiro
2020-11-26 8:15 ` Adrian Hunter
2020-11-30 5:21 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 10/27] mmc: sdhci: add UHS-II module AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 11/27] mmc: sdhci-uhs2: dump UHS-II registers AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 12/27] mmc: sdhci-uhs2: add reset function AKASHI Takahiro
2020-11-26 8:16 ` Adrian Hunter
2020-11-30 6:20 ` AKASHI Takahiro
2020-11-30 7:37 ` Adrian Hunter
2020-11-06 2:27 ` [RFC PATCH v3.1 13/27] mmc: sdhci-uhs2: add set_power() to support vdd2 AKASHI Takahiro
2020-11-06 8:11 ` kernel test robot [this message]
2020-11-26 8:16 ` Adrian Hunter
2020-11-30 7:15 ` AKASHI Takahiro
2020-11-30 7:44 ` Adrian Hunter
2020-11-06 2:27 ` [RFC PATCH v3.1 14/27] mmc: sdhci-uhs2: skip signal_voltage_switch() AKASHI Takahiro
2020-11-26 8:16 ` Adrian Hunter
2020-11-30 7:38 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 15/27] mmc: sdhci-uhs2: add set_timeout() AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 16/27] mmc: sdhci-uhs2: add set_ios() AKASHI Takahiro
2020-11-06 4:06 ` kernel test robot
2020-11-06 8:40 ` kernel test robot
2020-11-26 8:17 ` Adrian Hunter
2020-11-30 7:51 ` AKASHI Takahiro
2020-12-03 9:51 ` Adrian Hunter
2020-11-06 2:27 ` [RFC PATCH v3.1 17/27] mmc: sdhci-uhs2: add detect_init() to detect the interface AKASHI Takahiro
2020-11-26 8:17 ` Adrian Hunter
2020-12-01 2:25 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 18/27] mmc: sdhci-uhs2: add clock operations AKASHI Takahiro
2020-11-26 8:17 ` Adrian Hunter
2020-12-01 2:27 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 19/27] mmc: sdhci-uhs2: add set_reg() to initialise the interface AKASHI Takahiro
2020-11-26 8:18 ` Adrian Hunter
2020-12-01 2:28 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 20/27] mmc: sdhci-uhs2: add request() and others AKASHI Takahiro
2020-11-06 4:47 ` kernel test robot
2020-11-26 8:18 ` Adrian Hunter
2020-12-01 2:40 ` AKASHI Takahiro
2020-12-01 11:24 ` Adrian Hunter
2020-11-06 2:27 ` [RFC PATCH v3.1 21/27] mmc: sdhci-uhs2: add irq() " AKASHI Takahiro
2020-12-01 16:46 ` Adrian Hunter
2020-12-08 7:37 ` AKASHI Takahiro
2020-12-08 8:37 ` Adrian Hunter
2020-11-06 2:27 ` [RFC PATCH v3.1 22/27] mmc: sdhci-uhs2: add add_host() and others to set up the driver AKASHI Takahiro
2020-12-03 9:42 ` Adrian Hunter
2020-12-08 7:42 ` AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 23/27] mmc: sdhci-uhs2: add pre-detect_init hook AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 24/27] mmc: core: add post-mmc_attach_sd hook AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 25/27] mmc: sdhci-uhs2: " AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 26/27] mmc: sdhci-pci: add UHS-II support framework AKASHI Takahiro
2020-11-06 2:27 ` [RFC PATCH v3.1 27/27] mmc: sdhci-pci-gli: enable UHS-II mode for GL9755 AKASHI Takahiro
2020-11-06 4:47 ` kernel test robot
2020-11-09 4:38 ` kernel test robot
2020-11-25 7:41 ` [RFC PATCH v3.1 00/27] Add support UHS-II " AKASHI Takahiro
2020-11-25 10:43 ` Ulf Hansson
2020-11-26 0:06 ` AKASHI Takahiro
2020-11-26 8:18 ` Adrian Hunter
2020-12-01 3:09 ` AKASHI Takahiro
2020-12-03 9:55 ` Adrian Hunter
2020-12-08 7:58 ` AKASHI Takahiro
2020-12-08 8:48 ` Adrian Hunter
2020-12-03 10:02 ` Adrian Hunter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202011061654.dGiuyglX-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.