From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephan Linz <linz@li-pro.net>
Date: Tue, 29 Nov 2011 20:55:58 +0100
Subject: [U-Boot] [PATCH v6] net: ll_temac: Add LL TEMAC driver to u-boot
In-Reply-To: <20111127222939.C78671FFB3AD@gemini.denx.de>
References: <1322419397-26326-1-git-send-email-linz@li-pro.net>
	<1322419397-26326-2-git-send-email-linz@li-pro.net>
	<20111127190930.0186D1FFB3AB@gemini.denx.de>
	<1322424370.3870.370.camel@keto>
	<20111127222939.C78671FFB3AD@gemini.denx.de>
Message-ID: <1322596559.22856.7.camel@keto>
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

Am Sonntag, den 27.11.2011, 23:29 +0100 schrieb Wolfgang Denk: 
> Dear Stephan Linz,
> 
--snip-- 
> 
> > This special 'DCR' function can map into indirect DCR access. But
> > unfortunately in_be32() and out_be32() for Microblaze are not real
> > functions. They are CPP defines :-(
> 
> Change it?  Turning them into inline functions should be not too
> difficult.

Hello Wolfgang,
Hello Michal,

I have sync arch/microblaze/include/asm/io.h with Linux kernel code. Now
I ask me whether there is any option to test all the io helper functions
in a simple way? Do you know one?

Best regards,
Stephan

> 
> > I will review arch/microblaze/include/asm/io.h together with Michal.
> > Until then it would be nice if we could keep this code as it is. I will
> > fix the dublicate code if we have in/out functions -- no defines.
> 
> Hm... past experience has shown that the stragey of adding bad code
> first and then cleaning it up has never worked really well, nor fast. 
> Insisting on clean patches from the beginning is probably more
> painful, but almost always faster, and more reliable.
> 
> Best regards,
> 
> Wolfgang Denk
> 

-- 
Viele Gr??e,
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