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=-15.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, 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 AA0CAC433E0 for ; Mon, 25 Jan 2021 12:01:45 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 5E5FE22D50 for ; Mon, 25 Jan 2021 12:01:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5E5FE22D50 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-ID: Subject: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=codB1p9ihsj7W9/M79DIkeZl9TrxOpPuhqAOQ6HyxdI=; b=YwwC1TV+HLH75XMX9IXnjMnb+ UY01/ZyUAFKLG6pXjGuFWZ5x34LcwrzqM8YForHU2hwyiviXtm3phAjOUaB3cnSLLze+FUgIlDEin 4Kxd9muCMdfCWku3FhCqTDYPiL4+u+21sGQ9Cc4MEgvhtA+HAxcuJO1rxDU9jPTlbwW7QrM+6/fpp 1feA95v08uJND16scrzNYnA16hFKv4l/IYzjGt+u08Un0b9M9brlKoo5fdSee/zWlbcNwoLru1wkq u9x0hqRaBev/aps1zEnlfhpz9Nsqh5xsJfZNCd0yx4niU//2kVPV/Tf/OuOI5O3xDQfUMxKE0Xilb 9PZVeEQpQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l40Xj-0004ME-R7; Mon, 25 Jan 2021 12:00:27 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l40Xg-0004LX-P5 for linux-arm-kernel@lists.infradead.org; Mon, 25 Jan 2021 12:00:25 +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 9DB8A11FB; Mon, 25 Jan 2021 04:00:13 -0800 (PST) Received: from slackpad.fritz.box (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A8D0B3F66E; Mon, 25 Jan 2021 04:00:10 -0800 (PST) Date: Mon, 25 Jan 2021 11:59:32 +0000 From: Andre Przywara To: Samuel Holland Subject: Re: [PATCH v3 18/21] dt-bindings: allwinner: Add H616 compatible strings Message-ID: <20210125115932.6b9f148d@slackpad.fritz.box> In-Reply-To: References: <20210118020848.11721-1-andre.przywara@arm.com> <20210118020848.11721-19-andre.przywara@arm.com> Organization: Arm Ltd. X-Mailer: Claws Mail 3.17.1 (GTK+ 2.24.31; x86_64-slackware-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210125_070024_907615_9489FC57 X-CRM114-Status: GOOD ( 28.50 ) 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: Alexandre Belloni , Yangtao Li , Linus Walleij , linux-sunxi@googlegroups.com, linux-i2c@vger.kernel.org, linux-rtc@vger.kernel.org, Gregory CLEMENT , Chen-Yu Tsai , =?UTF-8?B?Q2zDqW1lbnQgUMOpcm9u?= , Shuosheng Huang , linux-media@vger.kernel.org, devicetree@vger.kernel.org, Maxime Ripard , Mark Brown , Mauro Carvalho Chehab , linux-arm-kernel@lists.infradead.org, Alessandro Zummo , Jernej Skrabec , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, Icenowy Zheng Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, 17 Jan 2021 22:28:47 -0600 Samuel Holland wrote: Hi, > On 1/17/21 8:08 PM, Andre Przywara wrote: > > Add simple "allwinner,sun50i-h616-xxx" compatible names to existing > > bindings, and pair them with an existing fallback compatible string, > > as the devices are compatible. > > This covers I2C, infrared, RTC and SPI. > > > > Use enums to group all compatible devices together. > > > > Signed-off-by: Andre Przywara > > Acked-by: Rob Herring > > Acked-by: Wolfram Sang # for I2C > > --- > > .../bindings/i2c/marvell,mv64xxx-i2c.yaml | 21 +++++++------------ > > .../media/allwinner,sun4i-a10-ir.yaml | 16 ++++++-------- > > .../bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 3 +++ > > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > > 4 files changed, 17 insertions(+), 24 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > > index 5b5ae402f97a..eb72dd571def 100644 > > --- a/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > > +++ b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > > @@ -18,21 +18,14 @@ properties: > > - const: allwinner,sun4i-a10-i2c > > - const: allwinner,sun6i-a31-i2c > > - items: > > - - const: allwinner,sun8i-a23-i2c > > + - enum: > > + - allwinner,sun8i-a23-i2c > > + - allwinner,sun8i-a83t-i2c > > + - allwinner,sun50i-a64-i2c > > + - allwinner,sun50i-a100-i2c > > + - allwinner,sun50i-h6-i2c > > + - allwinner,sun50i-h616-i2c > > - const: allwinner,sun6i-a31-i2c > > - - items: > > - - const: allwinner,sun8i-a83t-i2c > > - - const: allwinner,sun6i-a31-i2c > > - - items: > > - - const: allwinner,sun50i-a64-i2c > > - - const: allwinner,sun6i-a31-i2c > > - - items: > > - - const: allwinner,sun50i-a100-i2c > > - - const: allwinner,sun6i-a31-i2c > > - - items: > > - - const: allwinner,sun50i-h6-i2c > > - - const: allwinner,sun6i-a31-i2c > > - > > - const: marvell,mv64xxx-i2c > > - const: marvell,mv78230-i2c > > - const: marvell,mv78230-a0-i2c > > diff --git a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-ir.yaml b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-ir.yaml > > index 5fa19d4aeaf3..6d8395d6bca0 100644 > > --- a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-ir.yaml > > +++ b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-ir.yaml > > @@ -20,16 +20,12 @@ properties: > > - const: allwinner,sun5i-a13-ir > > - const: allwinner,sun6i-a31-ir > > - items: > > - - const: allwinner,sun8i-a83t-ir > > - - const: allwinner,sun6i-a31-ir > > - - items: > > - - const: allwinner,sun8i-r40-ir > > - - const: allwinner,sun6i-a31-ir > > - - items: > > - - const: allwinner,sun50i-a64-ir > > - - const: allwinner,sun6i-a31-ir > > - - items: > > - - const: allwinner,sun50i-h6-ir > > + - enum: > > + - allwinner,sun8i-a83t-ir > > + - allwinner,sun8i-r40-ir > > + - allwinner,sun50i-a64-ir > > + - allwinner,sun50i-h6-ir > > + - allwinner,sun50i-h616-ir > > - const: allwinner,sun6i-a31-ir > > > > reg: > > diff --git a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > index 37c2a601c3fa..97928efd2bc9 100644 > > --- a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > +++ b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > @@ -26,6 +26,9 @@ properties: > > - const: allwinner,sun50i-a64-rtc > > - const: allwinner,sun8i-h3-rtc > > - const: allwinner,sun50i-h6-rtc > > + - items: > > + - const: allwinner,sun50i-h616-rtc > > + - const: allwinner,sun50i-h6-rtc > > Since H6, the RTC manages the 24 MHz DCXO, so it provides a fourth clock > output. If this is easy to change later, then it is fine for now, but > maybe it is better to get the H616 binding correct from the beginning? So you mean that RTC register +0x160 controls the system HOSC clock, so the main input clock for all the PLLs and other clocks? And by clearing bit 1 in there we can stop this? And if that is the case, do you suggest that we should model this in the DT, so that the fixed-clock "<&osc24M>" should be replaced with "<&rtc 3>"? So from a "the DT describes the hardware" point of view that would probably the right way, but not sure if Linux is happy about that. At the very least that would mean to extend the RTC driver to export a fourth clock, and all devices would now depend on the RTC (also for probing!). And Linux can realistically never turn that clock off anyway (without grinding to a halt), so this register is more useful for ARISC firmware? So I am somewhat undecided: changing this for the H6 would make newer DTs unusable on older kernels, without anything we really gain. When we really want this, we should indeed use the opportunity to introduce this at least for the H616 from day one, to avoid this situation here. But this requires more changes than just the binding, doesn't it? Cheers, Andre > > > reg: > > maxItems: 1 > > diff --git a/Documentation/devicetree/bindings/spi/allwinner,sun6i-a31-spi.yaml b/Documentation/devicetree/bindings/spi/allwinner,sun6i-a31-spi.yaml > > index 7866a655d81c..908248260afa 100644 > > --- a/Documentation/devicetree/bindings/spi/allwinner,sun6i-a31-spi.yaml > > +++ b/Documentation/devicetree/bindings/spi/allwinner,sun6i-a31-spi.yaml > > @@ -25,6 +25,7 @@ properties: > > - enum: > > - allwinner,sun8i-r40-spi > > - allwinner,sun50i-h6-spi > > + - allwinner,sun50i-h616-spi > > - const: allwinner,sun8i-h3-spi > > > > reg: > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel