Hi Meghana,

[auto build test WARNING on v4.14-rc3]
[also build test WARNING on next-20171009]
[cannot apply to drm/drm-next backlight/for-backlight-next 
drm-exynos/exynos-drm/for-next]
[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/Meghana-Madhyastha/Move-backlight-helper-functions-from-tinydrm-helpers-to-linux-backlight/20171013-062032
config: i386-randconfig-x001-201741 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/video/backlight/backlight.c: In function 'backlight_get':
>> drivers/video/backlight/backlight.c:609:39: warning: character constant too 
>> long for its type
      np = of_parse_phandle(dev->of_node, 'backlight', 0);
                                          ^~~~~~~~~~~
>> drivers/video/backlight/backlight.c:609:39: warning: passing argument 2 of 
>> 'of_parse_phandle' makes pointer from integer without a cast 
>> [-Wint-conversion]
   In file included from include/linux/irqdomain.h:34:0,
                    from include/linux/i2c.h:34,
                    from include/uapi/linux/fb.h:5,
                    from include/linux/fb.h:5,
                    from include/linux/backlight.h:12,
                    from drivers/video/backlight/backlight.c:13:
   include/linux/of.h:358:28: note: expected 'const char *' but argument is of 
type 'int'
    extern struct device_node *of_parse_phandle(const struct device_node *np,
                               ^~~~~~~~~~~~~~~~

vim +609 drivers/video/backlight/backlight.c

   582  
   583  /**
   584   * backlight_get - Get backlight device
   585   * @dev: Device
   586   *
   587   * This function looks for a property named 'backlight' on the DT node
   588   * connected to @dev and looks up the backlight device.
   589   *
   590   * Call backlight_put() to drop the reference on the backlight device.
   591   *
   592   * Returns:
   593   * A pointer to the backlight device if found.
   594   * Error pointer -EPROBE_DEFER if the DT property is set, but no 
backlight
   595   * device is found.
   596   * NULL if there's no backlight property.
   597   */
   598  
   599  #if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
   600  struct backlight_device *backlight_get(struct device *dev)
   601  {
   602          struct backlight_device *bd = NULL;
   603          struct device_node *np;
   604  
   605          if (!dev)
   606                  return NULL;
   607  
   608          if (IS_ENABLED(CONFIG_OF) && dev->of_node) {
 > 609                  np = of_parse_phandle(dev->of_node, 'backlight', 0);
   610                  if (np) {
   611                          bd = of_find_backlight_by_node(np);
   612                          of_node_put(np);
   613                          if (!bd)
   614                                  return ERR_PTR(-EPROBE_DEFER);
   615                  }
   616          }
   617  
   618          return bd;
   619  }
   620  EXPORT_SYMBOL(backlight_get);
   621  #endif
   622  

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

Attachment: .config.gz
Description: application/gzip

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to