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 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E9417C7EE2E for ; Tue, 13 Jun 2023 02:16:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) id 9638EC433A0; Tue, 13 Jun 2023 02:16:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4EFE0C433D2; Tue, 13 Jun 2023 02:16:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686622565; bh=y+tpkmHrhhV80BKkf0IRp3DxiL2VcneOgIpNiVwOdWs=; h=In-Reply-To:References:Subject:From:List-Id:Cc:To:Date:From; b=rePw19zwE9GhrlFO5ez6Zv7JrNsZKq2b5KihZPsopJy5mv7LfIJiBkXtRidvrLP58 K7ExCf7v3IDwDdpB+qcDB+9PTCAx54zH9UMo7JeSpOOV73Q43kWvQe3W19JasCudaL rmrCpBl/u2aRupE3OAvjA6oSns3zlkcYWXEUjnXhrFa9RUBXPL9p7lnr54l97KCIvQ 8czF+76X7KufQb+cEa3sZJhqRgNgTKRAzqqtna/vC1YvUNSTWhesc8QVjUjurTg/FJ DMyf8Jad2MO9ps8oF6JbiiKkuyibMNceUplWRVuncl9DK3SLKbzwa0DU9NBdb2d4Qa 1eKpxkximL1Bw== Message-ID: Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable In-Reply-To: <233f4e83-e872-5499-2ca7-a1c277425fb0@gmail.com> References: <20230605040749.67964-1-ychuang570808@gmail.com> <20230605040749.67964-9-ychuang570808@gmail.com> <233f4e83-e872-5499-2ca7-a1c277425fb0@gmail.com> Subject: Re: [PATCH v13 08/10] clk: nuvoton: Add clock driver for ma35d1 clock controller From: Stephen Boyd List-Id: Cc: devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-serial@vger.kernel.org, arnd@arndb.de, soc@kernel.org, schung@nuvoton.com, mjchen@nuvoton.com, Jacky Huang , Krzysztof Kozlowski To: Jacky Huang , catalin.marinas@arm.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, krzysztof.kozlowski+dt@linaro.org, lee@kernel.org, mturquette@baylibre.com, p.zabel@pengutronix.de, robh+dt@kernel.org, tmaimon77@gmail.com, will@kernel.org Date: Mon, 12 Jun 2023 19:16:03 -0700 User-Agent: alot/0.10 Quoting Jacky Huang (2023-06-12 18:37:31) > On 2023/6/13 =E4=B8=8A=E5=8D=88 09:02, Stephen Boyd wrote: > > Quoting Jacky Huang (2023-06-04 21:07:47) > >> diff --git a/drivers/clk/nuvoton/clk-ma35d1-divider.c b/drivers/clk/nu= voton/clk-ma35d1-divider.c > >> + int num_pdata) > >> +{ > >> + return clk_hw_register_mux_parent_data(dev, name, pdata, num_p= data, > >> + CLK_SET_RATE_NO_REPAREN= T, reg, shift, > >> + width, 0, &ma35d1_lock); > >> +} > >> + > >> +static struct clk_hw *ma35d1_clk_mux(struct device *dev, const char *= name, > >> + void __iomem *reg, u8 shift, u8 w= idth, > >> + const char *const *parents, int n= um_parents) > > Please don't use string arrays for parent descriptions. Everything > > should use clk_parent_data or direct clk_hw pointers. >=20 > I will use clk_parent_data instead of strings. >=20 When you use clk_parent_data, just don't set anything for the "dummy" clks besides a -1 .index to indicate there isn't a parent for that entry. 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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 55ED2C7EE43 for ; Tue, 13 Jun 2023 02:16:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Date:To:Cc:From:Subject:References: In-Reply-To:MIME-Version:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=GwSkH861PZmE1cvrejUlwbr17CtdfnIGzdgqaBxt+rs=; b=psug0rzLPYj2bL Kp+m9ZJi1W9L1mljapE++ug40xaT/aRHcDWIVKPY4RJyK7gF8fmNfcBagYRzd57/TlLV0R9WHpJ52 /qG8t0IF7ty7PBp1NI1t3wLkImgvjSBQ91G6IP439xWKMbKmCsIh0Yqjhz796+j4kINh82K2Q0QdQ L+fFOwhJNBkD1zQC5mlVX9V4iQAABxTEoouHC4PYrDQ/cimYYdFKGpol49B/chfq609eCL2hx+t1i oybymSeDotKNGakXy2Pm3MJT2YN4vopncnLEVA+nSpROzmWHjGMxQpLsbu5mlEavwyO3eeFoMg6If vj/KcwIrz1kSLnpeOLBg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q8tZp-006eX2-13; Tue, 13 Jun 2023 02:16:09 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q8tZn-006eWS-1y for linux-arm-kernel@lists.infradead.org; Tue, 13 Jun 2023 02:16:08 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 107D86309A; Tue, 13 Jun 2023 02:16:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4EFE0C433D2; Tue, 13 Jun 2023 02:16:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686622565; bh=y+tpkmHrhhV80BKkf0IRp3DxiL2VcneOgIpNiVwOdWs=; h=In-Reply-To:References:Subject:From:List-Id:Cc:To:Date:From; b=rePw19zwE9GhrlFO5ez6Zv7JrNsZKq2b5KihZPsopJy5mv7LfIJiBkXtRidvrLP58 K7ExCf7v3IDwDdpB+qcDB+9PTCAx54zH9UMo7JeSpOOV73Q43kWvQe3W19JasCudaL rmrCpBl/u2aRupE3OAvjA6oSns3zlkcYWXEUjnXhrFa9RUBXPL9p7lnr54l97KCIvQ 8czF+76X7KufQb+cEa3sZJhqRgNgTKRAzqqtna/vC1YvUNSTWhesc8QVjUjurTg/FJ DMyf8Jad2MO9ps8oF6JbiiKkuyibMNceUplWRVuncl9DK3SLKbzwa0DU9NBdb2d4Qa 1eKpxkximL1Bw== Message-ID: MIME-Version: 1.0 In-Reply-To: <233f4e83-e872-5499-2ca7-a1c277425fb0@gmail.com> References: <20230605040749.67964-1-ychuang570808@gmail.com> <20230605040749.67964-9-ychuang570808@gmail.com> <233f4e83-e872-5499-2ca7-a1c277425fb0@gmail.com> Subject: Re: [PATCH v13 08/10] clk: nuvoton: Add clock driver for ma35d1 clock controller From: Stephen Boyd Cc: devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-serial@vger.kernel.org, arnd@arndb.de, soc@kernel.org, schung@nuvoton.com, mjchen@nuvoton.com, Jacky Huang , Krzysztof Kozlowski To: Jacky Huang , catalin.marinas@arm.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, krzysztof.kozlowski+dt@linaro.org, lee@kernel.org, mturquette@baylibre.com, p.zabel@pengutronix.de, robh+dt@kernel.org, tmaimon77@gmail.com, will@kernel.org Date: Mon, 12 Jun 2023 19:16:03 -0700 User-Agent: alot/0.10 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230612_191607_689420_718A9090 X-CRM114-Status: GOOD ( 12.52 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org UXVvdGluZyBKYWNreSBIdWFuZyAoMjAyMy0wNi0xMiAxODozNzozMSkKPiBPbiAyMDIzLzYvMTMg 5LiK5Y2IIDA5OjAyLCBTdGVwaGVuIEJveWQgd3JvdGU6Cj4gPiBRdW90aW5nIEphY2t5IEh1YW5n ICgyMDIzLTA2LTA0IDIxOjA3OjQ3KQo+ID4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2Nsay9udXZv dG9uL2Nsay1tYTM1ZDEtZGl2aWRlci5jIGIvZHJpdmVycy9jbGsvbnV2b3Rvbi9jbGstbWEzNWQx LWRpdmlkZXIuYwo+ID4+ICsgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgaW50IG51bV9wZGF0YSkKPiA+PiArewo+ID4+ICsgICAgICAgcmV0dXJuIGNsa19od19yZWdp c3Rlcl9tdXhfcGFyZW50X2RhdGEoZGV2LCBuYW1lLCBwZGF0YSwgbnVtX3BkYXRhLAo+ID4+ICsg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgQ0xLX1NFVF9SQVRF X05PX1JFUEFSRU5ULCByZWcsIHNoaWZ0LAo+ID4+ICsgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgd2lkdGgsIDAsICZtYTM1ZDFfbG9jayk7Cj4gPj4gK30KPiA+ PiArCj4gPj4gK3N0YXRpYyBzdHJ1Y3QgY2xrX2h3ICptYTM1ZDFfY2xrX211eChzdHJ1Y3QgZGV2 aWNlICpkZXYsIGNvbnN0IGNoYXIgKm5hbWUsCj4gPj4gKyAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgIHZvaWQgX19pb21lbSAqcmVnLCB1OCBzaGlmdCwgdTggd2lkdGgsCj4gPj4g KyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGNvbnN0IGNoYXIgKmNvbnN0ICpw YXJlbnRzLCBpbnQgbnVtX3BhcmVudHMpCj4gPiBQbGVhc2UgZG9uJ3QgdXNlIHN0cmluZyBhcnJh eXMgZm9yIHBhcmVudCBkZXNjcmlwdGlvbnMuIEV2ZXJ5dGhpbmcKPiA+IHNob3VsZCB1c2UgY2xr X3BhcmVudF9kYXRhIG9yIGRpcmVjdCBjbGtfaHcgcG9pbnRlcnMuCj4gCj4gSSB3aWxsIHVzZSBj bGtfcGFyZW50X2RhdGEgaW5zdGVhZCBvZiBzdHJpbmdzLgo+IAoKV2hlbiB5b3UgdXNlIGNsa19w YXJlbnRfZGF0YSwganVzdCBkb24ndCBzZXQgYW55dGhpbmcgZm9yIHRoZSAiZHVtbXkiCmNsa3Mg YmVzaWRlcyBhIC0xIC5pbmRleCB0byBpbmRpY2F0ZSB0aGVyZSBpc24ndCBhIHBhcmVudCBmb3Ig dGhhdAplbnRyeS4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMu aW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2xpbnV4LWFybS1rZXJuZWwK