Subject: Re: linux-next: Tree for July 4 (misc/pti)

From: J Freyensee <[email-masked]>
Date: Thu, 07 Jul 2011 09:29:59 -0700

On 07/07/2011 08:14 AM, Randy Dunlap wrote:
> On Mon, 4 Jul 2011 13:45:41 -0700 Randy Dunlap wrote:
>
>> On Mon, 4 Jul 2011 17:09:52 +1000 Stephen Rothwell wrote:
>>
>>> Hi all,
>>>
>>> Changes since 20110701:
>>
>> when CONFIG_PCI is not enabled:
>>
>> drivers/misc/pti.c:410: error: implicit declaration of function 'pci_release_region'
>> drivers/misc/pti.c:859: error: implicit declaration of function 'pci_request_region'
>>
>>
>> Stubs for these 2 functions could be added to include/linux/pci.h, but it would
>> make more sense to me just to make the driver depend on PCI since it seems to rely
>> on so many PCI functions.  or are the PCI pieces optional?
>
> ping.  still a problem in linux-next 20110706.

Thanks for the ping.

Is there something you need me to do or look at?

For Intel-Atom architectures, the MIPI PTI module does sit on the PCI 
bus.  It's very much dependent upon the PCI bus for working 
functionality and it's not optional.  So yes, it makes sense for the 
driver to depend on PCI to be there and if PCI is not enabled on the 
system, don't build the pti driver.

Maybe it's a Kconfig dependency?

Thanks,
Jay
> ---
> ~Randy
> *** Remember to use Documentation/SubmitChecklist when testing your code ***