Re: [Powertop] [PATCH] conditionally disable pci support on ARM platforms

On 6/22/2012 11:42 AM, Rajagopal Venkat wrote:

+case "$host" in +arm*)

• AC_DEFINE([HAVE_NO_PCI],[1],[Define if host platform is ARM])
• ;;

+*)

• PKG_CHECK_MODULES([PCIUTILS], [libpci],[],[

• AC_SEARCH_LIBS([pci_get_dev], [pci], [], AC_MSG_ERROR([libpci is required but was not found]), [])
  

• ])
• ;;

+esac

I don't like this part.

if libpci is option, it is optional.

being arm or not is completely irrelevant in this regard. (and there will undoubtedly ARM systems at some point that will have PCI in them)