From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 398C3C43461 for ; Tue, 8 Sep 2020 10:23:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F2FE3206A4 for ; Tue, 8 Sep 2020 10:23:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1599560602; bh=drmUtGRANKaPpuo0fEc4pTijLivbP5AYadC3KsIBz8o=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=1nCv6r+aTT6N8Z85c0YZsP4D8Nj2Tdyj2hjC7RYFjm9WtwCqUCs8r7cr1bcWgdrhP wTM3QFL48N4FEJIS3cd92UP4vz6cvQja5oq3ZLo8PQVlMwMLVdveo5cbGGpU/98dYv 13/xiHo2yHS5/WiHvVkJFRaq2yuimSr/j7ESI26Q= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729341AbgIHKXT (ORCPT ); Tue, 8 Sep 2020 06:23:19 -0400 Received: from mail.kernel.org ([198.145.29.99]:35124 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726801AbgIHKXR (ORCPT ); Tue, 8 Sep 2020 06:23:17 -0400 Received: from coco.lan (ip5f5ad5ce.dynamic.kabel-deutschland.de [95.90.213.206]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5B3AF206A4; Tue, 8 Sep 2020 10:23:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1599560596; bh=drmUtGRANKaPpuo0fEc4pTijLivbP5AYadC3KsIBz8o=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=lXs2aGzFVeTyQ5337C0MHe4BOHlN5Es246OyW7YVqfFwu79eJWstkcdVotqOuVKG/ f+7Sd8XGix3AGnXB8RkqRww1Ps7R7eNVlxfI3ayf3ishPPUqmBehUrAuhS3UUNt2Gl ENvs3Ii10UMAhQjssGExMIIsVroN5XkIhCESFGu0= Date: Tue, 8 Sep 2020 12:23:11 +0200 From: Mauro Carvalho Chehab To: Felipe Balbi Cc: Greg Kroah-Hartman , linuxarm@huawei.com, mauro.chehab@huawei.com, John Stultz , Manivannan Sadhasivam , Philipp Zabel , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] dwc3-of-simple: add support for Hikey 970 Message-ID: <20200908122311.55346806@coco.lan> In-Reply-To: <87k0x4lh7i.fsf@kernel.org> References: <731e13f9fbba3a81bedb39f1c1deaf41200acd0c.1599559004.git.mchehab+huawei@kernel.org> <87k0x4lh7i.fsf@kernel.org> X-Mailer: Claws Mail 3.17.6 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Felipe, Em Tue, 08 Sep 2020 13:02:09 +0300 Felipe Balbi escreveu: > Hi, > > Mauro Carvalho Chehab 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"; + }; }; }; };