public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] xilinx drivers
Date: Mon, 26 Mar 2007 08:28:10 +0200	[thread overview]
Message-ID: <200703260828.11757.sr@denx.de> (raw)
In-Reply-To: <3275.5961-16246-190946868-1174889434@seznam.cz>

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
=====================================================================

  reply	other threads:[~2007-03-26  6:28 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-26  6:10 [U-Boot-Users] xilinx drivers mONStR
2007-03-26  6:28 ` Stefan Roese [this message]
2007-03-26  6:36   ` mONStR
2007-03-26  6:39   ` Grant Likely
2007-03-26  7:12   ` Wolfgang Denk
2007-03-26  7:20     ` Stefan Roese
2007-03-26  7:57       ` Monstr at seznam.cz
2007-03-26 11:43         ` Wolfgang Denk
2007-03-26 19:01           ` Michal Simek
2007-03-26  8:23     ` Monstr at seznam.cz
2007-03-26 11:46       ` Wolfgang Denk
2007-03-26 15:36   ` Grant Likely
2007-03-26 19:35     ` Michal Simek
2007-03-26  7:09 ` Wolfgang Denk
2007-03-26 14:59 ` Grant Likely

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200703260828.11757.sr@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox