Re: [pci:pci/host-mediatek 11/11] drivers/pci/host/pcie-mediatek.c:541:43: error: 'node' undeclared

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

 



On Sat, Aug 26, 2017 at 06:55:08AM -0500, Bjorn Helgaas wrote:
On Sat, Aug 26, 2017 at 08:56:27AM +0800, Ryder Lee wrote:
Hi,

On Fri, 2017-08-25 at 08:53 +0800, Honghui Zhang wrote:
> On Thu, 2017-08-24 at 08:44 -0500, Bjorn Helgaas wrote:
> > On Thu, Aug 24, 2017 at 11:14:07AM +0800, mtk11102 wrote:
> > > On Wed, 2017-08-23 at 17:52 -0500, Bjorn Helgaas wrote:
> > > > On Wed, Aug 23, 2017 at 10:53:07PM +0800, kbuild test robot wrote:

....
> > > I pull the your host-mediatek branch and seems the build error is still
> > > there.
> > > Should I send a new patch base on your pci/host-mediatek to fix the
> > > build warnings, or should I wait for your push for build error and then
> > > send the patch?
> >
> > Sorry, I must have forgotten to push it.  Should be there now:
> > 9f3ec1e47377 ("PCI: mediatek: Add MSI support for MT2712 and MT7622")
> >
> > Hopefully that has everything we need (I included your fix above).
> >
>
> Thanks very much for your help.
>
> > Bjorn
>
>

I just noticed that 9f3ec1e47377 could not be built pass.

In function 'mtk_pcie_enable_msi':
 error: 'return' with a value, in function returning void [-Werror]
  return 0;

In function 'mtk_pcie_init_irq_domain': error: unused variable
'err' [-Werror=unused-variable]
  int err;

Fixed in 0eb463c096c4 ("PCI: mediatek: Add MSI support for MT2712 and
MT7622").

I wonder why I didn't get a kbuild test report for this?

CC Philip. AFAIK 0day's master server went broken for several days..

Thanks,
Fengguang



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux