From: "Mike Frysinger" <vapier.adi@gmail.com>
To: "Johannes Berg" <johannes@sipsolutions.net>
Cc: "Mike Frysinger" <vapier@gentoo.org>,
"Michael Buesch" <mb@bu3sch.de>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] SSB: make it depend on PCI
Date: Thu, 6 Nov 2008 06:26:36 -0500 [thread overview]
Message-ID: <8bd0f97a0811060326n5bcfc17fm5b5a65b12e71ea61@mail.gmail.com> (raw)
In-Reply-To: <1225970573.3619.208.camel@johannes.berg>
On Thu, Nov 6, 2008 at 06:22, Johannes Berg wrote:
> On Thu, 2008-11-06 at 06:15 -0500, Mike Frysinger wrote:
>> while that may be the intent, it doesnt work today.
>>
>> drivers/ssb/main.c always includes linux/ssb/ssb.h
>>
>> linux/ssb/ssb.h always calls pci_dma_* functions
>>
>> arches that do not support PCI fail to build:
>> CC drivers/ssb/main.o
>> In file included from drivers/ssb/ssb_private.h:5,
>> from drivers/ssb/main.c:12:
>> include/linux/ssb/ssb.h: In function 'ssb_dma_mapping_error':
>> include/linux/ssb/ssb.h:430: error: implicit declaration of function 'pci_dma_mapping_error'
>> include/linux/ssb/ssb.h: In function 'ssb_dma_sync_single_for_cpu':
>> include/linux/ssb/ssb.h:475: error: implicit declaration of function 'pci_dma_sync_single_for_cpu'
>> include/linux/ssb/ssb.h: In function 'ssb_dma_sync_single_for_device':
>> include/linux/ssb/ssb.h:493: error: implicit declaration of function 'pci_dma_sync_single_for_device'
>> include/linux/ssb/ssb.h: In function 'ssb_dma_sync_single_range_for_cpu':
>> include/linux/ssb/ssb.h:517: error: implicit declaration of function 'dma_sync_single_range_for_cpu'
>> include/linux/ssb/ssb.h: In function 'ssb_dma_sync_single_range_for_device':
>> include/linux/ssb/ssb.h:538: error: implicit declaration of function 'dma_sync_single_range_for_device'
>> drivers/ssb/main.c: In function 'ssb_dma_set_mask':
>> drivers/ssb/main.c:1183: error: implicit declaration of function 'dma_set_mask'
>
>
> That seems odd, shouldn't at least the dma_sync_* functions and
> dma_set_mask be supported?
i'm highlighting the pci-specific functions. the common dma functions
missing are not a ssb problem, so ignore those ;).
-mike
next prev parent reply other threads:[~2008-11-06 11:26 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-06 9:51 Mike Frysinger
2008-11-06 11:07 ` Johannes Berg
2008-11-06 11:15 ` Mike Frysinger
2008-11-06 11:22 ` Johannes Berg
2008-11-06 11:26 ` Mike Frysinger [this message]
2008-11-06 11:29 ` Johannes Berg
2008-11-06 11:32 ` Mike Frysinger
2008-11-06 11:35 ` Johannes Berg
2008-11-06 16:59 ` Michael Buesch
2008-11-06 21:14 ` Mike Frysinger
2008-11-06 21:24 ` Michael Buesch
2008-11-06 16:56 ` Michael Buesch
2008-11-09 10:51 ` Geert Uytterhoeven
2008-11-09 11:49 ` Michael Buesch
2008-11-06 16:55 ` Michael Buesch
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8bd0f97a0811060326n5bcfc17fm5b5a65b12e71ea61@mail.gmail.com \
--to=vapier.adi@gmail.com \
--cc=johannes@sipsolutions.net \
--cc=linux-kernel@vger.kernel.org \
--cc=mb@bu3sch.de \
--cc=netdev@vger.kernel.org \
--cc=vapier@gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®