drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[0]' is used uninitialized in this function
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 18b7fd1c93e5204355ddbf2608a097d64df81b88
commit: 5d1ebbda0318b1ba55eaa1fae3fd867af17b0774 phy: mapphone-mdm6600: Add USB
PHY driver for MDM6600 on Droid 4
date: 4 weeks ago
config: x86_64-randconfig-a0-04151127 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
git checkout 5d1ebbda0318b1ba55eaa1fae3fd867af17b0774
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
drivers/phy/motorola/phy-mapphone-mdm6600.c: In function
'phy_mdm6600_status':
>> drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[0]' is
>> used uninitialized in this function [-Wuninitialized]
val |= values[i] << i;
~~^~~
drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[1]' is
used uninitialized in this function [-Wuninitialized]
drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[2]' is
used uninitialized in this function [-Wuninitialized]
vim +188 drivers/phy/motorola/phy-mapphone-mdm6600.c
166
167 /**
168 * phy_mdm6600_status() - read mdm6600 status lines
169 * @ddata: device driver data
170 */
171 static void phy_mdm6600_status(struct work_struct *work)
172 {
173 struct phy_mdm6600 *ddata;
174 struct device *dev;
175 int values[PHY_MDM6600_NR_STATUS_LINES];
176 int error, i, val = 0;
177
178 ddata = container_of(work, struct phy_mdm6600,
status_work.work);
179 dev = ddata->dev;
180
181 error = gpiod_get_array_value_cansleep(PHY_MDM6600_NR_CMD_LINES,
182
ddata->status_gpios->desc,
183 values);
184 if (error)
185 return;
186
187 for (i = 0; i < PHY_MDM6600_NR_CMD_LINES; i++) {
> 188 val |= values[i] << i;
189 dev_dbg(ddata->dev, "XXX %s: i: %i values[i]: %i val:
%i\n",
190 __func__, i, values[i], val);
191 }
192 ddata->status = val;
193
194 dev_info(dev, "modem status: %i %s\n",
195 ddata->status,
196 phy_mdm6600_status_name[ddata->status & 7]);
197 complete(>ack);
198 }
199
---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[0]' is used uninitialized in this function
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 18b7fd1c93e5204355ddbf2608a097d64df81b88
commit: 5d1ebbda0318b1ba55eaa1fae3fd867af17b0774 phy: mapphone-mdm6600: Add USB
PHY driver for MDM6600 on Droid 4
date: 4 weeks ago
config: x86_64-randconfig-a0-04151127 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
git checkout 5d1ebbda0318b1ba55eaa1fae3fd867af17b0774
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
drivers/phy/motorola/phy-mapphone-mdm6600.c: In function
'phy_mdm6600_status':
>> drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[0]' is
>> used uninitialized in this function [-Wuninitialized]
val |= values[i] << i;
~~^~~
drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[1]' is
used uninitialized in this function [-Wuninitialized]
drivers/phy/motorola/phy-mapphone-mdm6600.c:188:16: warning: 'values[2]' is
used uninitialized in this function [-Wuninitialized]
vim +188 drivers/phy/motorola/phy-mapphone-mdm6600.c
166
167 /**
168 * phy_mdm6600_status() - read mdm6600 status lines
169 * @ddata: device driver data
170 */
171 static void phy_mdm6600_status(struct work_struct *work)
172 {
173 struct phy_mdm6600 *ddata;
174 struct device *dev;
175 int values[PHY_MDM6600_NR_STATUS_LINES];
176 int error, i, val = 0;
177
178 ddata = container_of(work, struct phy_mdm6600,
status_work.work);
179 dev = ddata->dev;
180
181 error = gpiod_get_array_value_cansleep(PHY_MDM6600_NR_CMD_LINES,
182
ddata->status_gpios->desc,
183 values);
184 if (error)
185 return;
186
187 for (i = 0; i < PHY_MDM6600_NR_CMD_LINES; i++) {
> 188 val |= values[i] << i;
189 dev_dbg(ddata->dev, "XXX %s: i: %i values[i]: %i val:
%i\n",
190 __func__, i, values[i], val);
191 }
192 ddata->status = val;
193
194 dev_info(dev, "modem status: %i %s\n",
195 ddata->status,
196 phy_mdm6600_status_name[ddata->status & 7]);
197 complete(>ack);
198 }
199
---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
