On Wed, Jul 15, 2009 at 12:09 PM, pradeep
sawlani<[email protected]> wrote:
> Hi Andrew,
>
> On Tue, Jul 14, 2009 at 3:35 PM, Andrew Dennison<[email protected]> 
> wrote:
>> On Wed, Jul 15, 2009 at 6:01 AM, pradeep
>> sawlani<[email protected]> wrote:
>>> During compilation it results in error :
>>> ./arm/locore.S: Assembler messages:
>>> ../arm/locore.S:134: Error: register expected, not '#0x03' -- `orr r0,#0x03'
>>> ../arm/locore.S:141: Error: register expected, not '#0x1000' -- `add 
>>> r0,#0x1000'
>>> ../arm/locore.S:152: Error: register expected, not '#0x03' -- `orr r0,#0x03'
>>> ../arm/locore.S:179: Error: register expected, not '#0x80000000' --
>>> `add r1,#0x80000000'
>>
>> This will be a problem with your binutils, not gcc, in particular
>> arm-elf-as. Maybe the version you were using needed a command line
>> switch to accept that style of literal value?
>
> Is there any particular reason for using this kind of format instead
> of standard ?
> if you agree I can submit the patch to correct this and it makes
> instructions more readable.

Hi Pradeep,

I guess Kohsuke liked this style? I would prefer to leave these as
they are unless there is a compelling reason to change. This type of
"whitespace" change makes merging versions much more painful. I have
had to merge Prex tarballs to often to want to make this process any
harder.

>> Soulds like this libgcc.a is compiled with exception support. This
>> toolchain is probably built with multilib - maybe there is another
>> libgcc.a you should be linking against? what subdirectories are there
>> under  
>> /home/spradeep/devel/tools/arm-2008q3/bin/../lib/gcc/arm-none-linux-gnueabi/4.3.2?
> Here are the files/directories under arm-none-linux-gnueabi/4.3.2 :
> armv4t  crtbegin.o  crtbeginS.o  crtbeginT.o  crtend.o  crtendS.o
> include  include-fixed  install-tools  libgcc.a  libgcc_eh.a
> libgcov.a  thumb2

Doesn't look like there is anything useful there.

I did some reading and raise() is called because ldiv can signal for
divide by 0. You can either stub this (in the loader and kernel) or
build a toolchain that doesn't do this. Some Googling showed that
uboot uses functions like do_div() instead of the divide operator to
work around this. Seems unnecessary for prex as building a arm-elf
toolchain is quick and easy to do.

Note that raise() is available to prex user code.

[chop]

> git status shows  *.o and *.mo files which makes it difficult to make
> sense of the output.
> I have added .gitignore from Linux git tree with some modifications to
> my local prex git tree to resolve this.
> Please see the attachment for the corresponding patch.

This is not needed. My preferred approach is to configure and build
into a separate directory as per the examples I've given in some of my
emails. If you do this "git status" will _only_ show edited files,
there will be no build output to clutter up the listing. I may even
remove the ability to build into the same tree as the source, as this
just causes problems like you have seen.

I really should edit the README to reflect the current state of the
published Prex tree. I have even considered removing the HTML
documentation from the tree as I have no way of maintaining it - I
don't have the original source documents.

Also note that copying from a GPL licensed project into a BSD licensed
project isn't really the correct thing to do. I can't accept patches
like that.

Andrew

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
Prex-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/prex-devel

Reply via email to