diff mbox

[10/12,v9] target-tilegx: Add TILE-Gx building files

Message ID BLU436-SMTP21C98F396FCFB064545E40B9090@phx.gbl
State New
Headers show

Commit Message

Chen Gang March 27, 2015, 11 a.m. UTC 
Add related configuration, make files for tilegx. Now, qemu tilegx can
pass building.

Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
 configure                             | 3 +++
 default-configs/tilegx-linux-user.mak | 1 +
 target-tilegx/Makefile.objs           | 1 +
 3 files changed, 5 insertions(+)
 create mode 100644 default-configs/tilegx-linux-user.mak
 create mode 100644 target-tilegx/Makefile.objs

Comments

Peter Maydell April 9, 2015, 10:10 p.m. UTC | #1 
On 27 March 2015 at 11:00, Chen Gang <xili_gchen_5257@hotmail.com> wrote:
> Add related configuration, make files for tilegx. Now, qemu tilegx can
> pass building.
>
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
> ---
>  configure                             | 3 +++
>  default-configs/tilegx-linux-user.mak | 1 +
>  target-tilegx/Makefile.objs           | 1 +
>  3 files changed, 5 insertions(+)
>  create mode 100644 default-configs/tilegx-linux-user.mak
>  create mode 100644 target-tilegx/Makefile.objs
>
> diff --git a/configure b/configure
> index 589798e..a8704ec 100755
> --- a/configure
> +++ b/configure
> @@ -5215,6 +5215,9 @@ case "$target_name" in
>    s390x)
>      gdb_xml_files="s390x-core64.xml s390-acr.xml s390-fpr.xml"
>    ;;
> +  tilegx)
> +    TARGET_ARCH=tilegx
> +  ;;

TARGET_ARCH already defaults to the target name, so you don't
need to set it again unless it's different. You can just have
an empty case here, like unicore32 does immediately below:

>    unicore32)
>    ;;
>    xtensa|xtensaeb)

-- PMM
Chen Gang April 10, 2015, 9:11 p.m. UTC | #2 
On 4/10/15 06:10, Peter Maydell wrote:
> On 27 March 2015 at 11:00, Chen Gang <xili_gchen_5257@hotmail.com> wrote:

[...]
>>    ;;
>> +  tilegx)
>> +    TARGET_ARCH=tilegx
>> +  ;;
> 
> TARGET_ARCH already defaults to the target name, so you don't
> need to set it again unless it's different. You can just have
> an empty case here, like unicore32 does immediately below:
> 
>>    unicore32)
>>    ;;
>>    xtensa|xtensaeb)
> 

OK, thanks.
diff mbox

Patch

diff --git a/configure b/configure
index 589798e..a8704ec 100755
--- a/configure
+++ b/configure
@@ -5215,6 +5215,9 @@  case "$target_name" in
   s390x)
     gdb_xml_files="s390x-core64.xml s390-acr.xml s390-fpr.xml"
   ;;
+  tilegx)
+    TARGET_ARCH=tilegx
+  ;;
   unicore32)
   ;;
   xtensa|xtensaeb)
diff --git a/default-configs/tilegx-linux-user.mak b/default-configs/tilegx-linux-user.mak
new file mode 100644
index 0000000..3e47493
--- /dev/null
+++ b/default-configs/tilegx-linux-user.mak
@@ -0,0 +1 @@ 
+# Default configuration for tilegx-linux-user
diff --git a/target-tilegx/Makefile.objs b/target-tilegx/Makefile.objs
new file mode 100644
index 0000000..8b3dc76
--- /dev/null
+++ b/target-tilegx/Makefile.objs
@@ -0,0 +1 @@ 
+obj-y += cpu.o translate.o helper.o