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 31D6EE77188 for ; Mon, 30 Dec 2024 14:09:00 +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:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=YL8J9Xu+G5URlUghB/P+Q8FJUnxHud8TU5M5hm3CpnM=; b=McBdVCEwcaFQSU umT7sHulpc89JapwNyJ3pJywlqPF5mxdZn6++QpwA5tnbIHOpl8iHt24R/TOujxz1f0PudZ9P+phn 7+wJyIjw7mepK8nNujgDpGyItX5tsCnuuDT2vcVNgr9HZglC0QYQDjc88kXQfYxD0KP8xNCn/QDfq dgJsx1bBB9TQEUdtdzgRV7u0/gfSS98ot+QF1znOj0W3gVPLu366qy70qMuOKCTwvmEXhNZmBik7J IF/yiof7NoLJTBLMLsJi0ysfzzKZA0AHrCtvYomR8ffeXkdlwK7sFtEvXOkolbMKwQ5T2Dib6xJh6 Z9Ca8W10mWw0Eq1RUwhg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tSGS3-00000005JzS-1l01; Mon, 30 Dec 2024 14:08:59 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tSGQt-00000005Jt2-3ubz; Mon, 30 Dec 2024 14:07:49 +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 EDE42143D; Mon, 30 Dec 2024 06:08:11 -0800 (PST) Received: from donnerap.manchester.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D758B3F59E; Mon, 30 Dec 2024 06:07:38 -0800 (PST) Date: Mon, 30 Dec 2024 14:07:28 +0000 From: Andre Przywara To: Parthiban Nallathambi Cc: Joerg Roedel , Will Deacon , Robin Murphy , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Maxime Ripard , Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Michael Turquette , Stephen Boyd , Philipp Zabel , Linus Walleij , Vinod Koul , Kishon Vijay Abraham I , iommu@lists.linux.dev, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-phy@lists.infradead.org Subject: Re: [PATCH RESEND 04/22] dt-bindings: clock: sun8i de2 clock: Add a100/a133 compatible Message-ID: <20241230140728.78629cd3@donnerap.manchester.arm.com> In-Reply-To: <20241227-a133-display-support-v1-4-abad35b3579c@linumiz.com> References: <20241227-a133-display-support-v1-0-abad35b3579c@linumiz.com> <20241227-a133-display-support-v1-4-abad35b3579c@linumiz.com> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241230_060748_053589_52FC3D2F X-CRM114-Status: GOOD ( 19.78 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On Fri, 27 Dec 2024 18:30:53 +0530 Parthiban Nallathambi wrote: Hi, > A100/A133 uses one mixer without rotation support, which is same > as sun8i v3s. Add it with fallback to v3s compatible. > > Signed-off-by: Parthiban Nallathambi > --- > .../devicetree/bindings/clock/allwinner,sun8i-a83t-de2-clk.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/allwinner,sun8i-a83t-de2-clk.yaml b/Documentation/devicetree/bindings/clock/allwinner,sun8i-a83t-de2-clk.yaml > index 3e00905b66ca..ed038967929b 100644 > --- a/Documentation/devicetree/bindings/clock/allwinner,sun8i-a83t-de2-clk.yaml > +++ b/Documentation/devicetree/bindings/clock/allwinner,sun8i-a83t-de2-clk.yaml > @@ -23,6 +23,7 @@ properties: > - const: allwinner,sun8i-h3-de2-clk > - const: allwinner,sun8i-v3s-de2-clk > - const: allwinner,sun50i-a64-de2-clk > + - const: allwinner,sun50i-a100-de2-clk > - const: allwinner,sun50i-h5-de2-clk > - const: allwinner,sun50i-h6-de3-clk > - items: > @@ -31,6 +32,9 @@ properties: > - items: > - const: allwinner,sun20i-d1-de2-clk > - const: allwinner,sun50i-h5-de2-clk > + - items: > + - const: allwinner,sun50i-a100-de2-clk > + - const: allwinner,sun8i-v3s-de2-clk Wait, does this add the same compatible string twice, once as a standalone, once with a fallback? This doesn't look right. If you positively know already about an incompatibility, just provide the single string. If you suspect there *might* be something, use the fallback version for now, and we can revisit that later. Also please note that the semantics of the fallback is: "Any driver supporting only the V3s can also support the A100 version of the clock, as the new device is either fully compatible or a subset of the old one." So what would it be then here? The driver patch [16/22] suggests it's compatible, but that might just cover the part that the Linux driver implements? If it is compatible, you wouldn't need any change to the driver code at all, the DT core code takes care of matching the driver using the fallback. Cheers, Andre > > reg: > maxItems: 1 > -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy