Re: [PATCH v10 1/1] [media] i2c: add support for OV13858 sensor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hyungwoo,

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on v4.12-rc5 next-20170609]
[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/Hyungwoo-Yang/i2c-add-support-for-OV13858-sensor/20170613-122218
base:   git://linuxtv.org/media_tree.git master
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All error/warnings (new ones prefixed by >>):

   drivers/media/i2c/ov13858.c: In function 'ov13858_read_platform_data':
>> drivers/media/i2c/ov13858.c:1704:9: error: 'struct ov13858' has no member named 'num_of_skip_frames'
     ov13858->num_of_skip_frames = val;
            ^~
>> drivers/media/i2c/ov13858.c:1729:33: error: assignment of read-only location 'link_freq_menu_items[freq_id]'
      link_freq_menu_items[freq_id] = val;
                                    ^
>> drivers/media/i2c/ov13858.c:1731:12: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
      freq_cfg = &link_freq_configs[freq_id];
               ^

vim +1704 drivers/media/i2c/ov13858.c

  1698			return -EINVAL;
  1699		}
  1700	
  1701		ret = device_property_read_u32(dev, "skip-frames", &val);
  1702		if (ret)
  1703			return ret;
> 1704		ov13858->num_of_skip_frames = val;
  1705	
  1706		device_for_each_child_node(dev, child) {
  1707			if (!fwnode_property_present(child, "link"))
  1708				continue;
  1709	
  1710			/* Limited number of link frequencies are allowed */
  1711			if (freq_id == OV13858_NUM_OF_LINK_FREQS) {
  1712				dev_err(dev, "no more than two freqs\n");
  1713				ret = -EINVAL;
  1714				goto error;
  1715			}
  1716	
  1717			fwn_freq = fwnode_get_next_child_node(child, NULL);
  1718			if (!fwn_freq) {
  1719				ret = -EINVAL;
  1720				goto error;
  1721			}
  1722	
  1723			/* Get link freq menu item for LINK_FREQ control */
  1724			ret = fwnode_property_read_u32(fwn_freq, "link-rate", &val);
  1725			if (ret) {
  1726				dev_err(dev, "link-rate error : %d\n",  ret);
  1727				goto error;
  1728			}
> 1729			link_freq_menu_items[freq_id] = val;
  1730	
> 1731			freq_cfg = &link_freq_configs[freq_id];
  1732			ret = fwnode_property_read_u32(fwn_freq, "pixel-rate", &val);
  1733			if (ret) {
  1734				dev_err(dev, "pixel-rate error : %d\n",  ret);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux