All these warnings are corret but the patch just moves code around, the
warnings aren't new.
regards,
dan carpenter
On Tue, Oct 31, 2017 at 11:11:27PM +0800, kbuild test robot wrote:
> CC: kbuild-...@01.org
> CC: linux-me...@vger.kernel.org
> TO: Sakari Ailus <sakari.ai...@linux.intel.com>
> CC: Mauro Carvalho Chehab <m.che...@samsung.com>
> CC: linux-me...@vger.kernel.org
>
> tree: git://linuxtv.org/media_tree.git master
> head: 3fa013be95bb046490d3b11d546bf06d73c7e0e3
> commit: 90ebe55ab88635a19af06d923bb70ef236123399 [2809/2878] media: staging:
> atomisp: Add driver prefix to Kconfig option and module names
> :::::: branch date: 4 hours ago
> :::::: commit date: 4 days ago
>
> drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c:1698
> ov5693_detect() error: uninitialized symbol 'high'.
> drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c:1703
> ov5693_detect() error: uninitialized symbol 'low'.
> drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c:1563 mt9m114_detect()
> error: uninitialized symbol 'retvalue'.
> drivers/staging/media/atomisp/i2c/atomisp-gc2235.c:281
> gc2235_get_intg_factor() error: uninitialized symbol 'reg_val_h'.
> drivers/staging/media/atomisp/i2c/atomisp-gc2235.c:862 gc2235_detect() error:
> uninitialized symbol 'high'.
> drivers/staging/media/atomisp/i2c/atomisp-gc2235.c:867 gc2235_detect() error:
> uninitialized symbol 'low'.
> drivers/staging/media/atomisp/i2c/atomisp-ov2722.c:996 ov2722_detect() error:
> uninitialized symbol 'high'.
> drivers/staging/media/atomisp/i2c/atomisp-ov2722.c:1001 ov2722_detect()
> error: uninitialized symbol 'low'.
> drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:1177 ov2680_detect()
> error: uninitialized symbol 'high'.
> drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:1182 ov2680_detect()
> error: uninitialized symbol 'low'.
>
> git remote add linuxtv-media git://linuxtv.org/media_tree.git
> git remote update linuxtv-media
> git checkout 90ebe55ab88635a19af06d923bb70ef236123399
> vim +/high +1698 drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c
>
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1683
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1684 static int ov5693_detect(struct i2c_client *client)
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1685 {
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1686 struct i2c_adapter *adapter = client->adapter;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1687 u16 high, low;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1688 int ret;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1689 u16 id;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1690 u8 revision;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1691
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1692 if (!i2c_check_functionality(adapter, I2C_FUNC_I2C))
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1693 return -ENODEV;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1694
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1695 ret = ov5693_read_reg(client, OV5693_8BIT,
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1696
> OV5693_SC_CMMN_CHIP_ID_H, &high);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1697 if (ret) {
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 @1698 dev_err(&client->dev, "sensor_id_high =
> 0x%x\n", high);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1699 return -ENODEV;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1700 }
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1701 ret = ov5693_read_reg(client, OV5693_8BIT,
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1702
> OV5693_SC_CMMN_CHIP_ID_L, &low);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 @1703 id = ((((u16) high) << 8) | (u16) low);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1704
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1705 if (id != OV5693_ID) {
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1706 dev_err(&client->dev, "sensor ID error
> 0x%x\n", id);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1707 return -ENODEV;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1708 }
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1709
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1710 ret = ov5693_read_reg(client, OV5693_8BIT,
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1711
> OV5693_SC_CMMN_SUB_ID, &high);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1712 revision = (u8) high & 0x0f;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1713
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1714 dev_dbg(&client->dev, "sensor_revision = 0x%x\n",
> revision);
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1715 dev_dbg(&client->dev, "detect ov5693 success\n");
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1716 return 0;
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1717 }
> a49d2536 drivers/staging/media/atomisp/i2c/ov5693/ov5693.c Alan Cox
> 2017-02-17 1718
>
> :::::: The code at line 1698 was first introduced by commit
> :::::: a49d25364dfb9f8a64037488a39ab1f56c5fa419 staging/atomisp: Add support
> for the Intel IPU v2
>
> :::::: TO: Alan Cox <a...@linux.intel.com>
> :::::: CC: Greg Kroah-Hartman <gre...@linuxfoundation.org>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
_______________________________________________
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild
