Am Dienstag, den 31.01.2012, 10:00 +0100 schrieb Michal Simek:
Stephan Linz wrote:
U-Boot's multipple network supports enables to use several ethernet drivers but microblaze-generic platform config file select only one driver.
Reported-by: Michal Simek monstr@monstr.eu Signed-off-by: Stephan Linz linz@li-pro.net
include/configs/microblaze-generic.h | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index 03a6f5a..09ac870 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -65,11 +65,15 @@
/* ethernet */ #undef CONFIG_SYS_ENET -#ifdef XILINX_EMACLITE_BASEADDR -# define CONFIG_XILINX_EMACLITE 1 +#if defined(XILINX_EMACLITE_BASEADDR) +# define CONFIG_XILINX_EMACLITE 1 # define CONFIG_SYS_ENET -#elif XILINX_LLTEMAC_BASEADDR -# define CONFIG_XILINX_LL_TEMAC 1 +#endif +#if defined(XILINX_LLTEMAC_BASEADDR) || defined(XILINX_LLTEMAC_BASEADDR1) || \
- defined(XILINX_LLTEMAC_BASEADDR2) || defined(XILINX_LLTEMAC_BASEADDR3) || \
- defined(XILINX_LLTEMAC_BASEADDR4) || defined(XILINX_LLTEMAC_BASEADDR5) || \
- defined(XILINX_LLTEMAC_BASEADDR6) || defined(XILINX_LLTEMAC_BASEADDR7)
+# define CONFIG_XILINX_LL_TEMAC 1
Why to do it in this way? I believe that the first lltemac driver will have XILINX_LLTEMAC_BASEADDR and then others.
Right, but we need a definitive agreement that it is so (Xilinx BSP generator). Did we them? If yes I will drop this part and insert your original condition (test only XILINX_LLTEMAC_BASEADDR).
But for all configurations if there is lltemac driver then will have anyone just BASEADDR.
yep
br, Stephan