Re: [PATCH 1/6] bus/dpaa: fix the curly braces mismatch problem

[Bruce Richardson <bruce.richardson@intel.com>]

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.jain@nxp.com
> 
> Signed-off-by: Weiguo Li <liwg06@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