[Qemu-trivial] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Qemu-trivial] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Hu Tao]

target_phys_addr_t has been already replaced by hwaddr, but this
one is introduced after.

Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
---
 target-moxie/helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target-moxie/helper.c b/target-moxie/helper.c
index 6e0ac2a..5cfe889 100644
--- a/target-moxie/helper.c
+++ b/target-moxie/helper.c
@@ -116,7 +116,7 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
     return 1;
 }
 
-target_phys_addr_t cpu_get_phys_page_debug(CPUState *env, target_ulong addr)
+hwaddr cpu_get_phys_page_debug(CPUState *env, target_ulong addr)
 {
     return addr;
 }
-- 
1.8.1.4

[Qemu-devel] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Hu Tao]

target_phys_addr_t has been already replaced by hwaddr, but this
one is introduced after.

Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
---
 target-moxie/helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target-moxie/helper.c b/target-moxie/helper.c
index 6e0ac2a..5cfe889 100644
--- a/target-moxie/helper.c
+++ b/target-moxie/helper.c
@@ -116,7 +116,7 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
     return 1;
 }
 
-target_phys_addr_t cpu_get_phys_page_debug(CPUState *env, target_ulong addr)
+hwaddr cpu_get_phys_page_debug(CPUState *env, target_ulong addr)
 {
     return addr;
 }
-- 
1.8.1.4

Re: [Qemu-trivial] [Qemu-devel] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Peter Maydell]

On 15 May 2013 07:04, Hu Tao <hutao@cn.fujitsu.com> wrote:
> target_phys_addr_t has been already replaced by hwaddr, but this
> one is introduced after.
>
> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

This doesn't cause compile errors because it's in a bit
of code that can never get compiled (it's inside a
CONFIG_USER_ONLY block but there is no moxie-linux-user
target in configure).

thanks
-- PMM

Re: [Qemu-devel] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Peter Maydell]

On 15 May 2013 07:04, Hu Tao <hutao@cn.fujitsu.com> wrote:
> target_phys_addr_t has been already replaced by hwaddr, but this
> one is introduced after.
>
> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

This doesn't cause compile errors because it's in a bit
of code that can never get compiled (it's inside a
CONFIG_USER_ONLY block but there is no moxie-linux-user
target in configure).

thanks
-- PMM

Re: [Qemu-trivial] [Qemu-devel] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Andreas Färber]

Anthony G.,

Am 15.05.2013 10:23, schrieb Peter Maydell:
> On 15 May 2013 07:04, Hu Tao <hutao@cn.fujitsu.com> wrote:
>> target_phys_addr_t has been already replaced by hwaddr, but this
>> one is introduced after.
>>
>> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> 
> This doesn't cause compile errors because it's in a bit
> of code that can never get compiled (it's inside a
> CONFIG_USER_ONLY block but there is no moxie-linux-user
> target in configure).

A similar issue had come up before and I don't remember seeing a clear
answer:

Are you planning to actually implement moxie-linux-user and enable it by
default, so that we can catch any errors? Or should we rather drop any
such unused CONFIG_USER_ONLY code from target-moxie?

Regards,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Andreas Färber]

Anthony G.,

Am 15.05.2013 10:23, schrieb Peter Maydell:
> On 15 May 2013 07:04, Hu Tao <hutao@cn.fujitsu.com> wrote:
>> target_phys_addr_t has been already replaced by hwaddr, but this
>> one is introduced after.
>>
>> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> 
> This doesn't cause compile errors because it's in a bit
> of code that can never get compiled (it's inside a
> CONFIG_USER_ONLY block but there is no moxie-linux-user
> target in configure).

A similar issue had come up before and I don't remember seeing a clear
answer:

Are you planning to actually implement moxie-linux-user and enable it by
default, so that we can catch any errors? Or should we rather drop any
such unused CONFIG_USER_ONLY code from target-moxie?

Regards,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-trivial] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Michael Tokarev]

15.05.2013 10:04, Hu Tao wrote:
> target_phys_addr_t has been already replaced by hwaddr, but this
> one is introduced after.
> 
> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>

Thanks, added to the trivial patches queue.

/mjt

Re: [Qemu-devel] [Qemu-trivial] [PATCH] target-moxie: replace target_phys_addr_t with hwaddr

[Michael Tokarev]

15.05.2013 10:04, Hu Tao wrote:
> target_phys_addr_t has been already replaced by hwaddr, but this
> one is introduced after.
> 
> Signed-off-by: Hu Tao <hutao@cn.fujitsu.com>

Thanks, added to the trivial patches queue.

/mjt