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=-6.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 393F0C2BA19 for ; Mon, 13 Apr 2020 10:47:04 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 004FC206E9 for ; Mon, 13 Apr 2020 10:47:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="rLRm8VBT" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 004FC206E9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: 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=epjgDKUO7VcGV7KHx1rf2WeXCCokg6OdPGnpCUwvR5Y=; b=rLRm8VBTRpcf++ 1X3yieoEG/EOY5aKKnMhl/JrN6tSXOrNA4jFSQDkQzC+AwgXBx4ulwYZW5kEjW8/2l12UfwLQoAYe vlH3ZSKSAFRTk6hehPiDoenW8Q/PomfIPBZD76C/duxddflzH/9Y7HdB3FVJpHvLuOehkqTiAY/cU VYpDwh/NRYFl7Dnss6jsD9rcjnMXiSveFSwrpF3vqZwez835JRDrWz4CUSr3i+N3rgZX2eHXZ4uLS Rl0G3Ocyu56Z2HMgpQKwSCg25YW2WMEAkB1PYv7FqMdPtOhS9XyZ6zEo+ffhxasZCPDTUwdmX60G7 /eLQFuD5GsBO1wc4+PeQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jNwcH-0006LL-57; Mon, 13 Apr 2020 10:47:01 +0000 Received: from relay1-d.mail.gandi.net ([217.70.183.193]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jNwcE-0006KX-0H for linux-arm-kernel@lists.infradead.org; Mon, 13 Apr 2020 10:46:59 +0000 X-Originating-IP: 86.202.105.35 Received: from localhost (lfbn-lyo-1-9-35.w86-202.abo.wanadoo.fr [86.202.105.35]) (Authenticated sender: alexandre.belloni@bootlin.com) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 1FA72240004; Mon, 13 Apr 2020 10:46:52 +0000 (UTC) Date: Mon, 13 Apr 2020 12:46:52 +0200 From: Alexandre Belloni To: Claudiu.Beznea@microchip.com Subject: Re: [PATCH 4/5] ARM: dts: sam9x60: add rtt Message-ID: <20200413104652.GE3628@piout.net> References: <1586536019-12348-1-git-send-email-claudiu.beznea@microchip.com> <1586536019-12348-5-git-send-email-claudiu.beznea@microchip.com> <20200410222658.GB3628@piout.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200413_034658_186519_AA9B9196 X-CRM114-Status: GOOD ( 22.21 ) 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: mark.rutland@arm.com, a.zummo@towertech.it, jason@lakedaemon.net, devicetree@vger.kernel.org, maz@kernel.org, linux-kernel@vger.kernel.org, Ludovic.Desroches@microchip.com, robh+dt@kernel.org, tglx@linutronix.de, linux-arm-kernel@lists.infradead.org, linux-rtc@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 13/04/2020 08:51:12+0000, Claudiu.Beznea@microchip.com wrote: > > > On 11.04.2020 01:26, Alexandre Belloni wrote: > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > > > On 10/04/2020 19:26:58+0300, Claudiu Beznea wrote: > >> Add RTT. > >> > >> Signed-off-by: Claudiu Beznea > >> --- > >> arch/arm/boot/dts/at91-sam9x60ek.dts | 5 +++++ > >> arch/arm/boot/dts/sam9x60.dtsi | 7 +++++++ > >> 2 files changed, 12 insertions(+) > >> > >> diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts > >> index ab3d2d9a420a..4020e79a958e 100644 > >> --- a/arch/arm/boot/dts/at91-sam9x60ek.dts > >> +++ b/arch/arm/boot/dts/at91-sam9x60ek.dts > >> @@ -617,6 +617,11 @@ > >> }; > >> }; > >> > >> +&rtt { > >> + atmel,rtt-rtc-time-reg = <&gpbr 0x0>; > >> + status = "okay"; > > > > Is there any point using a gpbr register while there is already a much > > better RTC in the system? > > Only to have it also enabled. > Why would one use the RTT while the RTC is far superior? > > > > In any case, this diff should be merge with the other at91-sam9x60ek.dts > > change instead of being with the dtsi change. > > The changes in this patch are related to enabling the RTT. The other dts > change is related to enabling gpbr. The RTT uses that enabled gpbr -> one > change per patch. > > If you still want to merge then, I'll do it, but then it becomes mixed. > This patch is already mixing add the gpbr in sam9x60ek and add the node in sam9x60.dtsi which is worse. Just have one patch adding the rtt node to the sam9x60.dtsi and then a patch adding the RTT to sam9x60ek. Because the RTT uses the gpbr, it is a good time to add enable the gpbr, this is a single functionnal change. Let's say that for some reason, the RTT patch on sam9x60ek has to be reverted, then the RTT node is still defined which is good for all the other eventual users. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel