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 D98DAC433EF for ; Fri, 10 Jun 2022 10:41:32 +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:MIME-Version:References: 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=hLG38dZOgEieE0Rly7R4G5ka/9CPjOHa6d4Tp5G9eso=; b=gVa7Rextgo3YMj 29zhEBGIrR/JD4reIv6950d8mh/GloCKi/ilp55Nl6ykh6bHO6GbNewai485Y/XD3phgcndQLf9Rk Ily7GXK4y2NF0cTthM+1gIkzNh3tZNrWsHppm3Hpn3ZH6vkHrHzNoS+O6GMFaQyIAKXsZm7K+79lp C84ZGfTQsl2TP4sv8VLUjWuVZHG5o4t5Tc227B5c1gjkr74HXMSac2uh2YXZaNFfKQaSfvr43jtEe cAzVEN3l/g0FjHpbfOBlDVgmb1/guDnFqRtWF8A9dgyNek/FNkpAx3XhK8WIUW4ct5aMVrh27wX/i IEsLdRXUgP0EQjawfF4g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nzc48-007dMM-3M; Fri, 10 Jun 2022 10:40:32 +0000 Received: from relay11.mail.gandi.net ([217.70.178.231]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nzc44-007dKY-Dy for linux-arm-kernel@lists.infradead.org; Fri, 10 Jun 2022 10:40:30 +0000 Received: (Authenticated sender: alexandre.belloni@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 80A23100003; Fri, 10 Jun 2022 10:40:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1654857622; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=CPqa3snjp+V3swy+paQVl3SWlOSGmf0pkLwY6FTDI9U=; b=boAs/cHxD1v49A/IVeVtIuEz8fBNz17KEqTSD8TF0pb3tuwqKcx8ieyXiAvkchawghAV6Z PAZwqCV0PZ49VSLYs9l/wEVXJZQbf/mZhMpMTRM/w0eZnFZr0GbP1cCJvU5zhJebNZDtIl kx90AMeV+gMRuniOD3vAbNsbhrjHAf/rHRuYiHs+VQAQSOtjE/Q9DASLesAreXrLPNIXS6 rop2qSp/Nok87u1iW2IvGGldHlL9pQzrqPcK8lSQDinIvuQOmGjXzZdUW0MePrVKLB6hPy pGiDY36c8H4pB8JHRoGFI8lNpN6NZ0+wMLtQElyftguowNu+A67z91TE8dBikQ== Date: Fri, 10 Jun 2022 12:40:16 +0200 From: Alexandre Belloni To: Peter Korsgaard Cc: "Neeli, Srinivas" , Srinivas Neeli , "a.zummo@towertech.it" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "michal.simek@xilinx.com" , "sgoud@xilinx.com" , "shubhraj@xilinx.com" , "neelisrinivas18@gmail.com" , "devicetree@vger.kernel.org" , "linux-rtc@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "git@xilinx.com" Subject: Re: [PATCH V6 2/2] rtc: zynqmp: Add calibration set and get support Message-ID: References: <20220609140833.1784738-1-srinivas.neeli@xilinx.com> <20220609140833.1784738-2-srinivas.neeli@xilinx.com> <87r13x28o4.fsf@dell.be.48ers.dk> <87mtel287y.fsf@dell.be.48ers.dk> <87pmjgyghw.fsf@dell.be.48ers.dk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <87pmjgyghw.fsf@dell.be.48ers.dk> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220610_034028_646940_C1B06FF0 X-CRM114-Status: GOOD ( 19.25 ) 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 10/06/2022 12:12:11+0200, Peter Korsgaard wrote: > >>>>> "Neeli," == Neeli, Srinivas writes: > > Hi, > > >> >> Where do you see that? ug1085 rev2.2 still lists: > >> >> > >> >> Programming Sequences > >> >> init rtc > >> >> 1. Write the value 0019_8231h into the calibration register, CALIB_WRITE. > >> >> > >> > >> > This is a very weird value. Last time I had a look, I couldn't find an > > >> explanation for it. > >> > >> Agreed. I am not arguing that it is sensible, I just wondered where this new > >> value comes from? > >> > > This information is missing in our TRM, as per the internal RTC IP > > design specification document, the calibration value should be crystal > > clock frequency minus one. > > For a crystal oscillator of 32.768KHz(0x8000) frequency, the calibration value will be 0x7FFF. > > > Will update TRM with the required information. > > Ok, great! > > Please put this change in a separate commit from the get/set logic. I'm not sure it is worth it as it is probably being set (to the wrong value) from your device tree anyway and the driver won't forcefully change it in that case. The worse that can happen is that your new boards will have the correct value. -- Alexandre Belloni, co-owner and COO, 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