From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Mon, 8 Jun 2015 10:26:29 +0200
Subject: [U-Boot] [PATCH v2 1/3] ARM: cache: add an empty stub function
	for invalidate/flush dcache
In-Reply-To: <5574F850.7090901@atmel.com>
References: <1433490452-26402-1-git-send-email-josh.wu@atmel.com>
 <201506051518.49587.marex@denx.de> <5574F850.7090901@atmel.com>
Message-ID: <201506081026.29339.marex@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

On Monday, June 08, 2015 at 04:05:04 AM, Josh Wu wrote:
> Hi, Marek
> 
> On 6/5/2015 9:18 PM, Marek Vasut wrote:
> > On Friday, June 05, 2015 at 09:47:30 AM, Josh Wu wrote:
> >> Since some driver like ohci, lcd used dcache functions. But some ARM
> >> cpu don't implement the invalidate_dcache_range()/flush_dcache_range()
> >> functions.
> >> 
> >> To avoid compiling errors this patch adds an weak empty stub function
> >> for all ARM cpu.
> >> 
> >> And each cpu can implement its own implementation. If not implemented
> >> by default it will use an empty function.
> >> 
> >> Signed-off-by: Josh Wu <josh.wu@atmel.com>
> >> ---
> >> 
> >> Changes in v2: new added
> >> 
> >>   arch/arm/lib/cache.c | 9 +++++++++
> >>   1 file changed, 9 insertions(+)
> > 
> > So, why exactly can't this be entirely common code , but a CPU-specific
> > code ? :)
> 
> Do you mean to make those empty functions as common code for all arch
> cpu to use?

Yes.

> It seems there is no place to put common code for all arch cpu.

Isn't that what the common/ directory is for ? ;-)

Best regards,
Marek Vasut