Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1360318 > unrolled thread
| Started by | Guenter Roeck <linux@roeck-us.net> |
|---|---|
| First post | 2016-03-18 04:00 +0100 |
| Last post | 2016-03-18 14:30 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address Guenter Roeck <linux@roeck-us.net> - 2016-03-18 04:00 +0100
Re: [PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address Peter Hurley <peter@hurleysoftware.com> - 2016-03-18 04:30 +0100
Re: [PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address Peter Hurley <peter@hurleysoftware.com> - 2016-03-18 14:30 +0100
| From | Guenter Roeck <linux@roeck-us.net> |
|---|---|
| Date | 2016-03-18 04:00 +0100 |
| Subject | [PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address |
| Message-ID | <rdSNQ-7PY-7@gated-at.bofh.it> |
nios2 builds fail with the following build error.
arch/nios2/kernel/prom.c: In function 'early_init_dt_scan_serial':
arch/nios2/kernel/prom.c:100:2: error:
implicit declaration of function 'fdt_translate_address'
Commit c90fe9c0394b ("of: earlycon: Move address translation to
of_setup_earlycon()") replaced fdt_translate_address() with
of_flat_dt_translate_address() but missed updating the nios2 code.
Fixes: c90fe9c0394b ("of: earlycon: Move address translation to of_setup_earlycon()")
Cc: Peter Hurley <peter@hurleysoftware.com>
Cc: Rob Herring <robh@kernel.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
arch/nios2/kernel/prom.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/nios2/kernel/prom.c b/arch/nios2/kernel/prom.c
index 718dd197909f..367c5426157b 100644
--- a/arch/nios2/kernel/prom.c
+++ b/arch/nios2/kernel/prom.c
@@ -97,8 +97,7 @@ static int __init early_init_dt_scan_serial(unsigned long node,
return 0;
#endif
- *addr64 = fdt_translate_address((const void *)initial_boot_params,
- node);
+ *addr64 = of_flat_dt_translate_address(node);
return *addr64 == OF_BAD_ADDR ? 0 : 1;
}
--
2.5.0
[toc] | [next] | [standalone]
| From | Peter Hurley <peter@hurleysoftware.com> |
|---|---|
| Date | 2016-03-18 04:30 +0100 |
| Subject | Re: [PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address |
| Message-ID | <rdTgS-8mY-5@gated-at.bofh.it> |
| In reply to | #1360318 |
Hi Guenter,
On 03/17/2016 07:49 PM, Guenter Roeck wrote:
> nios2 builds fail with the following build error.
>
> arch/nios2/kernel/prom.c: In function 'early_init_dt_scan_serial':
> arch/nios2/kernel/prom.c:100:2: error:
> implicit declaration of function 'fdt_translate_address'
>
> Commit c90fe9c0394b ("of: earlycon: Move address translation to
> of_setup_earlycon()") replaced fdt_translate_address() with
> of_flat_dt_translate_address() but missed updating the nios2 code.
My bad.
The sad part is I even noted this other user of fdt_translate_address()
in a review of a different patch, and then overlooked it in my own patch. :/
Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
> Fixes: c90fe9c0394b ("of: earlycon: Move address translation to of_setup_earlycon()")
> Cc: Peter Hurley <peter@hurleysoftware.com>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
> ---
> arch/nios2/kernel/prom.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/nios2/kernel/prom.c b/arch/nios2/kernel/prom.c
> index 718dd197909f..367c5426157b 100644
> --- a/arch/nios2/kernel/prom.c
> +++ b/arch/nios2/kernel/prom.c
> @@ -97,8 +97,7 @@ static int __init early_init_dt_scan_serial(unsigned long node,
> return 0;
> #endif
>
> - *addr64 = fdt_translate_address((const void *)initial_boot_params,
> - node);
> + *addr64 = of_flat_dt_translate_address(node);
>
> return *addr64 == OF_BAD_ADDR ? 0 : 1;
> }
>
[toc] | [prev] | [next] | [standalone]
| From | Peter Hurley <peter@hurleysoftware.com> |
|---|---|
| Date | 2016-03-18 14:30 +0100 |
| Subject | Re: [PATCH] nios2: Replace fdt_translate_address with of_flat_dt_translate_address |
| Message-ID | <re2Dw-6m4-7@gated-at.bofh.it> |
| In reply to | #1360323 |
On 03/17/2016 10:58 PM, Ley Foon Tan wrote:
> On Thu, 2016-03-17 at 20:22 -0700, Peter Hurley wrote:
>> Hi Guenter,
>>
>> On 03/17/2016 07:49 PM, Guenter Roeck wrote:
>>> nios2 builds fail with the following build error.
>>>
>>> arch/nios2/kernel/prom.c: In function 'early_init_dt_scan_serial':
>>> arch/nios2/kernel/prom.c:100:2: error:
>>> implicit declaration of function 'fdt_translate_address'
>>>
>>> Commit c90fe9c0394b ("of: earlycon: Move address translation to
>>> of_setup_earlycon()") replaced fdt_translate_address() with
>>> of_flat_dt_translate_address() but missed updating the nios2 code.
>>
>> My bad.
>> The sad part is I even noted this other user of fdt_translate_address()
>> in a review of a different patch, and then overlooked it in my own patch. :/
>>
>> Reviewed-by: Peter Hurley <peter@hurleysoftware.com>
>>
> Hi Peter,
>
> Will you take this patch or you want me to take this?
> Thanks.
Please take this patch.
Regards,
Peter Hurley
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web