[Buildroot] [PATCH v3] package/iqvlinux: new package

Romain Naour romain.naour at openwide.fr
Tue Oct 13 09:00:55 UTC 2015


Hi Thomas,

Le 12/10/2015 23:17, Thomas Petazzoni a écrit :
> Dear Romain Naour,
> 
> On Fri,  9 Oct 2015 15:26:02 +0200, Romain Naour wrote:
>> The PCI support needs to be checked since this driver is based
>> on it. Otherwise the build fail with:
>>  #error "This driver requires PCI support to be available"
>>
>> But this message is concealed by several occurrence of this
>> one:
>>  error: implicit declaration of function 'pci_find_bus' [-Werror=implicit-function-declaration]
>>
>> Signed-off-by: Romain Naour <romain.naour at openwide.fr>
>> Cc: Yann E. MORIN <yann.morin.1998 at free.fr>
>> Cc: Arnout Vandecappelle <arnout at mind.be>
>> ---
>> v3: changes suggested by Arnout
>>     - remove the patch and use CC=$(TARGET_CC) instead.
>>     - Add a 'or' between GPLv2 and BSD-3c for license information
>>     - Add some more license files (files.txt)
>>     - Fixes typos and indentation.
>>     - move the check for PCI option to LINUX_POST_CONFIGURE_HOOKS
>>     - redirect echo to stderr.
>> v2: - rename the package simply to iqvlinux (ThomasP)
>>     - move it to "Hardware Handling" menu (ThomasP)
>>     - Cc Yann for the kernel-module infra
>>     - Add a check for CONFIG_PCI even if it's redundant with
>>       the message from the Makefile.
>>       (Do we really need this check ?)
>> ---
>>  package/Config.in              |  1 +
>>  package/iqvlinux/Config.in     | 18 ++++++++++++++++++
>>  package/iqvlinux/iqvlinux.hash |  5 +++++
>>  package/iqvlinux/iqvlinux.mk   | 34 ++++++++++++++++++++++++++++++++++
>>  4 files changed, 58 insertions(+)
>>  create mode 100644 package/iqvlinux/Config.in
>>  create mode 100644 package/iqvlinux/iqvlinux.hash
>>  create mode 100644 package/iqvlinux/iqvlinux.mk
> 
> Applied after doing the following changes:
> 
>     [Thomas:
>      - fix minor typo in Config.in: s/Enthernet/Ethernet/
>      - license is "GPLv2, BSD-3c", not "GPLv2 or BSD-3c"
>      - remove IQVLINUX_PCI_CHECK, until a proper generic solution is
>        implemented.]
> 
> As you can see, I've removed the IQVLINUX_PCI_CHECK so that a nicer
> solution can be discussed/implemented with Yann. However, since this
> check was not really mandatory, I decided to add the package
> nonetheless (albeit without the check).

Ok, it's not a problem for me. The kernel I use already have CONFIG_PCI enabled.
I'll follow the discussion about the kernel option check.

Best regards,
Romain

> 
> Thanks!
> 
> Thomas
> 



More information about the buildroot mailing list