On Sun, Feb 06, 2022 at 08:44:09AM +0800, Weiguo Li wrote:
> Supplement the missing half of braces for the extern "C" block
>
> Fixes: 6d6b4f49a155 ("bus/dpaa: add FMAN hardware operations")
> Fixes: 919eeaccb2ba ("bus/dpaa: introduce NXP DPAA bus driver skeleton")
> Cc: shreyansh.j...@nxp.com
>
> Signed-off-by: Weiguo Li <liw...@foxmail.com>
> ---
> drivers/bus/dpaa/include/fsl_fman.h | 5 +++++
> drivers/bus/dpaa/rte_dpaa_bus.h | 4 ++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/drivers/bus/dpaa/include/fsl_fman.h
> b/drivers/bus/dpaa/include/fsl_fman.h
> index acb344584f..a10845f0a0 100644
> --- a/drivers/bus/dpaa/include/fsl_fman.h
> +++ b/drivers/bus/dpaa/include/fsl_fman.h
> @@ -186,4 +186,9 @@ static inline void fman_if_disable_all_rx(void)
> list_for_each_entry(__if, fman_if_list, node)
> fman_if_disable_rx(__if);
> }
> +
> +#ifdef __cplusplus
> +}
> +#endif
> +
> #endif /* __FSL_FMAN_H */
Judging from the filename, this looks to be an internal-only header. Does
it really need the c++ guards?
/Bruce
