Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1647686

Re: [PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA

From Kishon Vijay Abraham I <kishon@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA
Date 2017-05-23 08:50 +0200
Message-ID <tKbNM-7U4-13@gated-at.bofh.it> (permalink)
References <tHOsi-8uX-17@gated-at.bofh.it> <tK33Q-2tD-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Bjorn,

On Tuesday 23 May 2017 02:56 AM, Bjorn Helgaas wrote:
> On Tue, May 16, 2017 at 07:21:08PM +0200, Geert Uytterhoeven wrote:
>> If NO_DMA=y:
>>
>>     drivers/built-in.o: In function `__pci_epc_create':
>>     (.text+0xef4e): undefined reference to `bad_dma_ops'
>>     drivers/built-in.o: In function `pci_epc_add_epf':
>>     (.text+0xf676): undefined reference to `bad_dma_ops'
>>     drivers/built-in.o: In function `pci_epf_alloc_space':
>>     (.text+0xfa32): undefined reference to `bad_dma_ops'
>>     drivers/built-in.o: In function `pci_epf_free_space':
>>     (.text+0xfac4): undefined reference to `bad_dma_ops'
>>
>> Add a dependency on HAS_DMA to fix this.
>>
>> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> 
> Applied to for-linus for v4.12, thanks!
> 
>> ---
>> Should PCI_ENDPOINT depend on CONFIG_PCI instead?
> 
> I *think* we want HAS_DMA, not CONFIG_PCI, because CONFIG_PCI means we
> want the infrastructure for PCI host bridges, enumerating children of
> the bridges, etc.  I think we want to allow PCI_ENDPOINT without all
> that.
> 
> Kishon?

right. CONFIG_PCI is for enabling the host infrastructure and PCI_ENDPOINT and
CONFIG_PCI should be completely independent of each other.

Thanks
Kishon

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA Geert Uytterhoeven <geert@linux-m68k.org> - 2017-05-16 19:30 +0200
  Re: [PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA Bjorn Helgaas <helgaas@kernel.org> - 2017-05-22 23:30 +0200
    Re: [PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA Kishon Vijay Abraham I <kishon@ti.com> - 2017-05-23 08:50 +0200
      Re: [PATCH] PCI: endpoint: PCI_ENDPOINT should depend on HAS_DMA Geert Uytterhoeven <geert@linux-m68k.org> - 2017-05-23 09:20 +0200

csiph-web