Received: by 2002:a05:6a10:6d10:0:0:0:0 with SMTP id gq16csp759688pxb; Tue, 12 Apr 2022 12:42:46 -0700 (PDT) X-Google-Smtp-Source: ABdhPJztyJv4VWBgPiGoEcJrxQvTY5OR46GtCeVE48qqNFpCvqSfeGpvnjMAKzAmgnEQ/nx1CnCQ X-Received: by 2002:a17:90b:3802:b0:1c6:905c:af2e with SMTP id mq2-20020a17090b380200b001c6905caf2emr6640732pjb.236.1649792566517; Tue, 12 Apr 2022 12:42:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1649792566; cv=none; d=google.com; s=arc-20160816; b=kBC4G9ourgzdHzQvpiAECTg2kKcGFtCsIR7ciwMAfsJXJpCwJ1inavjF5PjY/xKfTd TvCuVg8uGR6+0VFCuMwIbq1nHHbE1oKtr9wTBU39C1Z86CaRoZ3VdiyrJzKkts3qURDO o+ECYV3pKxsFpBMBvJHfrVS50P/2aaRhoKJkyHah77mYE2q4sUTQBQKMK9Oqjr6alGyW d319oOZmS4jks1hVgt2xQGLdH6qGt0gJmgKUy5IPKSoWyou76MG8x5rr57OFQ0awqb4U Uv7MgcF07a9Z44YS2WRg32jnfBmNV+j4jn8+cxj/sOQVB87kzYKx2zW1R/eIsaNbM0AO otXQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=cR2pF5s8t+OdbSSxVuJNcvAN3V/unWJTiAFDdZ/69qU=; b=Az5JMGWeTraDB4WOMQd539T6U2/TdbzAEUO3JaU18XKqXWX+dqtvoxvmp8W5DBSZaT 5aqAocLtZY4ocb+ZeUr52ZUrpEu1fITC6/WrMzO1SWiiYaBLCUvChs+0kevnDRd1J3Tw I8OWIfqa8Qx2TXcJ1thefQdUrjJKZP46tZMMYqwJnnRz/hS3qIQ3tAO7GrKTq4nAaHur DXGEO3+IL5K8hB4tNDC2nTKSTniC0hqr5r5K96/jBDWUQODFMn8fNxMSCGoC1EDWF/KY UKW3rzaFnwcJyAzBYZYQ/xbjcQEcpffYiP2qC/AUV14xqLb20N+cBchMQ741XVsvKBuy SpwA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=GiGHHDgO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [2620:137:e000::1:18]) by mx.google.com with ESMTPS id t5-20020a170902e84500b00153b2d164cdsi2028176plg.213.2022.04.12.12.42.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Apr 2022 12:42:46 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=GiGHHDgO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C65B5644CE; Tue, 12 Apr 2022 12:41:39 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242541AbiDIP2V (ORCPT + 99 others); Sat, 9 Apr 2022 11:28:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47982 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242536AbiDIP2U (ORCPT ); Sat, 9 Apr 2022 11:28:20 -0400 Received: from mail-yb1-xb29.google.com (mail-yb1-xb29.google.com [IPv6:2607:f8b0:4864:20::b29]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7EDBF33A; Sat, 9 Apr 2022 08:26:13 -0700 (PDT) Received: by mail-yb1-xb29.google.com with SMTP id d138so19976066ybc.13; Sat, 09 Apr 2022 08:26:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=cR2pF5s8t+OdbSSxVuJNcvAN3V/unWJTiAFDdZ/69qU=; b=GiGHHDgOsTCZkVZ3qA13Sb6FvHIZGDKjASv/UYR1gRUxpg4ro6pgjjruE+1alUxjvw 9c/GDyeCONs+IqKo2Ih6w8bnMsIo3HpqNprf4VdbfduInTTJVOYRdlsuQNfouTh3AD+W oOafjH2YZ3EJJU8zCNpPqoWpjPHxfyXlgRhWtfbj801Fd8hgas74RpAPB0r1aO7h0fiu wyajkNmZl/hfF9qj8hc5k/C3IQLGwg5B1AkVm1FtHRV1RkaOF0L02PV+pU2fgqHzLGyu qLpT8KDMV5s7LckQ8unn6h6/eZdhs2+bxCxX/tMdPRnPWl+Z1U11qi8OZamegXOW0g7y skmQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=cR2pF5s8t+OdbSSxVuJNcvAN3V/unWJTiAFDdZ/69qU=; b=iJ+uQul/Ovvp+DXj4e50WT/rQhXc95FNEGTueFEgG3QqO/wM+UqlL7bGUUH9DJOEXR i/0AYwhISxYOq+beAmBJTn/iitPxmgW5h1daD6mCDmDZhLNSOPCt5VGorYYDw1DXw3DK xuLq2nzH0os8pEiLmapWTkrON6snZWgljDpRmRS3rdJNTwHr5XMvSfCFyHT2uJc5tqeU bYIMwbN6ujlzNQx6gAfgVJbIuoA0W63wcp/piU9tSRhXQgTW6BnulRmJJhj9C0khqrob Edd4QZ2VR2SUs9i7GjsdERu+GG/kPN22gkCfdTfuqKb9Jg3hJHgy70QSu2aWz9VF4x1u WapQ== X-Gm-Message-State: AOAM533YwFlrhrIsAnMrXEgUtyr8HHMud28Uf/2EV+rYOQDv6Siq07vB 19lwMdKCzskk68NMOahjPej3QXJi1aDAE1wRjso= X-Received: by 2002:a05:6902:143:b0:628:7cf1:f2a9 with SMTP id p3-20020a056902014300b006287cf1f2a9mr16357436ybh.51.1649517972377; Sat, 09 Apr 2022 08:26:12 -0700 (PDT) MIME-Version: 1.0 References: <20220409075147.136187-1-linux@fw-web.de> <4232341.5fSG56mABF@phil> In-Reply-To: From: Peter Geis Date: Sat, 9 Apr 2022 11:26:01 -0400 Message-ID: Subject: Re: Re: Re: Re: [PATCH] arm64: dts: rockchip: Fix clocks for rk356x usb To: Frank Wunderlich Cc: Heiko Stuebner , Dan Johansen , Frank Wunderlich , "open list:ARM/Rockchip SoC..." , Rob Herring , Krzysztof Kozlowski , Michael Riesch , devicetree , arm-mail-list , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RDNS_NONE, SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Apr 9, 2022 at 7:56 AM Frank Wunderlich wrote: > > Hi, > > so to not break the binding and other boards the right Patch should be like this > > +++ b/drivers/usb/dwc3/core.c > @@ -1691,17 +1691,17 @@ static int dwc3_probe(struct platform_device *pdev) > * Clocks are optional, but new DT platforms should support all > * clocks as required by the DT-binding. > */ > - dwc->bus_clk = devm_clk_get_optional(dev, "bus_early"); > + dwc->bus_clk = devm_clk_get_optional(dev, "bus_clk"); > if (IS_ERR(dwc->bus_clk)) > return dev_err_probe(dev, PTR_ERR(dwc->bus_clk), > "could not get bus clock\n"); > > - dwc->ref_clk = devm_clk_get_optional(dev, "ref"); > + dwc->ref_clk = devm_clk_get_optional(dev, "ref_clk"); > if (IS_ERR(dwc->ref_clk)) > return dev_err_probe(dev, PTR_ERR(dwc->ref_clk), > "could not get ref clock\n"); > > - dwc->susp_clk = devm_clk_get_optional(dev, "suspend"); > + dwc->susp_clk = devm_clk_get_optional(dev, "suspend_clk"); > if (IS_ERR(dwc->susp_clk)) > return dev_err_probe(dev, PTR_ERR(dwc->susp_clk), > "could not get suspend clock\n"); > > but this needs fixing dts using the new clock names > > this is a link to the series moving from bulk_clk to named clocks: > > https://patchwork.kernel.org/project/linux-usb/patch/20220127200636.1456175-3-sean.anderson@seco.com/ > > regards Frank I've submitted a fix for the backwards compatibility issue. https://patchwork.kernel.org/project/linux-rockchip/patch/20220409152116.3834354-1-pgwipeout@gmail.com/ This fix is standalone and necessary no matter which route we decide to go with this series (and the rk3328/rk3399 support as well). With this patch, dwc3 is functional on the rk356x as the series was submitted, so if we decide to fix everything all at once, that is a viable option.