* [PATCH] mips/octeon_3xxx: Fix a warning on octeon_3xxx
@ 2014-03-17 6:06 ` Wei.Yang
0 siblings, 0 replies; 4+ messages in thread
From: Wei.Yang @ 2014-03-17 6:06 UTC (permalink / raw)
To: david.daney, ralf; +Cc: linux-mips, wei.yang
From: Yang Wei <Wei.Yang@windriver.com>
Since the xlate of interrupts property of GPIO on octeon 3xxx
does not success, so the following warning would be triggerred
while invoking of_device_alloc to create platform device.
So we need to remove it to avoid the warning.
WARNING: CPU: 1 PID: 1 at drivers/of/platform.c:173 of_device_alloc+0x294/0x2a0()
Modules linked in:
CPU: 1 PID: 1 Comm: swapper/0 Not tainted 3.14.0-rc6- #11
Stack : ffffffff81a20000 0000000000000001 0000000000000004 ffffffff81b50000
0000000000000001 0000000000000000 0000000000000000 ffffffff8119e878
ffffffff81a20000 ffffffff8119ee98 0000000000000000 0000000000000000
ffffffff81b30000 ffffffff81b20000 ffffffff81932900 ffffffff81a11077
ffffffff81b27a08 800000041f8704a8 0000000000000001 0000000000000001
0000000000000000 800000041fbf7438 0000000000000001 ffffffff81800d90
800000041f85fa68 ffffffff8114a60c 0000000000000000 ffffffff811a0838
800000041f870000 800000041f85f980 0000000000000001 ffffffff81805080
0000000000000000 0000000000000000 0000000000000000 0000000000000000
0000000000000000 ffffffff81122620 0000000000000000 0000000000000000
...
Call Trace:
[<ffffffff81122620>] show_stack+0xc0/0xe0
[<ffffffff81805080>] dump_stack+0x8c/0xe0
[<ffffffff8114a7ac>] warn_slowpath_common+0x94/0xc8
[<ffffffff81693b1c>] of_device_alloc+0x294/0x2a0
[<ffffffff81693b74>] of_platform_device_create_pdata+0x4c/0xf0
[<ffffffff81693d58>] of_platform_bus_create+0x128/0x1a8
[<ffffffff81693da0>] of_platform_bus_create+0x170/0x1a8
[<ffffffff81693e8c>] of_platform_bus_probe+0xb4/0x110
[<ffffffff81100598>] do_one_initcall+0xe8/0x130
[<ffffffff81a92c5c>] kernel_init_freeable+0x1d4/0x2bc
[<ffffffff817fe140>] kernel_init+0x20/0x118
[<ffffffff8111d024>] ret_from_kernel_thread+0x14/0x1c
Signed-off-by: Yang Wei <Wei.Yang@windriver.com>
---
arch/mips/cavium-octeon/octeon_3xxx.dts | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/mips/cavium-octeon/octeon_3xxx.dts b/arch/mips/cavium-octeon/octeon_3xxx.dts
index fa33115..9fe94d5 100644
--- a/arch/mips/cavium-octeon/octeon_3xxx.dts
+++ b/arch/mips/cavium-octeon/octeon_3xxx.dts
@@ -43,11 +43,6 @@
*/
interrupt-controller;
#interrupt-cells = <2>;
- /* The GPIO pin connect to 16 consecutive CUI bits */
- interrupts = <0 16>, <0 17>, <0 18>, <0 19>,
- <0 20>, <0 21>, <0 22>, <0 23>,
- <0 24>, <0 25>, <0 26>, <0 27>,
- <0 28>, <0 29>, <0 30>, <0 31>;
};
smi0: mdio@1180000001800 {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] mips/octeon_3xxx: Fix a warning on octeon_3xxx
@ 2014-03-17 6:06 ` Wei.Yang
0 siblings, 0 replies; 4+ messages in thread
From: Wei.Yang @ 2014-03-17 6:06 UTC (permalink / raw)
To: david.daney, ralf; +Cc: linux-mips, wei.yang
From: Yang Wei <Wei.Yang@windriver.com>
Since the xlate of interrupts property of GPIO on octeon 3xxx
does not success, so the following warning would be triggerred
while invoking of_device_alloc to create platform device.
So we need to remove it to avoid the warning.
WARNING: CPU: 1 PID: 1 at drivers/of/platform.c:173 of_device_alloc+0x294/0x2a0()
Modules linked in:
CPU: 1 PID: 1 Comm: swapper/0 Not tainted 3.14.0-rc6- #11
Stack : ffffffff81a20000 0000000000000001 0000000000000004 ffffffff81b50000
0000000000000001 0000000000000000 0000000000000000 ffffffff8119e878
ffffffff81a20000 ffffffff8119ee98 0000000000000000 0000000000000000
ffffffff81b30000 ffffffff81b20000 ffffffff81932900 ffffffff81a11077
ffffffff81b27a08 800000041f8704a8 0000000000000001 0000000000000001
0000000000000000 800000041fbf7438 0000000000000001 ffffffff81800d90
800000041f85fa68 ffffffff8114a60c 0000000000000000 ffffffff811a0838
800000041f870000 800000041f85f980 0000000000000001 ffffffff81805080
0000000000000000 0000000000000000 0000000000000000 0000000000000000
0000000000000000 ffffffff81122620 0000000000000000 0000000000000000
...
Call Trace:
[<ffffffff81122620>] show_stack+0xc0/0xe0
[<ffffffff81805080>] dump_stack+0x8c/0xe0
[<ffffffff8114a7ac>] warn_slowpath_common+0x94/0xc8
[<ffffffff81693b1c>] of_device_alloc+0x294/0x2a0
[<ffffffff81693b74>] of_platform_device_create_pdata+0x4c/0xf0
[<ffffffff81693d58>] of_platform_bus_create+0x128/0x1a8
[<ffffffff81693da0>] of_platform_bus_create+0x170/0x1a8
[<ffffffff81693e8c>] of_platform_bus_probe+0xb4/0x110
[<ffffffff81100598>] do_one_initcall+0xe8/0x130
[<ffffffff81a92c5c>] kernel_init_freeable+0x1d4/0x2bc
[<ffffffff817fe140>] kernel_init+0x20/0x118
[<ffffffff8111d024>] ret_from_kernel_thread+0x14/0x1c
Signed-off-by: Yang Wei <Wei.Yang@windriver.com>
---
arch/mips/cavium-octeon/octeon_3xxx.dts | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/mips/cavium-octeon/octeon_3xxx.dts b/arch/mips/cavium-octeon/octeon_3xxx.dts
index fa33115..9fe94d5 100644
--- a/arch/mips/cavium-octeon/octeon_3xxx.dts
+++ b/arch/mips/cavium-octeon/octeon_3xxx.dts
@@ -43,11 +43,6 @@
*/
interrupt-controller;
#interrupt-cells = <2>;
- /* The GPIO pin connect to 16 consecutive CUI bits */
- interrupts = <0 16>, <0 17>, <0 18>, <0 19>,
- <0 20>, <0 21>, <0 22>, <0 23>,
- <0 24>, <0 25>, <0 26>, <0 27>,
- <0 28>, <0 29>, <0 30>, <0 31>;
};
smi0: mdio@1180000001800 {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] mips/octeon_3xxx: Fix a warning on octeon_3xxx
@ 2014-03-17 20:37 ` David Daney
0 siblings, 0 replies; 4+ messages in thread
From: David Daney @ 2014-03-17 20:37 UTC (permalink / raw)
To: Wei.Yang; +Cc: david.daney, ralf, linux-mips, Andreas Herrmann
On 03/16/2014 11:06 PM, Wei.Yang@windriver.com wrote:
> From: Yang Wei <Wei.Yang@windriver.com>
>
> Since the xlate of interrupts property of GPIO on octeon 3xxx
> does not success, so the following warning would be triggerred
> while invoking of_device_alloc to create platform device.
> So we need to remove it to avoid the warning.
>
> WARNING: CPU: 1 PID: 1 at drivers/of/platform.c:173 of_device_alloc+0x294/0x2a0()
> Modules linked in:
> CPU: 1 PID: 1 Comm: swapper/0 Not tainted 3.14.0-rc6- #11
> Stack : ffffffff81a20000 0000000000000001 0000000000000004 ffffffff81b50000
> 0000000000000001 0000000000000000 0000000000000000 ffffffff8119e878
> ffffffff81a20000 ffffffff8119ee98 0000000000000000 0000000000000000
> ffffffff81b30000 ffffffff81b20000 ffffffff81932900 ffffffff81a11077
> ffffffff81b27a08 800000041f8704a8 0000000000000001 0000000000000001
> 0000000000000000 800000041fbf7438 0000000000000001 ffffffff81800d90
> 800000041f85fa68 ffffffff8114a60c 0000000000000000 ffffffff811a0838
> 800000041f870000 800000041f85f980 0000000000000001 ffffffff81805080
> 0000000000000000 0000000000000000 0000000000000000 0000000000000000
> 0000000000000000 ffffffff81122620 0000000000000000 0000000000000000
> ...
> Call Trace:
> [<ffffffff81122620>] show_stack+0xc0/0xe0
> [<ffffffff81805080>] dump_stack+0x8c/0xe0
> [<ffffffff8114a7ac>] warn_slowpath_common+0x94/0xc8
> [<ffffffff81693b1c>] of_device_alloc+0x294/0x2a0
> [<ffffffff81693b74>] of_platform_device_create_pdata+0x4c/0xf0
> [<ffffffff81693d58>] of_platform_bus_create+0x128/0x1a8
> [<ffffffff81693da0>] of_platform_bus_create+0x170/0x1a8
> [<ffffffff81693e8c>] of_platform_bus_probe+0xb4/0x110
> [<ffffffff81100598>] do_one_initcall+0xe8/0x130
> [<ffffffff81a92c5c>] kernel_init_freeable+0x1d4/0x2bc
> [<ffffffff817fe140>] kernel_init+0x20/0x118
> [<ffffffff8111d024>] ret_from_kernel_thread+0x14/0x1c
>
> Signed-off-by: Yang Wei <Wei.Yang@windriver.com>
NACK!
> ---
> arch/mips/cavium-octeon/octeon_3xxx.dts | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/mips/cavium-octeon/octeon_3xxx.dts b/arch/mips/cavium-octeon/octeon_3xxx.dts
> index fa33115..9fe94d5 100644
> --- a/arch/mips/cavium-octeon/octeon_3xxx.dts
> +++ b/arch/mips/cavium-octeon/octeon_3xxx.dts
> @@ -43,11 +43,6 @@
> */
> interrupt-controller;
> #interrupt-cells = <2>;
> - /* The GPIO pin connect to 16 consecutive CUI bits */
> - interrupts = <0 16>, <0 17>, <0 18>, <0 19>,
> - <0 20>, <0 21>, <0 22>, <0 23>,
> - <0 24>, <0 25>, <0 26>, <0 27>,
> - <0 28>, <0 29>, <0 30>, <0 31>;
On many boards, the device tree is supplied by the bootloader. So
changing the in-kernel device tree that is only used with old "legacy"
bootloaders, will not correct the problem for boards with newer bootloaders.
I think the proper fix is to modify octeon-irq.c so that it doesn't try
to reserve these numbers. We may work on a patch that takes this approach.
David Daney
> };
>
> smi0: mdio@1180000001800 {
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] mips/octeon_3xxx: Fix a warning on octeon_3xxx
@ 2014-03-17 20:37 ` David Daney
0 siblings, 0 replies; 4+ messages in thread
From: David Daney @ 2014-03-17 20:37 UTC (permalink / raw)
To: Wei.Yang; +Cc: david.daney, ralf, linux-mips, Andreas Herrmann
On 03/16/2014 11:06 PM, Wei.Yang@windriver.com wrote:
> From: Yang Wei <Wei.Yang@windriver.com>
>
> Since the xlate of interrupts property of GPIO on octeon 3xxx
> does not success, so the following warning would be triggerred
> while invoking of_device_alloc to create platform device.
> So we need to remove it to avoid the warning.
>
> WARNING: CPU: 1 PID: 1 at drivers/of/platform.c:173 of_device_alloc+0x294/0x2a0()
> Modules linked in:
> CPU: 1 PID: 1 Comm: swapper/0 Not tainted 3.14.0-rc6- #11
> Stack : ffffffff81a20000 0000000000000001 0000000000000004 ffffffff81b50000
> 0000000000000001 0000000000000000 0000000000000000 ffffffff8119e878
> ffffffff81a20000 ffffffff8119ee98 0000000000000000 0000000000000000
> ffffffff81b30000 ffffffff81b20000 ffffffff81932900 ffffffff81a11077
> ffffffff81b27a08 800000041f8704a8 0000000000000001 0000000000000001
> 0000000000000000 800000041fbf7438 0000000000000001 ffffffff81800d90
> 800000041f85fa68 ffffffff8114a60c 0000000000000000 ffffffff811a0838
> 800000041f870000 800000041f85f980 0000000000000001 ffffffff81805080
> 0000000000000000 0000000000000000 0000000000000000 0000000000000000
> 0000000000000000 ffffffff81122620 0000000000000000 0000000000000000
> ...
> Call Trace:
> [<ffffffff81122620>] show_stack+0xc0/0xe0
> [<ffffffff81805080>] dump_stack+0x8c/0xe0
> [<ffffffff8114a7ac>] warn_slowpath_common+0x94/0xc8
> [<ffffffff81693b1c>] of_device_alloc+0x294/0x2a0
> [<ffffffff81693b74>] of_platform_device_create_pdata+0x4c/0xf0
> [<ffffffff81693d58>] of_platform_bus_create+0x128/0x1a8
> [<ffffffff81693da0>] of_platform_bus_create+0x170/0x1a8
> [<ffffffff81693e8c>] of_platform_bus_probe+0xb4/0x110
> [<ffffffff81100598>] do_one_initcall+0xe8/0x130
> [<ffffffff81a92c5c>] kernel_init_freeable+0x1d4/0x2bc
> [<ffffffff817fe140>] kernel_init+0x20/0x118
> [<ffffffff8111d024>] ret_from_kernel_thread+0x14/0x1c
>
> Signed-off-by: Yang Wei <Wei.Yang@windriver.com>
NACK!
> ---
> arch/mips/cavium-octeon/octeon_3xxx.dts | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/mips/cavium-octeon/octeon_3xxx.dts b/arch/mips/cavium-octeon/octeon_3xxx.dts
> index fa33115..9fe94d5 100644
> --- a/arch/mips/cavium-octeon/octeon_3xxx.dts
> +++ b/arch/mips/cavium-octeon/octeon_3xxx.dts
> @@ -43,11 +43,6 @@
> */
> interrupt-controller;
> #interrupt-cells = <2>;
> - /* The GPIO pin connect to 16 consecutive CUI bits */
> - interrupts = <0 16>, <0 17>, <0 18>, <0 19>,
> - <0 20>, <0 21>, <0 22>, <0 23>,
> - <0 24>, <0 25>, <0 26>, <0 27>,
> - <0 28>, <0 29>, <0 30>, <0 31>;
On many boards, the device tree is supplied by the bootloader. So
changing the in-kernel device tree that is only used with old "legacy"
bootloaders, will not correct the problem for boards with newer bootloaders.
I think the proper fix is to modify octeon-irq.c so that it doesn't try
to reserve these numbers. We may work on a patch that takes this approach.
David Daney
> };
>
> smi0: mdio@1180000001800 {
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-03-17 20:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-17 6:06 [PATCH] mips/octeon_3xxx: Fix a warning on octeon_3xxx Wei.Yang
2014-03-17 6:06 ` Wei.Yang
2014-03-17 20:37 ` David Daney
2014-03-17 20:37 ` David Daney
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.