[PATCH] New target - CR16 Simulator port

[PATCH] New target - CR16 Simulator port

[M R Swami Reddy]

[-- Attachment #1: Type: text/plain, Size: 548 bytes --]

Hello,

Attached a patch which adds simulator support for CompactRISC processor - CR16 
architecture (ie cr16-elf).

Could you please review and approve the patch?

The patch file (ie cr16-simulator-port.patch.tar.bz2) contains:

1. cr16-sim.patch          -> CR16 Simulator patch
2. cr16-include.patch      -> Include directory patch
3. cr16-configure.patch    -> Top level configure patch for cr16 target
4. cr16-configure.ac.patch -> Top level configure.ac patch for cr16 target
5. cr16-ChangeLog.patch    -> Top level ChangeLog


Thanks,
Swami

[-- Attachment #2: cr16-simulator-port.patch.tar.bz2 --]
[-- Type: application/octet-stream, Size: 69102 bytes --]

Re: [PATCH] New target - CR16 Simulator port

[Daniel Jacobowitz]

On Fri, Feb 15, 2008 at 06:38:45PM +0530, M R Swami Reddy wrote:
> Hello,
>
> Attached a patch which adds simulator support for CompactRISC processor - 
> CR16 architecture (ie cr16-elf).
>
> Could you please review and approve the patch?
>
> The patch file (ie cr16-simulator-port.patch.tar.bz2) contains:
>
> 1. cr16-sim.patch          -> CR16 Simulator patch
> 2. cr16-include.patch      -> Include directory patch
> 3. cr16-configure.patch    -> Top level configure patch for cr16 target
> 4. cr16-configure.ac.patch -> Top level configure.ac patch for cr16 target
> 5. cr16-ChangeLog.patch    -> Top level ChangeLog

Why did you set testsuite=yes for v850?

Other than that, this looks OK.  Do you have write access?

-- 
Daniel Jacobowitz
CodeSourcery

Re: [PATCH] New target - CR16 Simulator port

[M R Swami Reddy]

Hi,

Daniel Jacobowitz wrote:

> Why did you set testsuite=yes for v850?
> 
> Other than that, this looks OK.  Do you have write access?

Thank you very much for the review and approval.

The line:144 @ cr16-sim.patch file (ie "+          testsuite=yes") is not 
required for cr16 simulator port. I will update it.

I don't have write access permissions. Could you please advise, to whom I can 
request to add this port to gdb source repository?

I have completed the paper work with FSF (RT 333793) on cr16 target port 
contribution. Now I will submit the maintainer file patch to get the cr16
port maintainership and write access.

Thanks,
Swami

Re: [PATCH] New target - CR16 Simulator port

[Daniel Jacobowitz]

On Mon, Mar 24, 2008 at 02:14:20PM +0530, M R Swami Reddy wrote:
> I don't have write access permissions. Could you please advise, to whom I 
> can request to add this port to gdb source repository?

Fill out the access form:
  http://sourceware.org/cgi-bin/pdw/ps_form.cgi


-- 
Daniel Jacobowitz
CodeSourcery

Re: [PATCH] New target - CR16 Simulator port

[M R Swami Reddy]

Hi,

Daniel Jacobowitz wrote:
> On Fri, Feb 15, 2008 at 06:38:45PM +0530, M R Swami Reddy wrote:
>> Hello,
>>
>> Attached a patch which adds simulator support for CompactRISC processor - 
>> CR16 architecture (ie cr16-elf).
>>
>> Could you please review and approve the patch?
>>
>> The patch file (ie cr16-simulator-port.patch.tar.bz2) contains:
>>
>> 1. cr16-sim.patch          -> CR16 Simulator patch
>> 2. cr16-include.patch      -> Include directory patch
>> 3. cr16-configure.patch    -> Top level configure patch for cr16 target
>> 4. cr16-configure.ac.patch -> Top level configure.ac patch for cr16 target
>> 5. cr16-ChangeLog.patch    -> Top level ChangeLog
> 
> Why did you set testsuite=yes for v850?
> 
> Other than that, this looks OK.  Do you have write access?

Thank you very much for review and approval.

Applied this patch.

Thanks,
Swami