From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 21 Oct 2009 11:05:09 +0200
Subject: [U-Boot] [PATCH v2] ppc4xx: Update flash size in reg property
	of the NOR flash node
In-Reply-To: <20091020211247.DBCB91A00B@gemini.denx.de>
References: <1256048896-10508-1-git-send-email-sr@denx.de>
	<20091020211247.DBCB91A00B@gemini.denx.de>
Message-ID: <200910211105.10055.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

Hi Wolfgang,

On Tuesday 20 October 2009 23:12:47 Wolfgang Denk wrote:
> > Till now only the ranges in the ebc node are updated with the values
> > currently configured in the PPC4xx EBC controller. With this patch now
> > the NOR flash size is updated in the device tree blob as well. This is
> > done by scanning the compatible nodes "cfi-flash" and "jedec-flash"
> > for the correct chip select number.
> >
> > Signed-off-by: Stefan Roese <sr@denx.de>
> > Cc: Wolfgang Denk <wd@denx.de>
> > ---
> > Changes in v2:
> > - NOR flash nodes are now scanned/detected via the compatible node.
> 
> Thanks.
> 
> Hm... thinking about it, this problem most probably affects other
> (non-4xx boards as well). I guess there is no easy way to generalize
> this code enough tomake it usable by other (or even all?) boards as
> well?

I see no way to easily fix the NOR flash "reg" property for all non-4xx boards 
as well. But what I can do is, move the fdt_update_nor_flash_node() function 
into some common code (commond/fdt_support.c). This way it could be used by 
other boards/platforms as well.

Should I send an updated patch for this?

Cheers,
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