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 D5C05C7EE23 for ; Tue, 30 May 2023 08:02:16 +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:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=54UdRR1mZC0P+/i5zBHR0o1AoBIt97Ij/8CPU5oE19M=; b=aYn03mYSP+pf4N jejymrKirlJYz5VOXYfy2lYXH66Eg70jeRTwuBQ9MDjUziweTReoHWJ7s5PBqmjERloO6JB/TEfVi rC88d/ExOrmvi7gWkxHNojgQ8UALbA0Kg1ZPqTDUEulvU93hOKXioDZZ3bUOtbVI/581N0YQyu2Wu tFhsUouuesexmkezIosQXcE/bR7PtXS2EFr4VNHo4PIPTxKlm4wEHrZIrfLSaXAe0jLz6p8anBAZh 42HYjGsbUE0ZJS2bQcQ3RqfuyOLHFtC6/+G2edL3E3uw2gPuw5Nv/v+0ZTeCafuGR3Am9+01ZzeE3 WMO4kzwdz+YKScyzh1vA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q3uIg-00Cqgc-2o; Tue, 30 May 2023 08:01:50 +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 1q3uIe-00CqfU-2s for linux-arm-kernel@lists.infradead.org; Tue, 30 May 2023 08:01:50 +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 DCD2062609; Tue, 30 May 2023 08:01:43 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 28BCDC433D2; Tue, 30 May 2023 08:01:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1685433703; bh=dVnNYFeOe8bYZ0kkyN7qk+7ollcpFAuTm5euN9ZMmtk=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=akSLrHLgxJfyL51neES4C4w6W3/I+AYA5O0vGEADptFBH4hsvLo0f+zg8qCDNxLgJ 5BJprPK8v3OP1BgMzMu8prWIHoyyEoy6TZYtCGrLRXaVOuEgwMkXokf8q7oRiLd25r +EiOFSuyVy2dJ0f+oValEDS0y3HjmF0lKJvxJjFChRiCALi0UBSuWoqHAQUOliuxjd tpjzLrmy5f524uM68ILReMj3vNO8IWZ9DyiTMD1vn2db5d3R56waEuCu+/6KXn/cdy CyJTuTR3ct2rtvARSOPahLJY2qD1qSgMGnqTAtbggigaxwgKmwZ+XbsYcYdTCLoyK6 G/em+f9V9wEwA== Message-ID: Date: Tue, 30 May 2023 10:01:39 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH 2/3] dt-bindings: rtc: Add Mstar SSD20xD RTC devicetree bindings documentation Content-Language: en-US To: Romain Perier Cc: Alessandro Zummo , Alexandre Belloni , Daniel Palmer , Rob Herring , linux-rtc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20230517144144.365631-1-romain.perier@gmail.com> <20230517144144.365631-3-romain.perier@gmail.com> <669d7b79-71a6-e1f9-8d7a-71c4b64de28d@kernel.org> From: Krzysztof Kozlowski In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230530_010148_988948_5DFB505C X-CRM114-Status: GOOD ( 22.28 ) 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="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 22/05/2023 13:27, Romain Perier wrote: >>> +properties: >>> + compatible: >>> + enum: >>> + - mstar,ssd20xd-rtc >> >> Why rtc suffix? Can it be anything else? > > Well, it is the dt-bindings for an RTC block ? suppose tomorrow we > have an ethernet block specific to the SoC SSD202D, it should be > "mstar,ssd202d-ethernet" , how do you make > the difference if you just put "mstar,sd202d" ? Plus a lot of rtc > dt-bindings have this suffix (when it is not an IP name). There are a lot of bad design choices or bugs - are you going to implement the same mistakes because someone did it? > This is > exactly the case for rtc-msc313e and it was not an issue. So that was my question - can it be anything else? There is literally no description of the hardware... Neither in commit msg nor in description: field in bindings. What is SSD202D? SoC? RTC? > >> >> Missing blank line > > ack > >> >>> + reg: >>> + maxItems: 1 >>> + >>> + start-year: true >> >> Drop >> >> What about interrupt line? > > There is currently no interrupt right now, we have not yet the irqchip > code for handling the alarm irq of this rtc block. So you are going to change the hardware and add the interrupt line? We do not talk about drivers, but hardware. Whether your driver handles it or not, matters less. Describe the hardware, not the current implementation of one driver. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel