On 4/30/25 16:40, Simon Glass wrote:
> We don't need to include command.h just for this declaration. It
> eventually pulls in linux/byteorder which causes a conflict with exfat
> which has its own byteorder functions.
> 
> Use a forward declaration instead.
> 
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
> 
>  include/net-common.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/net-common.h b/include/net-common.h
> index 12c50e2a41e..f2613a41ad8 100644
> --- a/include/net-common.h
> +++ b/include/net-common.h
> @@ -4,7 +4,6 @@
>  #define __NET_COMMON_H__
>  
>  #include <asm/cache.h>
> -#include <command.h>
>  #include <env.h>
>  #include <hexdump.h>
>  #include <linux/if_ether.h>
> @@ -13,6 +12,8 @@
>  #include <rand.h>
>  #include <time.h>
>  
> +struct cmd_tbl;
> +
>  #define DEBUG_NET_PKT_TRACE 0        /* Trace all packet data */
>  
>  /*

Reviewed-by: Jerome Forissier <jerome.foriss...@linaro.org>

Thanks,
-- 
Jerome

Reply via email to