[PATCH 1/5] ARM: phyCORE-AM335x: Updated SPI NOR dts entries

[Sascha Hauer]

On Fri, Sep 05, 2014 at 08:32:00AM +0200, Alexander Aring wrote:
> Hi Sascha,
> 
> On Fri, Sep 05, 2014 at 07:37:33AM +0200, Sascha Hauer wrote:
> > Hi Teresa,
> > 
> > On Thu, Sep 04, 2014 at 08:50:36AM +0200, Teresa Gámez wrote:
> > > Updated the SPI NOR flash entries. NOR flash got
> > > detected but did not work.
> > > 
> > > - Updated muxing
> > > - Fixed frequency
> > > - Fixed CS
> > > - Removed first compatible entry (the flashes used is changing frequently)
> > > 
> > > Signed-off-by: Teresa Gámez <t.gamez at phytec.de>
> > 
> > Applied up to 4/5. The Root partition start seems indeed wrong.
> > 
> 
> yea. But sometimes you also do a "fixed while applying". Why not now?
> Okay, maybe on the "good" days.

Maybe because you didn't write the correct start offset and I didn't
feel like doing maths before the first coffee ;)

> 
> Nevertheless, isn't there any mechanism to check on conflicts while
> creating partitions like request memory mapped regions?

No. Right now creating multiple conflicting partitions is a feature, see
for example /dev/env0 and /dev/nor0.bareboxenv. Those are two
partitions which both conflict but must both exist.

I must admit it's a misfeature though and I wish we could do without it
as it really can hide problems.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |