On 01/20/13 20:36, Matthias Klose wrote:
Am 18.01.2013 15:28, schrieb Ramana Radhakrishnan:
On 06/20/12 03:53, Yi-Hsiu Hsu wrote:
marvell-pj4 is added to BE8_LINK_SPEC.

Sorry about the time it's taken to finish this patch up. I seem to have missed
this one in the review process.

I've now applied the attached patch after taking into account the recent
attribute changes and treated alu_reg and simple_alu_imm as you have treated
alu, rebased to trunk to fix up some small issues with BE8_LINK_SPECS.

Additionally I've removed tune_marvell as that seems redundant at this point of
time - an additional attribute and testing that appears to be unnecessary
instead of just one more inequality test.

The only part I'm not sure about is how to treat the simple_alu_shift category
here , so a patch to handle them in the pj4 pipeline description would be 
welcome.


Thanks
Ramana

2013-01-18  Yi-Hsiu Hsu  <a...@marvell.com>
             Ramana Radhakrishnan  <ramana.radhakrish...@arm.com>

         * config/arm/marvell-pj4.md: New file.
         * config/arm/arm.c (arm_issue_rate): Add marvell_pj4.
         * config/arm/arm.md (generic_sched): Add marvell_pj4.
         (generic_vfp): Likewise.
         * config/arm/arm-cores.def: Add marvell-pj4.
         * config/arm/arm-tune.md: Regenerate.
         * config/arm/arm-tables.opt: Regenerate.
         * config/arm/bpabi.h (BE8_LINK_SPEC): Add marvell_pj4.
         * doc/invoke.texi: Document marvell-pj4.


Modified patch is attached.

the patch was missing the new file config/arm/marvell-pj4.md. Checked is as
obvious to restore the bootstrap.

Thanks - Looks like I missed adding that. Uggh.

Ramana




Reply via email to