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=-6.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 900C9C31E4A for ; Thu, 13 Jun 2019 16:56:58 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 684B820665 for ; Thu, 13 Jun 2019 16:56:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="tEnyQxaV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 684B820665 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=PYXJ1vA/BoMLT8c6Nj41OIfKnXdzMJMMvRgwQlqdO74=; b=tEnyQxaVVeHjel81I9YkbhkCQ llNloN8AOml9wilyp0zQdszg15DB3o4FXuAQBCbbTCgmvCbKA0kwY3wZoCv8hvL3DnuICMgLQk9m4 odrIhi8JYcuAuPX4xoJn5RN+Pm6M5Fxf5RLpbNyvG1qbQi1Vxu0Qhk4e+1ubsL4lxkMZaOq4Fl/EZ 4i+C3sHX8lpts3y/FG7uNsTAe1CEmO6egjEMOZSH1fdd1u4DDFMKpo6pMOVt6OLywgdt2SKZTXpwJ jhE9HLjLji+fOOe3maSvDQasOmKZSsLWm9444bLxYgWTfPq53PL3z9I8vxqmQRPNUmmAaYLiVUftT FmP4PwYOw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hbT20-0007bY-F7; Thu, 13 Jun 2019 16:56:56 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hbT1x-0007at-AN; Thu, 13 Jun 2019 16:56:54 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2C32D367; Thu, 13 Jun 2019 09:56:52 -0700 (PDT) Received: from [10.1.197.57] (e110467-lin.cambridge.arm.com [10.1.197.57]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 0A1293F694; Thu, 13 Jun 2019 09:56:49 -0700 (PDT) Subject: Re: [PATCH] arm64: dts: rockchip: Update DWC3 modules on RK3399 SoCs To: Enric Balletbo i Serra , devicetree@vger.kernel.org References: <20190613162745.12195-1-enric.balletbo@collabora.com> From: Robin Murphy Message-ID: <40d2260f-3925-acdc-eb02-8abb972f1056@arm.com> Date: Thu, 13 Jun 2019 17:56:48 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: <20190613162745.12195-1-enric.balletbo@collabora.com> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190613_095653_457700_60FDA344 X-CRM114-Status: GOOD ( 23.57 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Heiko Stuebner , Felipe Balbi , Masahiro Yamada , Christoph Muellner , Viresh Kumar , Randy Li , linux-kernel@vger.kernel.org, Vicente Bergas , linux-rockchip@lists.infradead.org, Tony Xie , Klaus Goger , Rob Herring , Collabora Kernel ML , linux-arm-kernel@lists.infradead.org, Matthias Brugger Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 13/06/2019 17:27, Enric Balletbo i Serra wrote: > As per binding documentation [1], the DWC3 core should have the "ref", > "bus_early" and "suspend" clocks. As explained in the binding, those > clocks are required for new platforms but not for existing platforms > before commit fe8abf332b8f ("usb: dwc3: support clocks and resets for > DWC3 core"). > > However, as those clocks are really treated as required, this ends with > having some annoying messages when the "rockchip,rk3399-dwc3" is used: > > [ 1.724107] dwc3 fe800000.dwc3: Failed to get clk 'ref': -2 > [ 1.731893] dwc3 fe900000.dwc3: Failed to get clk 'ref': -2 > [ 2.495937] dwc3 fe800000.dwc3: Failed to get clk 'ref': -2 > [ 2.647239] dwc3 fe900000.dwc3: Failed to get clk 'ref': -2 > > In order to remove those annoying messages, update the DWC3 hardware > module node and add all the required clocks. With this change, both, the > glue node and the DWC3 core node, have the clocks defined, but that's > not really a problem and there isn't a side effect on do this. So, we > can get rid of the annoying get clk error messages. Can we not just move these clocks entirely from the glue layer to the core layer? That didn't seem to break when I tried it, although I'll admit my 'testing' was no more than booting and mounting a USB 3.0 flash drive, no suspend or anything fancy. My own attempt to shut up these errors got sidetracked into c0c61471ef86 ("usb: dwc3: of-simple: Convert to bulk clk API"), then apparently stalled :) Robin. > > [1] Documentation/devicetree/bindings/usb/dwc3.txt > > Signed-off-by: Enric Balletbo i Serra > --- > > arch/arm64/boot/dts/rockchip/rk3399.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > index 196ac9b78076..a15348d185ce 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > @@ -414,6 +414,9 @@ > compatible = "snps,dwc3"; > reg = <0x0 0xfe800000 0x0 0x100000>; > interrupts = ; > + clocks = <&cru SCLK_USB3OTG0_REF>, <&cru ACLK_USB3OTG0>, > + <&cru SCLK_USB3OTG0_SUSPEND>; > + clock-names = "ref", "bus_early", "suspend"; > dr_mode = "otg"; > phys = <&u2phy0_otg>, <&tcphy0_usb3>; > phy-names = "usb2-phy", "usb3-phy"; > @@ -447,6 +450,9 @@ > compatible = "snps,dwc3"; > reg = <0x0 0xfe900000 0x0 0x100000>; > interrupts = ; > + clocks = <&cru SCLK_USB3OTG1_REF>, <&cru ACLK_USB3OTG1>, > + <&cru SCLK_USB3OTG1_SUSPEND>; > + clock-names = "ref", "bus_early", "suspend"; > dr_mode = "otg"; > phys = <&u2phy1_otg>, <&tcphy1_usb3>; > phy-names = "usb2-phy", "usb3-phy"; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel