Re: [PATCH] ARM: Runtime patch udiv/sdiv instructions into __aeabi_{u}idiv()

Nicolas Pitre
2015-12-12T00:01:03Z

On Fri, 11 Dec 2015, Arnd Bergmann wrote:

> On Friday 11 December 2015 22:34:16 Russell King - ARM Linux wrote:
> >
> > __v7_pj4b_proc_info:
> > - .long 0x562f5840
> > - .long 0xfffffff0
> > + .long 0x560f5800
> > + .long 0xff0fff00
> >
> > So it was to include Armada 370. So this now brings up the question...
> > what is the MIDR value used in Armada 370?
>
> I've listed them in an earlier thread, here is the list again:
>
> variant part revision name features
> mmp2: 0 0x581 5 PJ4 idivt
> dove: 0 0x581 5 PJ4 idivt
> Armada 370 1 0x581 1 PJ4B idivt
> mmp3: 2 0x584 2 PJ4-MP idiva idivt lpae
> Armada XP 2 0x584 2 PJ4-MP idiva idivt lpae
> Berlin 2 0x584 2 PJ4-MP idiva idivt lpae
>
> So the original table was wrong because it failed to include PJ4B (Armada 370),
> but the current version is wrong, because it also includes PJ4 (Dove and MMP2).

I'd suggest you add the above table and conclusion to the commit log for
your proposed fix. Next time the question comes up the info will be
right there.

Nicolas