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

Reply via email to