Re: [PATCH v2] video: fbdev: ssd1307fb: Added support to Column offset

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

 



Hi Rodrigo,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on next-20200512]
[cannot apply to robh/for-next linus/master linux/master v5.7-rc5 v5.7-rc4 v5.7-rc3 v5.7-rc5]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Rodrigo-Rolim-Mendes-de-Alencar/video-fbdev-ssd1307fb-Added-support-to-Column-offset/20200514-010137
base:    e098d7762d602be640c53565ceca342f81e55ad2
config: parisc-randconfig-r022-20200513 (attached as .config)
compiler: hppa-linux-gcc (GCC) 9.3.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=parisc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

drivers/video/fbdev/ssd1307fb.c: In function 'ssd1307fb_probe':
>> drivers/video/fbdev/ssd1307fb.c:630:31: error: 'node' undeclared (first use in this function); did you mean 'inode'?
630 |  if (device_property_read_u32(node, "solomon,col-offset", &par->col_offset))
|                               ^~~~
|                               inode
drivers/video/fbdev/ssd1307fb.c:630:31: note: each undeclared identifier is reported only once for each function it appears in

vim +630 drivers/video/fbdev/ssd1307fb.c

   579	
   580	static int ssd1307fb_probe(struct i2c_client *client)
   581	{
   582		struct device *dev = &client->dev;
   583		struct backlight_device *bl;
   584		char bl_name[12];
   585		struct fb_info *info;
   586		struct fb_deferred_io *ssd1307fb_defio;
   587		u32 vmem_size;
   588		struct ssd1307fb_par *par;
   589		void *vmem;
   590		int ret;
   591	
   592		info = framebuffer_alloc(sizeof(struct ssd1307fb_par), dev);
   593		if (!info)
   594			return -ENOMEM;
   595	
   596		par = info->par;
   597		par->info = info;
   598		par->client = client;
   599	
   600		par->device_info = device_get_match_data(dev);
   601	
   602		par->reset = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_LOW);
   603		if (IS_ERR(par->reset)) {
   604			dev_err(dev, "failed to get reset gpio: %ld\n",
   605				PTR_ERR(par->reset));
   606			ret = PTR_ERR(par->reset);
   607			goto fb_alloc_error;
   608		}
   609	
   610		par->vbat_reg = devm_regulator_get_optional(dev, "vbat");
   611		if (IS_ERR(par->vbat_reg)) {
   612			ret = PTR_ERR(par->vbat_reg);
   613			if (ret == -ENODEV) {
   614				par->vbat_reg = NULL;
   615			} else {
   616				dev_err(dev, "failed to get VBAT regulator: %d\n", ret);
   617				goto fb_alloc_error;
   618			}
   619		}
   620	
   621		if (device_property_read_u32(dev, "solomon,width", &par->width))
   622			par->width = 96;
   623	
   624		if (device_property_read_u32(dev, "solomon,height", &par->height))
   625			par->height = 16;
   626	
   627		if (device_property_read_u32(dev, "solomon,page-offset", &par->page_offset))
   628			par->page_offset = 1;
   629	
 > 630		if (device_property_read_u32(node, "solomon,col-offset", &par->col_offset))
   631			par->col_offset = 0;
   632	
   633		if (device_property_read_u32(dev, "solomon,com-offset", &par->com_offset))
   634			par->com_offset = 0;
   635	
   636		if (device_property_read_u32(dev, "solomon,prechargep1", &par->prechargep1))
   637			par->prechargep1 = 2;
   638	
   639		if (device_property_read_u32(dev, "solomon,prechargep2", &par->prechargep2))
   640			par->prechargep2 = 2;
   641	
   642		if (!device_property_read_u8_array(dev, "solomon,lookup-table",
   643						   par->lookup_table,
   644						   ARRAY_SIZE(par->lookup_table)))
   645			par->lookup_table_set = 1;
   646	
   647		par->seg_remap = !device_property_read_bool(dev, "solomon,segment-no-remap");
   648		par->com_seq = device_property_read_bool(dev, "solomon,com-seq");
   649		par->com_lrremap = device_property_read_bool(dev, "solomon,com-lrremap");
   650		par->com_invdir = device_property_read_bool(dev, "solomon,com-invdir");
   651		par->area_color_enable =
   652			device_property_read_bool(dev, "solomon,area-color-enable");
   653		par->low_power = device_property_read_bool(dev, "solomon,low-power");
   654	
   655		par->contrast = 127;
   656		par->vcomh = par->device_info->default_vcomh;
   657	
   658		/* Setup display timing */
   659		if (device_property_read_u32(dev, "solomon,dclk-div", &par->dclk_div))
   660			par->dclk_div = par->device_info->default_dclk_div;
   661		if (device_property_read_u32(dev, "solomon,dclk-frq", &par->dclk_frq))
   662			par->dclk_frq = par->device_info->default_dclk_frq;
   663	
   664		vmem_size = DIV_ROUND_UP(par->width, 8) * par->height;
   665	
   666		vmem = (void *)__get_free_pages(GFP_KERNEL | __GFP_ZERO,
   667						get_order(vmem_size));
   668		if (!vmem) {
   669			dev_err(dev, "Couldn't allocate graphical memory.\n");
   670			ret = -ENOMEM;
   671			goto fb_alloc_error;
   672		}
   673	
   674		ssd1307fb_defio = devm_kzalloc(dev, sizeof(*ssd1307fb_defio),
   675					       GFP_KERNEL);
   676		if (!ssd1307fb_defio) {
   677			dev_err(dev, "Couldn't allocate deferred io.\n");
   678			ret = -ENOMEM;
   679			goto fb_alloc_error;
   680		}
   681	
   682		ssd1307fb_defio->delay = HZ / refreshrate;
   683		ssd1307fb_defio->deferred_io = ssd1307fb_deferred_io;
   684	
   685		info->fbops = &ssd1307fb_ops;
   686		info->fix = ssd1307fb_fix;
   687		info->fix.line_length = DIV_ROUND_UP(par->width, 8);
   688		info->fbdefio = ssd1307fb_defio;
   689	
   690		info->var = ssd1307fb_var;
   691		info->var.xres = par->width;
   692		info->var.xres_virtual = par->width;
   693		info->var.yres = par->height;
   694		info->var.yres_virtual = par->height;
   695	
   696		info->screen_buffer = vmem;
   697		info->fix.smem_start = __pa(vmem);
   698		info->fix.smem_len = vmem_size;
   699	
   700		fb_deferred_io_init(info);
   701	
   702		i2c_set_clientdata(client, info);
   703	
   704		if (par->reset) {
   705			/* Reset the screen */
   706			gpiod_set_value_cansleep(par->reset, 1);
   707			udelay(4);
   708			gpiod_set_value_cansleep(par->reset, 0);
   709			udelay(4);
   710		}
   711	
   712		if (par->vbat_reg) {
   713			ret = regulator_enable(par->vbat_reg);
   714			if (ret) {
   715				dev_err(dev, "failed to enable VBAT: %d\n", ret);
   716				goto reset_oled_error;
   717			}
   718		}
   719	
   720		ret = ssd1307fb_init(par);
   721		if (ret)
   722			goto regulator_enable_error;
   723	
   724		ret = register_framebuffer(info);
   725		if (ret) {
   726			dev_err(dev, "Couldn't register the framebuffer\n");
   727			goto panel_init_error;
   728		}
   729	
   730		snprintf(bl_name, sizeof(bl_name), "ssd1307fb%d", info->node);
   731		bl = backlight_device_register(bl_name, dev, par, &ssd1307fb_bl_ops,
   732					       NULL);
   733		if (IS_ERR(bl)) {
   734			ret = PTR_ERR(bl);
   735			dev_err(dev, "unable to register backlight device: %d\n", ret);
   736			goto bl_init_error;
   737		}
   738	
   739		bl->props.brightness = par->contrast;
   740		bl->props.max_brightness = MAX_CONTRAST;
   741		info->bl_dev = bl;
   742	
   743		dev_info(dev, "fb%d: %s framebuffer device registered, using %d bytes of video memory\n", info->node, info->fix.id, vmem_size);
   744	
   745		return 0;
   746	
   747	bl_init_error:
   748		unregister_framebuffer(info);
   749	panel_init_error:
   750		if (par->device_info->need_pwm) {
   751			pwm_disable(par->pwm);
   752			pwm_put(par->pwm);
   753		}
   754	regulator_enable_error:
   755		if (par->vbat_reg)
   756			regulator_disable(par->vbat_reg);
   757	reset_oled_error:
   758		fb_deferred_io_cleanup(info);
   759	fb_alloc_error:
   760		framebuffer_release(info);
   761		return ret;
   762	}
   763	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux