[PATCH, V850] Add libgloss support for V850

Jeff Johnston jjohnstn@redhat.com
Wed Jun 16 13:24:00 GMT 2010


A few questions and some comments.  I cc'd Nick and DJ as they have 
performed some of the latest maintenance on sysnecv850 files and might 
have something to say about the proposed move to libgloss.

1. Why is v850ea support being removed?  Top-level files don't belong
    to newlib.  See the top-level MAINTAINERS file for info on
    who owns what files or what mailing lists need to be notified.

2. Does the gcc list know you are planning on removing v850 from newlib
    and is there a corresponding change that is occurring?
    The reason I mention this is that without a change to gcc, any v850
    users will start seeing link errors because they used to get syscalls
    thrown in when they linked in newlib.  Now linking requires
    additional reference to the new linker script sim.ld.

3. You don't need to remove libnosys support.

4. It would be good to mention that you are moving newlib files to
    libgloss in the ChangeLog entry.  Helps for tracking history.

5. Are you aware that it is possible to disable the newlib sys library
    as opposed to removing it outright via a configuration option:
    --disable-newlib-supplied-syscalls?  This allows a transition period
    if one needs to get compiler support in place without breaking
    things that assume that the syscalls will be automatically linked in.

6. You are calling your bsp library libsim.a and linker script sim.ld.
    Can it be confirmed that the moved code was targeted to run on a
    simulator vs a real board?

-- Jeff J.

On 06/08/2010 12:50 AM, Naveen H. S wrote:
> Hi,
>
> Please find attached the "v850-newlib-v850-libgloss-support.patch".
> The patch adds libgloss support for V850 target.
>
> Please review the patch and let me know if there should be any modifications in it.
>
> 2010-06-08  Naveen.H.S  naveenh1@kpitcummins.com
> 		Gina Verlekar  gina.verlekar@kpitcummins.com
>
>        * configure: Remove support for V850ea.
>        * configure.ac: Likewise.
> 	* libgloss/configure: Add libgloss support for V850.
> 	* libgloss/configure.in: Likewise.
> 	* libgloss/libnosys/configure.in: Remove support for V850
> 	* libgloss/libnosys/configure: Likewise
> 	* libgloss/v850/access.c: New file.
> 	* libgloss/v850/aclocal.m4: New.
> 	* libgloss/v850/chmod.c: New.
> 	* libgloss/v850/chown.c: New.
> 	* libgloss/v850/close.c: New.
> 	* libgloss/v850/configure: New.
> 	* libgloss/v850/configure.in: New.
> 	* libgloss/v850/crt0.S: New.
> 	* libgloss/v850/crt1.c: New.
> 	* libgloss/v850/execv.c: New.
> 	* libgloss/v850/execve.c: New.
> 	* libgloss/v850/_exit.c: New.
> 	* libgloss/v850/fork.c: New.
> 	* libgloss/v850/fstat.c: New.
> 	* libgloss/v850/getpid.c: New.
> 	* libgloss/v850/gettime.c: New.
> 	* libgloss/v850/isatty.c: New.
> 	* libgloss/v850/kill.c: New.
> 	* libgloss/v850/link.c: New.
> 	* libgloss/v850/lseek.c: New.
> 	* libgloss/v850/Makefile.in: New.
> 	* libgloss/v850/open.c: New.
> 	* libgloss/v850/pipe.c: New.
> 	* libgloss/v850/read.c: New.
> 	* libgloss/v850/sbrk.c: New.
> 	* libgloss/v850/sim.ld: New.
> 	* libgloss/v850/stat.c: New.
> 	* libgloss/v850/time.c: New.
> 	* libgloss/v850/times.c: New.
> 	* libgloss/v850/trap.S: New.
> 	* libgloss/v850/unlink.c: New.
> 	* libgloss/v850/utime.c: New.
> 	* libgloss/v850/wait.c: New.
> 	* libgloss/v850/write.c: New.
> 	* newlib/configure.host: Remove V850 support.
> 	* newlib/libc/sys/configure.in: Remove v850 support.
> 	* newlib/libc/sys/sysnecv850/access.c: Remove.
> 	* newlib/libc/sys/sysnecv850/aclocal.m4: Remove.
> 	* newlib/libc/sys/sysnecv850/chmod.c: Remove.
> 	* newlib/libc/sys/sysnecv850/chown.c: Remove.
> 	* newlib/libc/sys/sysnecv850/close.c: Remove.
> 	* newlib/libc/sys/sysnecv850/configure: Remove.
> 	* newlib/libc/sys/sysnecv850/configure.in: Remove.
> 	* newlib/libc/sys/sysnecv850/creat.c: Remove.
> 	* newlib/libc/sys/sysnecv850/crt0.S: Remove.
> 	* newlib/libc/sys/sysnecv850/crt1.c: Remove.
> 	* newlib/libc/sys/sysnecv850/execv.c: Remove.
> 	* newlib/libc/sys/sysnecv850/execve.c: Remove.
> 	* newlib/libc/sys/sysnecv850/_exit.c: Remove.
> 	* newlib/libc/sys/sysnecv850/fork.c: Remove.
> 	* newlib/libc/sys/sysnecv850/access.c: Remove.
> 	* newlib/libc/sys/sysnecv850/fstat.c: Remove.
> 	* newlib/libc/sys/sysnecv850/getpid.c: Remove.
> 	* newlib/libc/sys/sysnecv850/gettime.c: Remove.
> 	* newlib/libc/sys/sysnecv850/isatty.c: Remove.
> 	* newlib/libc/sys/sysnecv850/kill.c: Remove.
> 	* newlib/libc/sys/sysnecv850/link.c: Remove.
> 	* newlib/libc/sys/sysnecv850/lseek.c: Remove.
> 	* newlib/libc/sys/sysnecv850/Makefile.am: Remove.
> 	* newlib/libc/sys/sysnecv850/Makefile.in: Remove.
>
> Thanks&  Regards,
> Naveen.H.S



More information about the Newlib mailing list