DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] net/bonding: Fix header for C++
@ 2023-08-09 15:52 Visa Hankala
  2023-08-09 17:49 ` Tyler Retzlaff
  0 siblings, 1 reply; 3+ messages in thread
From: Visa Hankala @ 2023-08-09 15:52 UTC (permalink / raw)
  To: dev; +Cc: Chas Williams, Min Hu

Apply C linkage to the whole header to allow use with C++.

Fixes: dc40f17a36b ("net/bonding: allow external state machine in mode 4")

Signed-off-by: Visa Hankala <visa@hankala.org>

diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.h b/drivers/net/bonding/rte_eth_bond_8023ad.h
index 7ad8d6d00b..921b4446b7 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.h
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.h
@@ -197,10 +197,6 @@ int
 rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
 		struct rte_eth_bond_8023ad_slave_info *conf);
 
-#ifdef __cplusplus
-}
-#endif
-
 /**
  * Configure a slave port to start collecting.
  *
@@ -331,4 +327,9 @@ rte_eth_bond_8023ad_agg_selection_get(uint16_t port_id);
 int
 rte_eth_bond_8023ad_agg_selection_set(uint16_t port_id,
 		enum rte_bond_8023ad_agg_selection agg_selection);
+
+#ifdef __cplusplus
+}
+#endif
+
 #endif /* RTE_ETH_BOND_8023AD_H_ */

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] net/bonding: Fix header for C++
  2023-08-09 15:52 [PATCH] net/bonding: Fix header for C++ Visa Hankala
@ 2023-08-09 17:49 ` Tyler Retzlaff
  2023-08-14 16:51   ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Tyler Retzlaff @ 2023-08-09 17:49 UTC (permalink / raw)
  To: Visa Hankala; +Cc: dev, Chas Williams, Min Hu

On Wed, Aug 09, 2023 at 03:52:41PM +0000, Visa Hankala wrote:
> Apply C linkage to the whole header to allow use with C++.
> 
> Fixes: dc40f17a36b ("net/bonding: allow external state machine in mode 4")
> 
> Signed-off-by: Visa Hankala <visa@hankala.org>
> 

Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] net/bonding: Fix header for C++
  2023-08-09 17:49 ` Tyler Retzlaff
@ 2023-08-14 16:51   ` Ferruh Yigit
  0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2023-08-14 16:51 UTC (permalink / raw)
  To: Tyler Retzlaff, Visa Hankala; +Cc: dev, Chas Williams, Min Hu

On 8/9/2023 6:49 PM, Tyler Retzlaff wrote:
> On Wed, Aug 09, 2023 at 03:52:41PM +0000, Visa Hankala wrote:
>> Apply C linkage to the whole header to allow use with C++.
>>
>> Fixes: dc40f17a36b ("net/bonding: allow external state machine in mode 4")
>>
>> Signed-off-by: Visa Hankala <visa@hankala.org>
>>
> 
> Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>
> 

Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>

Applied to dpdk-next-net/main, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-08-14 16:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-09 15:52 [PATCH] net/bonding: Fix header for C++ Visa Hankala
2023-08-09 17:49 ` Tyler Retzlaff
2023-08-14 16:51   ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).