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) _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
