[U-Boot] [PATCH] arm: start.S: Fix startup of dragonboard410c

Tom Rini trini at konsulko.com
Thu Jul 14 23:33:50 CEST 2016


On Tue, Jul 12, 2016 at 11:10:28PM +0100, André Przywara wrote:
> On 12/07/16 21:15, Mateusz Kulikowski wrote:
> 
> Hi Mateusz,
> 
> > Commit d73718f3 breaks devices where:
> > - There is EL2/EL3 firmware and
> > - U-Boot starts in NS EL1 and
> > - EL2/EL3 firmware didn't unlocked write-access to CPUECTLR_EL1.
> > 
> > This patch makes that change opt-out configuration option,
> > and disables it for dragonboard410c.
> 
> thanks for bringing this up!
> You are totally right, as this register isn't architecturally defined,
> we should never set it without checking for a particular core.
> Actually even that is not enough, as it's also access restricted - even
> defaulting to RO - as you mentioned.
> 
> So I am very much for a board specific _opt-in_ solution, also as ARM
> Trusted Firmware takes care of the bit already, for instance.
> 
> Mingkai Hu, can you say for which board/SoC/family the original patch
> was needed for? Layerscape?

OK, I'm going to revert the original patch here (and would recommend
that distros that are picking up v2016.07 do the same).  Thanks all!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160714/03931d88/attachment.sig>


More information about the U-Boot mailing list