From: Stephan Linz <linz@li-pro.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PREVIEW] LL TEMAC v10 refactored -- for evaluation only
Date: Sun, 29 Jan 2012 16:51:41 +0100 [thread overview]
Message-ID: <1327852301.3638.177.camel@keto> (raw)
Hi Michal,
back to your requests and objections I've built up a new patch set for
evaluation. Some of the changes relate to the Microblaze platform rather
than the network driver.
1) U-Boot configuration variables vs. xparameters.h
----------------------------------------------------------------------
I've tried to find a compromise between the U-Boot coding and
configuration style and your preference. But at least in the point of
the many many CPP conditional directives I can not reduce the code. I
can move the code only to another place. I need a converstion from the
automaticaly generated content of xparameters.h to the U-Boot
configuration space. Now you can find this conversion matrix in a new
platform header, xconversions.h -- I've placed the header in
board/xilinx/microblaze-generic/ I think, we can find a better place.
The conversion matrix avoids all the wrong configuration variables
inside U-Boot code beginning with XILINX_ (and not CONFIG_) and we can
expand the configuration to new software related options we never get
from hardware (example special interface naming). This mechanism can
also be used by other driver and platform code, for example
arch/microblaze/cpu/cache.c and arch/microblaze/lib/bootm.c (use
XILINX_USE_DCACHE instead of ex. CONFIG_XILINX_DCACHE),
common/cmd_mfsl.c (use XILINX_FSL_NUMBER instead of ex.
CONFIG_XILINX_FSL), serial_xuartlite.c (use XILINX_UARTLITE_BASEADDR,
XILINX_UARTLITE_BASEADDR1, ...), and probably also in xilinx_emaclite.c
and xilinx_axi_emac.c when we extand a multi-IP support as we have done
for the LL TEMAC now. Sure we have to work on, but it's another case (I
can make some patches if you want)
Another but not currently existing feasibility would be to generate a
U-Boot conform xparameters.h, for example
CONFIG_XILINX_LL_TEMAC0_BASE_ADDR instead of XILINX_LLTEMAC_BASEADDR.
You and I, we have it in hand. We can change the Xilinx BSP in the near
future. You in PataLinux and I in my own home-brewed BSP generator TPOS.
2) Standard initialization vs. multiple direct calls from platform
----------------------------------------------------------------------
Returning to the topic "standard" initialization. The last patch will
show the differences bwetween your prefered way to do all in board init
and my applicable standard initializition (a single call to the driver).
Please look at it in the code and decide for yourself.
We need a forward-looking decision, what kind of initialization, we want
to use. Do we want to copy and paste the standard initialization steps
from ip core to ip core (board init) and from platform to platform
(Microplace, PPC405, PPC440) or make a simple call to the driver?
My preferences is to call the driver from platform and let him run all
the necessary steps, that I do not want knowing on platform side.
--
Best regards,
Stephan Linz
______________________________________________________________________________
MB-Ref: http://www.li-pro.de/xilinx_mb:mbref:start
OpenDCC: http://www.li-pro.net/opendcc.phtml
PC/M: http://www.li-pro.net/pcm.phtml
Sourceforge: http://sourceforge.net/users/slz
Gitorious: https://gitorious.org/~slz
next reply other threads:[~2012-01-29 15:51 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-29 15:51 Stephan Linz [this message]
2012-01-29 15:54 ` [U-Boot] [PATCH 1/8] net: ll_temac: Add LL TEMAC driver to u-boot Stephan Linz
2012-02-23 11:57 ` Michal Simek
2012-01-29 15:54 ` [U-Boot] [PATCH 2/8] microblaze: Enable several ethernet driver compilation Stephan Linz
2012-01-31 9:00 ` Michal Simek
2012-01-31 19:05 ` Stephan Linz
2012-02-23 11:49 ` Michal Simek
2012-01-29 15:54 ` [U-Boot] [PATCH 3/8] microblaze: Enable phylib and mii support Stephan Linz
2012-02-23 11:52 ` Michal Simek
2012-01-29 15:54 ` [U-Boot] [PATCH 4/8] microblaze: Enable TFTP put command Stephan Linz
2012-02-23 11:51 ` Michal Simek
2012-01-29 15:54 ` [U-Boot] [PATCH 5/8] microblaze: Xilinx BSP to U-Boot conversion matrix Stephan Linz
2012-01-29 15:54 ` [U-Boot] [PATCH 6/8] microblaze: Add faked LL_TEMAC driver configuration Stephan Linz
2012-01-31 9:01 ` Michal Simek
2012-01-31 19:27 ` Stephan Linz
2012-02-23 11:53 ` Michal Simek
2012-01-29 15:54 ` [U-Boot] [PATCH 7/8] microblaze: Wire up LL_TEMAC driver initialization Stephan Linz
2012-01-29 15:54 ` [U-Boot] [PATCH 8/8] microblaze: ll_temac: export xilinx_ll_temac_eth_init() for evaluation Stephan Linz
2012-02-23 11:56 ` Michal Simek
2012-02-23 19:02 ` Stephan Linz
2012-02-23 12:22 ` [U-Boot] [PREVIEW] LL TEMAC v10 refactored -- for evaluation only Michal Simek
2012-02-23 19:39 ` Stephan Linz
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=1327852301.3638.177.camel@keto \
--to=linz@li-pro.net \
--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