This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
| Other format: | [Raw text] | |
On Thu, Apr 07, 2005 at 03:22:17PM +0100, David Vrabel wrote:
> Andrew Lunn wrote:
> > Attached is a patch from Ian Campbell which adds the exec command to
> > Redboot on pc platforms. I've reworked the patch a bit.
>
> I've just resync'd our local CVS repo with upstream and I noticed
> something I think is wrong with this patch.
>
> > --- /dev/null
> > +++ hal/i386/pc/current/src/redboot_linux_exec.c
>
> I think this file is in the wrong place as it means it's not usable by
> anything other than the pc target. We have a number of different i386
> targets that use this.
>
> Ian had this file in hal/i386/arch/current/src which would match other
> architectures (e.g., arm and powerpc). Similarly the CDL change would go
> in hal/i386/arch/current/cdl/hal_i386.cdl
Yes, you are right. I have moved this code over to the arch directory.
Andrew
Attachment:
exec.diff
Description: Text document
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |