Re: [media 7/8] PCIE bridge driver for PT3 & PX-Q3PE

From: kbuild test robot <lkp@intel.com>
To: info@are.ma
Cc: kbuild-all@01.org, linux-media@vger.kernel.org, "Буди Романто,
	AreMa Inc" <knightrider@are.ma>,
	linux-kernel@vger.kernel.org, crope@iki.fi, m.chehab@samsung.com,
	mchehab@osg.samsung.com, hdegoede@redhat.com,
	laurent.pinchart@ideasonboard.com, mkrufky@linuxtv.org,
	sylvester.nawrocki@gmail.com, g.liakhovetski@gmx.de,
	peter.senna@gmail.com
Subject: Re: [media 7/8] PCIE bridge driver for PT3 & PX-Q3PE
Date: Fri, 1 Apr 2016 12:13:07 +0800	[thread overview]
Message-ID: <201604011220.3urAH9i3%fengguang.wu@intel.com> (raw)
In-Reply-To: <08c733e51c242ee9c9a2fee5bfac4a3fd70f0ec7.1459450632.git.knightrider@are.ma>

[-- Attachment #1: Type: text/plain, Size: 4151 bytes --]

Hi Буди,

[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.6-rc1 next-20160331]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/info-are-ma/DVB-driver-for-Earthsoft-PT3-PLEX-PX-Q3PE-ISDB-S-T-PCIE-cards-PX-BCUD-ISDB-S-USB-dongle/20160401-034740
base:   git://linuxtv.org/media_tree.git master
config: i386-allmodconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/media/pci/ptx/pxq3pe_pci.c: In function 'pxq3pe_remove':
   drivers/media/pci/ptx/pxq3pe_pci.c:484:3: error: implicit declaration of function 'vfree' [-Werror=implicit-function-declaration]
      vfree(p->sBuf);
      ^
   drivers/media/pci/ptx/pxq3pe_pci.c: In function 'pxq3pe_probe':
   drivers/media/pci/ptx/pxq3pe_pci.c:536:14: error: implicit declaration of function 'vzalloc' [-Werror=implicit-function-declaration]
      p->sBuf  = vzalloc(p->sBufSize);
                 ^
>> drivers/media/pci/ptx/pxq3pe_pci.c:536:12: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
      p->sBuf  = vzalloc(p->sBufSize);
               ^
   cc1: some warnings being treated as errors

vim +536 drivers/media/pci/ptx/pxq3pe_pci.c

   478		if (c->dma.dat)
   479			pci_free_consistent(card->pdev, c->dma.sz, c->dma.dat, c->dma.adr);
   480		for (i = 0; i < card->adapn; i++) {
   481			struct ptx_adap		*adap	= &card->adap[i];
   482			struct pxq3pe_adap	*p	= adap->priv;
   483	
 > 484			vfree(p->sBuf);
   485		}
   486		if (c->bar)
   487			pci_iounmap(pdev, c->bar);
   488		ptx_unregister_adap(card);
   489	}
   490	
   491	static const struct i2c_algorithm pxq3pe_algo = {
   492		.functionality	= ptx_i2c_func,
   493		.master_xfer	= pxq3pe_xfr,
   494	};
   495	
   496	static int pxq3pe_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id)
   497	{
   498		struct ptx_subdev_info	pxq3pe_subdev_info[] = {
   499			{SYS_ISDBT, 0x20, TC90522_MODNAME, 0x10, NM131_MODNAME},
   500			{SYS_ISDBS, 0x22, TC90522_MODNAME, 0x11, TDA2014X_MODNAME},
   501			{SYS_ISDBT, 0x24, TC90522_MODNAME, 0x12, NM131_MODNAME},
   502			{SYS_ISDBS, 0x26, TC90522_MODNAME, 0x13, TDA2014X_MODNAME},
   503			{SYS_ISDBT, 0x28, TC90522_MODNAME, 0x14, NM131_MODNAME},
   504			{SYS_ISDBS, 0x2A, TC90522_MODNAME, 0x15, TDA2014X_MODNAME},
   505			{SYS_ISDBT, 0x2C, TC90522_MODNAME, 0x16, NM131_MODNAME},
   506			{SYS_ISDBS, 0x2E, TC90522_MODNAME, 0x17, TDA2014X_MODNAME},
   507		};
   508		struct ptx_card		*card	= ptx_alloc(pdev, KBUILD_MODNAME, ARRAY_SIZE(pxq3pe_subdev_info),
   509							sizeof(struct pxq3pe_card), sizeof(struct pxq3pe_adap), pxq3pe_lnb);
   510		struct pxq3pe_card	*c;
   511		u8	regctl	= 0xA0,
   512			i;
   513		u16	cfg;
   514		int	err	= !card || pci_read_config_word(pdev, PCI_COMMAND, &cfg);
   515	
   516		if (err)
   517			return ptx_abort(pdev, pxq3pe_remove, err, "Memory/PCI error, card=%p", card);
   518		c	= card->priv;
   519		if (!(cfg & PCI_COMMAND_MASTER)) {
   520			pci_set_master(pdev);
   521			pci_read_config_word(pdev, PCI_COMMAND, &cfg);
   522			if (!(cfg & PCI_COMMAND_MASTER))
   523				return ptx_abort(pdev, pxq3pe_remove, -EIO, "Bus Mastering is disabled");
   524		}
   525		c->bar	= pci_iomap(pdev, 0, 0);
   526		if (!c->bar)
   527			return ptx_abort(pdev, pxq3pe_remove, -EIO, "I/O map failed");
   528		if (ptx_i2c_add_adapter(card, &pxq3pe_algo))
   529			return ptx_abort(pdev, pxq3pe_remove, -EIO, "Cannot add I2C");
   530	
   531		for (i = 0; i < card->adapn; i++) {
   532			struct ptx_adap		*adap	= &card->adap[i];
   533			struct pxq3pe_adap	*p	= adap->priv;
   534	
   535			p->sBufSize	= PTX_TS_SIZE * 100 << 9;
 > 536			p->sBuf		= vzalloc(p->sBufSize);
   537			if (!p->sBuf)
   538				return ptx_abort(pdev, pxq3pe_remove, -ENOMEM, "No memory for stream buffer");
   539		}

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 54429 bytes --]