From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Mon, 26 Mar 2007 08:28:10 +0200
Subject: [U-Boot-Users] xilinx drivers
In-Reply-To: <3275.5961-16246-190946868-1174889434@seznam.cz>
References: <3275.5961-16246-190946868-1174889434@seznam.cz>
Message-ID: <200703260828.11757.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Michal,

On Monday 26 March 2007 08:10, mONStR wrote:
> I would like to add xilinx drivers xilinx_enet and xilinx_iic to drivers
> (drivers/xilinx/xilinx_enet and xilinx_iic) folder

I would prefer something like:

drivers/net/xilinx_enet/*
drivers/i2c/xilinx_iic/*

> and change ifdef label 
> because I use this drivers for Microblaze processor and this drivers are
> used for ppc405 embedded core like ml300.
>
> Label as #if defined(XPAR_EMAC_0_DEVICE_ID) is board dependent and I would
> like to remove this define from config files for Microblaze boards.

But please make sure, that this file is only included in the targets that 
really need it.

> and then clean Makefiles...
>
> CFLAGS   += -I../ml300 -I../common -I../xilinx_enet -I../xilinx_iic
>
> OBJS	= $(BOARD).o \
> 	  serial.o \
> 	  ../xilinx_enet/emac_adapter.o  ../xilinx_enet/xemac.o \
> 	  ../xilinx_enet/xemac_options.o ../xilinx_enet/xemac_polled.o \
> 	  ../xilinx_enet/xemac_intr.o ../xilinx_enet/xemac_g.o \
> 	  ../xilinx_enet/xemac_intr_dma.o ../xilinx_iic/iic_adapter.o \
> 	  ../xilinx_iic/xiic_l.o ../common/xipif_v1_23_b.o \
> 	  ../common/xbasic_types.o ../common/xdma_channel.o \
> 	  ../common/xdma_channel_sg.o ../common/xpacket_fifo_v1_00_b.o \
> 	  ../common/xversion.o \

You are talking about the current file "board/xilinx/ml300/Makefile", right?

> Is it possible do this changes?

Yes, this is possible to change. This could definitely needs some cleanup.

Maybe Grant wants to add some remarks here? Grant?

> And after that add support for other xilinx peripherals as SPI, EMACLITE,
> etc.
>
> Isn't it problem with Xilinx licence?

I doesn't see to be. But I am not a lawyer.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, HRB 165235 Munich, CEO: Wolfgang Denk
Office:  Kirchenstr. 5,       D-82194 Groebenzell,            Germany
=====================================================================