From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 12 Dec 2007 11:24:14 +0100
Subject: [U-Boot-Users] [PATCH 6/6] cfi_flash: Use map_physmem() and
	unmap_physmem()
In-Reply-To: <20071212103632.5e0a0f97@dhcp-252-066.norway.atmel.com>
References: <1197386900-14570-1-git-send-email-hskinnemoen@atmel.com>
	<200712111949.27205.sr@denx.de>
	<20071212103632.5e0a0f97@dhcp-252-066.norway.atmel.com>
Message-ID: <200712121124.15239.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

On Wednesday 12 December 2007, Haavard Skinnemoen wrote:
> > When starting something like map_physmem(), we should consider the
> > following things:
> >
> > - Some platforms have >32bit physical address space, like PPC440 with
> > 36bits. So the type of phys_addr should not be fixed to u32. We need a
> > new typedef here. And/or perhaps something like the resource stuff from
> > Linux.
>
> Yeah, we probably need a typedef. How about phys_addr_t or something?

ACK.

> I can cook up a patch adding phys_addr_t typedef'd as unsigned long on
> all architectures. If someone wants something different, please let me
> know.

Yes, start with "unsigned long" and the platform maintainer can fix this later 
if needed. This way we can get this started right now.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================