[PATCH v3 3/4] bus/ifpga: use C linkage where appropriate in driver header

Mattias Rönnblom hofors at lysator.liu.se
Mon Oct 21 20:01:30 CEST 2024


On 2024-10-21 18:04, David Marchand wrote:
> This driver was missed when cleaning up most of the headers.
> 
> Fixes: 719834a6849e ("use C linkage where appropriate in headers")
> 
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
>   drivers/bus/ifpga/bus_ifpga_driver.h | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/bus/ifpga/bus_ifpga_driver.h b/drivers/bus/ifpga/bus_ifpga_driver.h
> index 5bbe36d6e0..a42afc7d75 100644
> --- a/drivers/bus/ifpga/bus_ifpga_driver.h
> +++ b/drivers/bus/ifpga/bus_ifpga_driver.h
> @@ -11,16 +11,16 @@
>    * RTE Intel FPGA Bus Interface
>    */
>   
> -#ifdef __cplusplus
> -extern "C" {
> -#endif /* __cplusplus */
> -
>   #include <rte_compat.h>
>   #include <dev_driver.h>
>   #include <rte_pci.h>
>   #include <rte_interrupts.h>
>   #include <rte_spinlock.h>
>   
> +#ifdef __cplusplus
> +extern "C" {
> +#endif /* __cplusplus */
> +
>   #define IFPGA_BUS_NAME ifpga
>   
>   /* Forward declarations */

My effort was limited to the headers passed to chkincs, which I'm 
guessing is the public API headers.

Internal headers shouldn't have any #ifdef __cpluspus at all, anywhere.



More information about the dev mailing list