On Tue, Dec 03, 2019 at 06:07:57PM +0100, Heiko Schocher wrote:
Hello Fabio,
Am 03.12.2019 um 12:56 schrieb Fabio Estevam:
Hi Tom,
On Tue, Dec 3, 2019 at 12:10 AM Tom Rini trini@konsulko.com wrote:
Thanks for the rework. Can you please look at what other i.MX boards are likely fine, or perhaps more clearly if any i.MX5/6 SoCs do have thumb2 related errata that might make it a bad idea to use thumb by default ?
It looks like i.MX51 and i.MX53 have the thumb related erratum: https://www.nxp.com/docs/en/errata/IMX53CE.pdf
i.MX6 series should work fine in thumb without issues and we could try to enable thumb by default for the next cycle.
I just enabled thumb build [1] for the DM/DTS rework for the imx6 based aristainetos board ... and as I can say, U-Boot works fine on it.
There's a number of other SoCs using it too :) It really just isn't default due to the problems of early-generation ARMv7 platforms and in turn my not wanting to break them by accident. But with that in mind, a deeper dive in to a few different vendor SoCs should tell us which ones need to default to off and what can be on, and we could get better size defaults all around (but not force, let people decide if thumb2-but-smaller is quicker boot than full-but-larger).