Comment 9 for bug 716627

On Tue, Sep 13, 2011 at 4:14 AM, Mattias Backman
<email address hidden> wrote:
> On Mon, Sep 12, 2011 at 5:11 PM, James Westby <email address hidden> wrote:
>> On Mon, 12 Sep 2011 14:51:41 -0000, Ricardo Salveti <email address hidden> wrote:
>>> With branch lp:~mabac/linaro-image-tools/igep-fix this error is gone,
>>> but then I noticed that the MLO is also not copied to the first
>>> partition, probably because IgepConfig defines it's own
>>> _make_boot_files, without calling install_omap_boot_loader.
>>
>> With hwpack v2 the IgepConfig class shouldn't be used, so that code
>> shouldn't be triggered, suggesting that the bug is somewhere else.
>>
>> At least, that's my understanding of what should be happening :-)
>
> That's what should be happening, yes. That will be the reality RSN. ;)
> I forgot that I identified one more thing that needs work which is
> exactly this: "Consolidate _make_boot_files() which is still board
> specific: TODO". Initially I thought this was semi trivial but I
> must've missed that IgepConfig does not look like the others.

I don't mind fixing the Igep support even for V1, if that helps
getting it to work with V2, but the only requirement is that it should
still be able to work with previous hwpacks (copying x-loader/u-boot
should not be mandatory).