* [PATCH v2] dwc3-of-simple: add support for Hikey 970
@ 2020-09-08 9:58 Mauro Carvalho Chehab
2020-09-08 10:02 ` Felipe Balbi
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2020-09-08 9:58 UTC (permalink / raw)
To: Felipe Balbi, Greg Kroah-Hartman
Cc: linuxarm, mauro.chehab, Mauro Carvalho Chehab, John Stultz,
Manivannan Sadhasivam, Philipp Zabel, linux-usb, linux-kernel
This binding driver is needed for Hikey 970 to work,
as otherwise a Serror is produced:
[ 1.837458] SError Interrupt on CPU0, code 0xbf000002 -- SError
[ 1.837462] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
[ 1.837463] Hardware name: HiKey970 (DT)
[ 1.837465] Workqueue: events deferred_probe_work_func
[ 1.837467] pstate: 20000005 (nzCv daif -PAN -UAO BTYPE=--)
[ 1.837468] pc : _raw_spin_unlock_irqrestore+0x18/0x50
[ 1.837469] lr : regmap_unlock_spinlock+0x14/0x20
[ 1.837470] sp : ffff8000124dba60
[ 1.837471] x29: ffff8000124dba60 x28: 0000000000000000
[ 1.837474] x27: ffff0001b7e854c8 x26: ffff80001204ea18
[ 1.837476] x25: 0000000000000005 x24: ffff800011f918f8
[ 1.837479] x23: ffff800011fbb588 x22: ffff0001b7e40e00
[ 1.837481] x21: 0000000000000100 x20: 0000000000000000
[ 1.837483] x19: ffff0001b767ec00 x18: 00000000ff10c000
[ 1.837485] x17: 0000000000000002 x16: 0000b0740fdb9950
[ 1.837488] x15: ffff8000116c1198 x14: ffffffffffffffff
[ 1.837490] x13: 0000000000000030 x12: 0101010101010101
[ 1.837493] x11: 0000000000000020 x10: ffff0001bf17d130
[ 1.837495] x9 : 0000000000000000 x8 : ffff0001b6938080
[ 1.837497] x7 : 0000000000000000 x6 : 000000000000003f
[ 1.837500] x5 : 0000000000000000 x4 : 0000000000000000
[ 1.837502] x3 : ffff80001096a880 x2 : 0000000000000000
[ 1.837505] x1 : ffff0001b7e40e00 x0 : 0000000100000001
[ 1.837507] Kernel panic - not syncing: Asynchronous SError Interrupt
[ 1.837509] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
[ 1.837510] Hardware name: HiKey970 (DT)
[ 1.837511] Workqueue: events deferred_probe_work_func
[ 1.837513] Call trace:
[ 1.837514] dump_backtrace+0x0/0x1e0
[ 1.837515] show_stack+0x18/0x24
[ 1.837516] dump_stack+0xc0/0x11c
[ 1.837517] panic+0x15c/0x324
[ 1.837518] nmi_panic+0x8c/0x90
[ 1.837519] arm64_serror_panic+0x78/0x84
[ 1.837520] do_serror+0x158/0x15c
[ 1.837521] el1_error+0x84/0x100
[ 1.837522] _raw_spin_unlock_irqrestore+0x18/0x50
[ 1.837523] regmap_write+0x58/0x80
[ 1.837524] hi3660_reset_deassert+0x28/0x34
[ 1.837526] reset_control_deassert+0x50/0x260
[ 1.837527] reset_control_deassert+0xf4/0x260
[ 1.837528] dwc3_probe+0x5dc/0xe6c
[ 1.837529] platform_drv_probe+0x54/0xb0
[ 1.837530] really_probe+0xe0/0x490
[ 1.837531] driver_probe_device+0xf4/0x160
[ 1.837532] __device_attach_driver+0x8c/0x114
[ 1.837533] bus_for_each_drv+0x78/0xcc
[ 1.837534] __device_attach+0x108/0x1a0
[ 1.837535] device_initial_probe+0x14/0x20
[ 1.837537] bus_probe_device+0x98/0xa0
[ 1.837538] deferred_probe_work_func+0x88/0xe0
[ 1.837539] process_one_work+0x1cc/0x350
[ 1.837540] worker_thread+0x2c0/0x470
[ 1.837541] kthread+0x154/0x160
[ 1.837542] ret_from_fork+0x10/0x30
[ 1.837569] SMP: stopping secondary CPUs
[ 1.837570] Kernel Offset: 0x1d0000 from 0xffff800010000000
[ 1.837571] PHYS_OFFSET: 0x0
[ 1.837572] CPU features: 0x240002,20882004
[ 1.837573] Memory Limit: none
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
v2: removed a hunk enabling a quirk that was causing Serror panic()
on arm. It turned that such hunk were not needed at all.
drivers/usb/dwc3/dwc3-of-simple.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of-simple.c
index 8852fbfdead4..336253ff5574 100644
--- a/drivers/usb/dwc3/dwc3-of-simple.c
+++ b/drivers/usb/dwc3/dwc3-of-simple.c
@@ -176,6 +176,7 @@ static const struct of_device_id of_dwc3_simple_match[] = {
{ .compatible = "cavium,octeon-7130-usb-uctl" },
{ .compatible = "sprd,sc9860-dwc3" },
{ .compatible = "allwinner,sun50i-h6-dwc3" },
+ { .compatible = "hisilicon,hi3670-dwc3" },
{ /* Sentinel */ }
};
MODULE_DEVICE_TABLE(of, of_dwc3_simple_match);
--
2.26.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dwc3-of-simple: add support for Hikey 970
2020-09-08 9:58 [PATCH v2] dwc3-of-simple: add support for Hikey 970 Mauro Carvalho Chehab
@ 2020-09-08 10:02 ` Felipe Balbi
2020-09-08 10:23 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Felipe Balbi @ 2020-09-08 10:02 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Greg Kroah-Hartman
Cc: linuxarm, mauro.chehab, Mauro Carvalho Chehab, John Stultz,
Manivannan Sadhasivam, Philipp Zabel, linux-usb, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 3745 bytes --]
Hi,
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:
> This binding driver is needed for Hikey 970 to work,
> as otherwise a Serror is produced:
you mentioned Serror doesn't happen anymore...
> [ 1.837458] SError Interrupt on CPU0, code 0xbf000002 -- SError
> [ 1.837462] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
> [ 1.837463] Hardware name: HiKey970 (DT)
> [ 1.837465] Workqueue: events deferred_probe_work_func
> [ 1.837467] pstate: 20000005 (nzCv daif -PAN -UAO BTYPE=--)
> [ 1.837468] pc : _raw_spin_unlock_irqrestore+0x18/0x50
> [ 1.837469] lr : regmap_unlock_spinlock+0x14/0x20
> [ 1.837470] sp : ffff8000124dba60
> [ 1.837471] x29: ffff8000124dba60 x28: 0000000000000000
> [ 1.837474] x27: ffff0001b7e854c8 x26: ffff80001204ea18
> [ 1.837476] x25: 0000000000000005 x24: ffff800011f918f8
> [ 1.837479] x23: ffff800011fbb588 x22: ffff0001b7e40e00
> [ 1.837481] x21: 0000000000000100 x20: 0000000000000000
> [ 1.837483] x19: ffff0001b767ec00 x18: 00000000ff10c000
> [ 1.837485] x17: 0000000000000002 x16: 0000b0740fdb9950
> [ 1.837488] x15: ffff8000116c1198 x14: ffffffffffffffff
> [ 1.837490] x13: 0000000000000030 x12: 0101010101010101
> [ 1.837493] x11: 0000000000000020 x10: ffff0001bf17d130
> [ 1.837495] x9 : 0000000000000000 x8 : ffff0001b6938080
> [ 1.837497] x7 : 0000000000000000 x6 : 000000000000003f
> [ 1.837500] x5 : 0000000000000000 x4 : 0000000000000000
> [ 1.837502] x3 : ffff80001096a880 x2 : 0000000000000000
> [ 1.837505] x1 : ffff0001b7e40e00 x0 : 0000000100000001
> [ 1.837507] Kernel panic - not syncing: Asynchronous SError Interrupt
> [ 1.837509] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
> [ 1.837510] Hardware name: HiKey970 (DT)
> [ 1.837511] Workqueue: events deferred_probe_work_func
> [ 1.837513] Call trace:
> [ 1.837514] dump_backtrace+0x0/0x1e0
> [ 1.837515] show_stack+0x18/0x24
> [ 1.837516] dump_stack+0xc0/0x11c
> [ 1.837517] panic+0x15c/0x324
> [ 1.837518] nmi_panic+0x8c/0x90
> [ 1.837519] arm64_serror_panic+0x78/0x84
> [ 1.837520] do_serror+0x158/0x15c
> [ 1.837521] el1_error+0x84/0x100
> [ 1.837522] _raw_spin_unlock_irqrestore+0x18/0x50
> [ 1.837523] regmap_write+0x58/0x80
> [ 1.837524] hi3660_reset_deassert+0x28/0x34
> [ 1.837526] reset_control_deassert+0x50/0x260
> [ 1.837527] reset_control_deassert+0xf4/0x260
> [ 1.837528] dwc3_probe+0x5dc/0xe6c
> [ 1.837529] platform_drv_probe+0x54/0xb0
> [ 1.837530] really_probe+0xe0/0x490
> [ 1.837531] driver_probe_device+0xf4/0x160
> [ 1.837532] __device_attach_driver+0x8c/0x114
> [ 1.837533] bus_for_each_drv+0x78/0xcc
> [ 1.837534] __device_attach+0x108/0x1a0
> [ 1.837535] device_initial_probe+0x14/0x20
> [ 1.837537] bus_probe_device+0x98/0xa0
> [ 1.837538] deferred_probe_work_func+0x88/0xe0
> [ 1.837539] process_one_work+0x1cc/0x350
> [ 1.837540] worker_thread+0x2c0/0x470
> [ 1.837541] kthread+0x154/0x160
> [ 1.837542] ret_from_fork+0x10/0x30
> [ 1.837569] SMP: stopping secondary CPUs
> [ 1.837570] Kernel Offset: 0x1d0000 from 0xffff800010000000
> [ 1.837571] PHYS_OFFSET: 0x0
> [ 1.837572] CPU features: 0x240002,20882004
> [ 1.837573] Memory Limit: none
is this splat still valid? I can edit commit while applying, just let me
know if I should remove this.
--
balbi
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 857 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dwc3-of-simple: add support for Hikey 970
2020-09-08 10:02 ` Felipe Balbi
@ 2020-09-08 10:23 ` Mauro Carvalho Chehab
2020-09-08 11:50 ` Felipe Balbi
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2020-09-08 10:23 UTC (permalink / raw)
To: Felipe Balbi
Cc: Greg Kroah-Hartman, linuxarm, mauro.chehab, John Stultz,
Manivannan Sadhasivam, Philipp Zabel, linux-usb, linux-kernel
Hi Felipe,
Em Tue, 08 Sep 2020 13:02:09 +0300
Felipe Balbi <balbi@kernel.org> escreveu:
> Hi,
>
> Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:
> > This binding driver is needed for Hikey 970 to work,
> > as otherwise a Serror is produced:
>
> you mentioned Serror doesn't happen anymore...
>
> > [ 1.837458] SError Interrupt on CPU0, code 0xbf000002 -- SError
> > [ 1.837462] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
> > [ 1.837463] Hardware name: HiKey970 (DT)
> > [ 1.837465] Workqueue: events deferred_probe_work_func
> > [ 1.837467] pstate: 20000005 (nzCv daif -PAN -UAO BTYPE=--)
> > [ 1.837468] pc : _raw_spin_unlock_irqrestore+0x18/0x50
> > [ 1.837469] lr : regmap_unlock_spinlock+0x14/0x20
> > [ 1.837470] sp : ffff8000124dba60
> > [ 1.837471] x29: ffff8000124dba60 x28: 0000000000000000
> > [ 1.837474] x27: ffff0001b7e854c8 x26: ffff80001204ea18
> > [ 1.837476] x25: 0000000000000005 x24: ffff800011f918f8
> > [ 1.837479] x23: ffff800011fbb588 x22: ffff0001b7e40e00
> > [ 1.837481] x21: 0000000000000100 x20: 0000000000000000
> > [ 1.837483] x19: ffff0001b767ec00 x18: 00000000ff10c000
> > [ 1.837485] x17: 0000000000000002 x16: 0000b0740fdb9950
> > [ 1.837488] x15: ffff8000116c1198 x14: ffffffffffffffff
> > [ 1.837490] x13: 0000000000000030 x12: 0101010101010101
> > [ 1.837493] x11: 0000000000000020 x10: ffff0001bf17d130
> > [ 1.837495] x9 : 0000000000000000 x8 : ffff0001b6938080
> > [ 1.837497] x7 : 0000000000000000 x6 : 000000000000003f
> > [ 1.837500] x5 : 0000000000000000 x4 : 0000000000000000
> > [ 1.837502] x3 : ffff80001096a880 x2 : 0000000000000000
> > [ 1.837505] x1 : ffff0001b7e40e00 x0 : 0000000100000001
> > [ 1.837507] Kernel panic - not syncing: Asynchronous SError Interrupt
> > [ 1.837509] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
> > [ 1.837510] Hardware name: HiKey970 (DT)
> > [ 1.837511] Workqueue: events deferred_probe_work_func
> > [ 1.837513] Call trace:
> > [ 1.837514] dump_backtrace+0x0/0x1e0
> > [ 1.837515] show_stack+0x18/0x24
> > [ 1.837516] dump_stack+0xc0/0x11c
> > [ 1.837517] panic+0x15c/0x324
> > [ 1.837518] nmi_panic+0x8c/0x90
> > [ 1.837519] arm64_serror_panic+0x78/0x84
> > [ 1.837520] do_serror+0x158/0x15c
> > [ 1.837521] el1_error+0x84/0x100
> > [ 1.837522] _raw_spin_unlock_irqrestore+0x18/0x50
> > [ 1.837523] regmap_write+0x58/0x80
> > [ 1.837524] hi3660_reset_deassert+0x28/0x34
> > [ 1.837526] reset_control_deassert+0x50/0x260
> > [ 1.837527] reset_control_deassert+0xf4/0x260
> > [ 1.837528] dwc3_probe+0x5dc/0xe6c
> > [ 1.837529] platform_drv_probe+0x54/0xb0
> > [ 1.837530] really_probe+0xe0/0x490
> > [ 1.837531] driver_probe_device+0xf4/0x160
> > [ 1.837532] __device_attach_driver+0x8c/0x114
> > [ 1.837533] bus_for_each_drv+0x78/0xcc
> > [ 1.837534] __device_attach+0x108/0x1a0
> > [ 1.837535] device_initial_probe+0x14/0x20
> > [ 1.837537] bus_probe_device+0x98/0xa0
> > [ 1.837538] deferred_probe_work_func+0x88/0xe0
> > [ 1.837539] process_one_work+0x1cc/0x350
> > [ 1.837540] worker_thread+0x2c0/0x470
> > [ 1.837541] kthread+0x154/0x160
> > [ 1.837542] ret_from_fork+0x10/0x30
> > [ 1.837569] SMP: stopping secondary CPUs
> > [ 1.837570] Kernel Offset: 0x1d0000 from 0xffff800010000000
> > [ 1.837571] PHYS_OFFSET: 0x0
> > [ 1.837572] CPU features: 0x240002,20882004
> > [ 1.837573] Memory Limit: none
>
> is this splat still valid?
What I tried to say, is that, if the dwc3 is described this way at the
DT bindings:
/ {
dwc3: dwc3@ff100000 {
compatible = "snps,dwc3";
reg = <0x0 0xff100000 0x0 0x100000>;
clocks = <&crg_ctrl HI3670_CLK_GATE_ABB_USB>,
<&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
<&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
<&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
...
The panic occurs, with the logs posted at the patch.
The fix is to use dwc3-of-simple to initialize the clocks earlier,
e. g., using this binding:
/ {
usb3: hisi_dwc3 {
compatible = "hisilicon,kirin970-dwc3";
#address-cells = <2>;
#size-cells = <2>;
ranges;
clocks = <&crg_ctrl HI3670_CLK_GATE_ABB_USB>,
<&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
<&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
<&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
dwc3: dwc3@ff100000 {
compatible = "snps,dwc3";
teg = <0x0 0xff100000 0x0 0x100000>;
...
> I can edit commit while applying, just let me know if I should remove this.
If you think the description is not OK, feel free to edit it to better
fit the needs.
Thanks,
Mauro
FYI, that's the diff between the two DTS versions.
diff --git a/arch/arm64/boot/dts/hisilicon/hi3670.dtsi b/arch/arm64/boot/dts/hisilicon/hi3670.dtsi
index 6d6863a05c76..9e87a0a0589e 100644
--- a/arch/arm64/boot/dts/hisilicon/hi3670.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hi3670.dtsi
@@ -825,14 +825,16 @@ usb31_misc_rst: usb31_misc_rst_controller {
hisi,rst-syscon = <&usb3_otg_bc>;
};
- dwc3: dwc3@ff100000 {
- compatible = "snps,dwc3";
- reg = <0x0 0xff100000 0x0 0x100000>;
+ usb3: hisi_dwc3 {
+ compatible = "hisilicon,kirin970-dwc3";
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
clocks = <&crg_ctrl HI3670_CLK_GATE_ABB_USB>,
- <&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
- <&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
- <&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
+ <&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
+ <&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
+ <&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
clock-names = "clk_gate_abb_usb",
"hclk_gate_usb3otg",
"clk_gate_usb3otg_ref",
@@ -845,11 +847,16 @@ dwc3: dwc3@ff100000 {
<&usb31_misc_rst 0xA0 8>,
<&usb31_misc_rst 0xA0 9>;
- interrupts = <0 159 IRQ_TYPE_LEVEL_HIGH>,
- <0 161 IRQ_TYPE_LEVEL_HIGH>;
+ dwc3: dwc3@ff100000 {
+ compatible = "snps,dwc3";
+ reg = <0x0 0xff100000 0x0 0x100000>;
- phys = <&usb_phy>;
- phy-names = "usb3-phy";
+ interrupts = <0 159 IRQ_TYPE_LEVEL_HIGH>,
+ <0 161 IRQ_TYPE_LEVEL_HIGH>;
+
+ phys = <&usb_phy>;
+ phy-names = "usb3-phy";
+ };
};
};
};
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dwc3-of-simple: add support for Hikey 970
2020-09-08 10:23 ` Mauro Carvalho Chehab
@ 2020-09-08 11:50 ` Felipe Balbi
0 siblings, 0 replies; 4+ messages in thread
From: Felipe Balbi @ 2020-09-08 11:50 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Greg Kroah-Hartman, linuxarm, mauro.chehab, John Stultz,
Manivannan Sadhasivam, Philipp Zabel, linux-usb, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 5135 bytes --]
Hi,
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:
>> Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:
>> > This binding driver is needed for Hikey 970 to work,
>> > as otherwise a Serror is produced:
>>
>> you mentioned Serror doesn't happen anymore...
>>
>> > [ 1.837458] SError Interrupt on CPU0, code 0xbf000002 -- SError
>> > [ 1.837462] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
>> > [ 1.837463] Hardware name: HiKey970 (DT)
>> > [ 1.837465] Workqueue: events deferred_probe_work_func
>> > [ 1.837467] pstate: 20000005 (nzCv daif -PAN -UAO BTYPE=--)
>> > [ 1.837468] pc : _raw_spin_unlock_irqrestore+0x18/0x50
>> > [ 1.837469] lr : regmap_unlock_spinlock+0x14/0x20
>> > [ 1.837470] sp : ffff8000124dba60
>> > [ 1.837471] x29: ffff8000124dba60 x28: 0000000000000000
>> > [ 1.837474] x27: ffff0001b7e854c8 x26: ffff80001204ea18
>> > [ 1.837476] x25: 0000000000000005 x24: ffff800011f918f8
>> > [ 1.837479] x23: ffff800011fbb588 x22: ffff0001b7e40e00
>> > [ 1.837481] x21: 0000000000000100 x20: 0000000000000000
>> > [ 1.837483] x19: ffff0001b767ec00 x18: 00000000ff10c000
>> > [ 1.837485] x17: 0000000000000002 x16: 0000b0740fdb9950
>> > [ 1.837488] x15: ffff8000116c1198 x14: ffffffffffffffff
>> > [ 1.837490] x13: 0000000000000030 x12: 0101010101010101
>> > [ 1.837493] x11: 0000000000000020 x10: ffff0001bf17d130
>> > [ 1.837495] x9 : 0000000000000000 x8 : ffff0001b6938080
>> > [ 1.837497] x7 : 0000000000000000 x6 : 000000000000003f
>> > [ 1.837500] x5 : 0000000000000000 x4 : 0000000000000000
>> > [ 1.837502] x3 : ffff80001096a880 x2 : 0000000000000000
>> > [ 1.837505] x1 : ffff0001b7e40e00 x0 : 0000000100000001
>> > [ 1.837507] Kernel panic - not syncing: Asynchronous SError Interrupt
>> > [ 1.837509] CPU: 0 PID: 74 Comm: kworker/0:1 Not tainted 5.8.0+ #205
>> > [ 1.837510] Hardware name: HiKey970 (DT)
>> > [ 1.837511] Workqueue: events deferred_probe_work_func
>> > [ 1.837513] Call trace:
>> > [ 1.837514] dump_backtrace+0x0/0x1e0
>> > [ 1.837515] show_stack+0x18/0x24
>> > [ 1.837516] dump_stack+0xc0/0x11c
>> > [ 1.837517] panic+0x15c/0x324
>> > [ 1.837518] nmi_panic+0x8c/0x90
>> > [ 1.837519] arm64_serror_panic+0x78/0x84
>> > [ 1.837520] do_serror+0x158/0x15c
>> > [ 1.837521] el1_error+0x84/0x100
>> > [ 1.837522] _raw_spin_unlock_irqrestore+0x18/0x50
>> > [ 1.837523] regmap_write+0x58/0x80
>> > [ 1.837524] hi3660_reset_deassert+0x28/0x34
>> > [ 1.837526] reset_control_deassert+0x50/0x260
>> > [ 1.837527] reset_control_deassert+0xf4/0x260
>> > [ 1.837528] dwc3_probe+0x5dc/0xe6c
>> > [ 1.837529] platform_drv_probe+0x54/0xb0
>> > [ 1.837530] really_probe+0xe0/0x490
>> > [ 1.837531] driver_probe_device+0xf4/0x160
>> > [ 1.837532] __device_attach_driver+0x8c/0x114
>> > [ 1.837533] bus_for_each_drv+0x78/0xcc
>> > [ 1.837534] __device_attach+0x108/0x1a0
>> > [ 1.837535] device_initial_probe+0x14/0x20
>> > [ 1.837537] bus_probe_device+0x98/0xa0
>> > [ 1.837538] deferred_probe_work_func+0x88/0xe0
>> > [ 1.837539] process_one_work+0x1cc/0x350
>> > [ 1.837540] worker_thread+0x2c0/0x470
>> > [ 1.837541] kthread+0x154/0x160
>> > [ 1.837542] ret_from_fork+0x10/0x30
>> > [ 1.837569] SMP: stopping secondary CPUs
>> > [ 1.837570] Kernel Offset: 0x1d0000 from 0xffff800010000000
>> > [ 1.837571] PHYS_OFFSET: 0x0
>> > [ 1.837572] CPU features: 0x240002,20882004
>> > [ 1.837573] Memory Limit: none
>>
>> is this splat still valid?
>
> What I tried to say, is that, if the dwc3 is described this way at the
> DT bindings:
>
>
> / {
> dwc3: dwc3@ff100000 {
> compatible = "snps,dwc3";
> reg = <0x0 0xff100000 0x0 0x100000>;
> clocks = <&crg_ctrl HI3670_CLK_GATE_ABB_USB>,
> <&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
> <&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
> <&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
> ...
>
> The panic occurs, with the logs posted at the patch.
>
> The fix is to use dwc3-of-simple to initialize the clocks earlier,
> e. g., using this binding:
>
> / {
> usb3: hisi_dwc3 {
> compatible = "hisilicon,kirin970-dwc3";
> #address-cells = <2>;
> #size-cells = <2>;
> ranges;
>
> clocks = <&crg_ctrl HI3670_CLK_GATE_ABB_USB>,
> <&crg_ctrl HI3670_HCLK_GATE_USB3OTG>,
> <&crg_ctrl HI3670_CLK_GATE_USB3OTG_REF>,
> <&crg_ctrl HI3670_ACLK_GATE_USB3DVFS>;
>
>
> dwc3: dwc3@ff100000 {
> compatible = "snps,dwc3";
> teg = <0x0 0xff100000 0x0 0x100000>;
> ...
now it's clear, I'll apply as is :-)
--
balbi
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 857 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-09-08 12:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-08 9:58 [PATCH v2] dwc3-of-simple: add support for Hikey 970 Mauro Carvalho Chehab
2020-09-08 10:02 ` Felipe Balbi
2020-09-08 10:23 ` Mauro Carvalho Chehab
2020-09-08 11:50 ` Felipe Balbi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).