This is the mail archive of the newlib@sourceware.org mailing list for the newlib 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]

RE: [PATCH] Add support for TIRTOS


Hi Jeff,

I tried submitting the configure.ac patch to gcc mailing list but have
not received a response yet. Is it possible to check the change into
sourceware/src first and later sync it with gcc ?

Also, my changes in config.sub are now in the GNU config project. Is
the sourcerware/src's copy of config.sub regularly sync'ed with
the config project ? If not, could you or someone on this list sync
it ?

Thanks,
Ashish

> -----Original Message-----
> From: Kapania, Ashish
> Sent: Friday, April 04, 2014 3:12 PM
> To: 'Jeff Johnston'
> Cc: newlib@sourceware.org
> Subject: RE: [PATCH] Add support for TIRTOS
> 
> Thanks Jeff. I will submit src/configure.ac changes to gcc list.
> 
> Best,
> Ashish
> 
> -----Original Message-----
> From: Jeff Johnston [mailto:jjohnstn@redhat.com]
> Sent: Friday, April 04, 2014 2:55 PM
> To: Kapania, Ashish
> Cc: newlib@sourceware.org
> Subject: Re: [PATCH] Add support for TIRTOS
> 
> Hi Ashish,
> 
> I have applied patch01 to newlib/libgloss and updated the existing TI
> license in COPYING.NEWLIB to include reference to tirtos and 2014.
> 
> For patch02, I would ask that you submit that to the gcc list.
> The configure.ac and configure files are kept in sync with gcc with
> usually the changes going to gcc first and synchronized into
> sourceware/src later.
> 
> I will be away next week, so if there are issues, either contact the
> list and Corinna will deal with them or wait until I return.
> 
> -- Jeff J.
> 
> ----- Original Message -----
> > From: "Ashish Kapania" <akapania@ti.com>
> > To: "Jeff Johnston" <jjohnstn@redhat.com>
> > Cc: newlib@sourceware.org
> > Sent: Wednesday, April 2, 2014 7:31:25 PM
> > Subject: RE: [PATCH] Add support for TIRTOS
> >
> > I attached an older incorrect patch01 file in my previous mail.
> Please
> > use the patch files attached to this mail.
> >
> > Thanks,
> > Ashish
> >
> > -----Original Message-----
> > From: Kapania, Ashish
> > Sent: Wednesday, April 02, 2014 4:10 PM
> > To: 'Jeff Johnston'
> > Cc: newlib@sourceware.org
> > Subject: RE: [PATCH] Add support for TIRTOS
> >
> > Hi Jeff,
> >
> > Thanks for your feedback. I will submit the change in src/config.sub
> > directly to the GNU config project as per the MAINTAINERS document. I
> > have updated my earlier patch file to remove the
> "may_supply_syscalls"
> > check and split it into
> > 2 separate patches. As per the MAINTAINERS doc any global maintainer
> > can approve changes to src/configure.ac so I am submitting the patch
> here.
> >
> > List of patches (attached):
> > tirtos_support.patch01 (contains newlib/ and libgloss/ changes)
> > tirtos_support.patch02 (contains src/configure.ac change)
> >
> > I would like to confirm that the changes in my patches were made by
> me
> > and I have my employer's consent to contribute these changes to the
> > newlib project.
> >
> > Best,
> > Ashish
> >
> > -----Original Message-----
> > From: Jeff Johnston [mailto:jjohnstn@redhat.com]
> > Sent: Wednesday, April 02, 2014 10:15 AM
> > To: Kapania, Ashish
> > Cc: newlib@sourceware.org
> > Subject: Re: [PATCH] Add support for TIRTOS
> >
> > Hi Ashish,
> >
> > The top-level files in src need to be submitted to a different
> mailing list.
> > Check out the top-level src/MAINTAINERS file for details.
> >
> > I have a few comments about the newlib/libgloss bits.
> >
> > You're logic in checking the may_supply_syscalls flag is incorrect.
> > The may_supply_sycalls option is designed to allow older sys dirs to
> > supply the low-level syscalls that libgloss is meant to supply.  The
> > libc/sys/arm directory is one of those.  The problem with the sys dir
> > supplying syscalls is it builds these into newlib itself and thus
> > changing board models or using a simulator can't be done by simple
> linker usage.
> > In your case, you are overriding the sys_dir directory so
> libc/sys/arm
> > won't even be used and thus there is no reason to set the
> ARM_RDI_MONITOR flag.
> > You do, however, need to set the have_crt0 flag to "no".  This is
> > because your sys directory does not create a crt0.o file which the
> > build will expect to copy up the chain.  I would have expected your
> > build to fail without you specifying --disable-newlib-may-supply-
> syscalls.
> >
> > Can you also please note whether you have written 100% of the code
> and
> > whether you have permission from your employer to donate this code to
> > newlib?
> >
> > Thanks,
> >
> > -- Jeff J.
> >
> > ----- Original Message -----
> > > From: "Ashish Kapania" <akapania@ti.com>
> > > To: newlib@sourceware.org
> > > Sent: Friday, March 28, 2014 9:52:44 PM
> > > Subject: [PATCH] Add support for TIRTOS
> > >
> > > Hi All,
> > >
> > > This is a request to apply the attached patch to newlib. This patch
> > > intends to add TIRTOS support
> > > (http://processors.wiki.ti.com/index.php/TI-RTOS) in newlib.
> > >
> > > I have not included a diff for the generated files. If that is
> > > required please let me know and I can share an updated patch.
> > >
> > > Thanks,
> > > Ashish
> > >
> > > ChangeLog:
> > >
> > > From 6d77663313dbe484a0447028bed227b2d881eaac Mon Sep 17 00:00:00
> > > 2001
> > > From: Ashish Kapania <akapania@ti.com>
> > > Date: Fri, 28 Mar 2014 18:32:04 -0700
> > > Subject: [PATCH] Add support for TIRTOS
> > >
> > > ---
> > > ChangeLogÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 5 ++ config.sub
> > > |ÂÂ 4 +- configure.acÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +-
> > > libgloss/ChangeLogÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 5 ++
> > > libgloss/arm/configure.inÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +- newlib/ChangeLog
> > > |Â 12 ++++ newlib/configure.hostÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 20 ++++++
> > > newlib/libc/stdio/local.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 2 +-
> > > newlib/libc/sys/configure.inÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 +
> > > newlib/libc/sys/tirtos/Makefile.amÂÂÂ ÂÂÂÂ|Â 16 +++++
> > > newlib/libc/sys/tirtos/configure.inÂÂÂÂÂÂ |Â 14 ++++
> > > newlib/libc/sys/tirtos/include/sys/lock.h | 109
> > > ++++++++++++++++++++++++++++++
> > > newlib/libc/sys/tirtos/lock.cÂÂÂÂÂÂÂÂÂÂÂÂ |Â 92
> > > +++++++++++++++++++++++++
> > > 13 files changed, 279 insertions(+), 5 deletions(-) create mode
> > > 100644 newlib/libc/sys/tirtos/Makefile.am
> > > create mode 100644 newlib/libc/sys/tirtos/configure.in
> > > create mode 100644 newlib/libc/sys/tirtos/include/sys/lock.h
> > > create mode 100644 newlib/libc/sys/tirtos/lock.c
> > >
> > >
> >
--- Begin Message ---
Pinging again. Requesting src/configure.ac owner to pick up my change.

Thanks,
Ashish

-----Original Message-----
From: Kapania, Ashish
Sent: Friday, April 11, 2014 3:39 PM
To: 'gcc-patches@gcc.gnu.org'
Subject: Ping - RE: PATCH] (configure.ac) Add support for TI-RTOS

Ping...

-----Original Message-----
From: Kapania, Ashish
Sent: Friday, April 04, 2014 4:53 PM
To: 'gcc-patches@gcc.gnu.org'
Subject: RE: [PATCH] (configure.ac) Add support for TI-RTOS

Forgot to attach the patch file :)

-----Original Message-----
From: Kapania, Ashish
Sent: Friday, April 04, 2014 4:50 PM
To: 'gcc-patches@gcc.gnu.org'
Subject: [PATCH] (configure.ac) Add support for TI-RTOS

Hi All,

This is a request to apply the attached patch to GCC project. I am adding support for TI-RTOS to newlib and require the attached change in configure.ac file.

Thanks,
Ashish

Change log:
From: Ashish Kapania <akapania@ti.com>
Date: Fri, 4 Apr 2014 16:34:20 -0700
Subject: [PATCH] configure.ac: Add support for TI-RTOS

---
ChangeLog    | 4 ++++
configure.ac | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)

Attachment: tirtos_support.patch
Description: tirtos_support.patch


--- End Message ---

Attachment: tirtos_support.patch02
Description: tirtos_support.patch02


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]