[U-Boot] [PATCH V3 0/11] ARM: OMAP3-DRA7: CP15 erratum workarounds and improvements

Siarhei Siamashka
2015-02-26T16:14:25Z

On Thu, 26 Feb 2015 08:50:09 -0600
Nishanth Menon <nm@ti.com> wrote:

> On Thu, Feb 26, 2015 at 1:40 AM, Siarhei Siamashka
> <siarhei.siamashka@gmail.com> wrote:
> > On Wed, 25 Feb 2015 14:55:08 -0600
> > Nishanth Menon <nm@ti.com> wrote:
> >
> >> Hi,
> >>
> >> The third incarnation of this series to address various ideas of
> >> previous V2 series. I will skip the full blurb and point to V1/V2
> >> links for the usual blurb.
> >>
> >> Changes since V2:
> >> - Added documentation revisioning info with ARM erratums
> >> - patch series are split up to address ARM erratums first followed by TI
> >> OMAP conversion
> >> - More OMAP3 now get erratum implementation - had to redo rx51 a little
> >> - I think is better helps exynos: https://patchwork.ozlabs.org/patch/443271/
> >> https://patchwork.ozlabs.org/patch/441863/
> >> - Rearranged the series to address generic ARM first followed by rest.
> >>
> >> V2: http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/213060
> >> V1: http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/212174
> >>
> >> Nishanth Menon (10):
> >> ARM: Introduce erratum workaround for 798870
> >> ARM: Introduce erratum workaround for 454179
> >> ARM: Introduce erratum workaround for 430973
> >> ARM: Introduce erratum workaround for 621766
> >> ARM: OMAP: Change set_pl310_ctrl_reg to be generic
> >> ARM: OMAP3: Rename omap3.h to omap.h to be generic as all SoCs
> >> ARM: OMAP3: Get rid of omap3_gp_romcode_call and replace with
> >> omap_smc1
> >> ARM: OMAP5 / DRA7: Setup L2 Aux Control Register with recommended
> >> configuration
> >> ARM: OMAP3: Enable workaround for ARM errata 454179, 430973, 621766
> >> ARM: OMAP3: rx51: Enable workaround for ARM errata 454179, 430973,
> >> 621766
> >>
> >> Praveen Rao (1):
> >> ARM: DRA7 / OMAP5: Add workaround for ARM errata 798870
> >
> > Could you perhaps also pick up a workaround for "725233: PLD
> > instructions executed with PLD data forwarding enabled can result
> > in a processor deadlock" for old OMAP3 (r1pX revision of Cortex-A8)?
> >
> > http://git.denx.de/?p=u-boot.git;a=commitdiff;h=041d42e789aba20296ffcde92173f100a9592880
> >
> > This used to be in U-Boot earlier, but seems to have vanished from the
> > current U-Boot code. I would probably submit a patch myself, but I
> > don't have such hardware anymore to test it.
>
> Could I have a review by - I think I screwed up r1-r3 handling in
> start.S when multiple erratas are enabled - which by some luck, I
> have'nt hit.. I should probably fix that.

Sadly, I don't have this old hardware anymore. I have lent my first
generation beagleboard to another person several years ago and have no
idea where it is now. Maybe I should try to get it back one day :)

But taking the "formal" route, I checked who is the current beagleboard
maintainer and looks like that's Tom Rini, who is already quite
conveniently in the CC ;-)

Basically, this old hardware is either maintained or not. If it is
maintained, then probably the board maintainer is the best person to
review and test the errata patches.

> I will hold off adding more erratas or changes for now and look for
> the current series getting in.. in fact, I am tempted to remove omap3
> entirely from this list for now, given the multiple platforms impacted
> and unless folks are willing to help test those, I am more than happy
> to just focus on a15 erratum in the first series which benefits my
> immediate needs and potentially helping exynos as well.

--
Best regards,
Siarhei Siamashka