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=-10.5 required=3.0 tests=BAYES_00,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,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 9DBEBC433FE for ; Sun, 13 Dec 2020 11:58:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5D8CB23121 for ; Sun, 13 Dec 2020 11:58:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2406196AbgLML54 (ORCPT ); Sun, 13 Dec 2020 06:57:56 -0500 Received: from mail.kernel.org ([198.145.29.99]:55352 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2406195AbgLML54 (ORCPT ); Sun, 13 Dec 2020 06:57:56 -0500 Received: from archlinux (cpc108967-cmbg20-2-0-cust86.5-4.cable.virginm.net [81.101.6.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id DEC4023121; Sun, 13 Dec 2020 11:57:14 +0000 (UTC) Date: Sun, 13 Dec 2020 11:57:11 +0000 From: Jonathan Cameron To: Anand Ashok Dumbre Cc: Michal Simek , Lars-Peter Clausen , Peter Meerwald-Stadler , "linux-iio@vger.kernel.org" Subject: Re: [PATCH 2/2] iio: xilinx-xadc: Add basic support for Ultrascale System Monitor Message-ID: <20201213115711.58733caa@archlinux> In-Reply-To: References: <20200922134624.13191-1-lars@metafoo.de> <20200922134624.13191-2-lars@metafoo.de> <20200923214728.6e0c07b0@archlinux> <20201129131441.3e9d83f8@archlinux> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Fri, 11 Dec 2020 09:57:30 +0000 Anand Ashok Dumbre wrote: > > -----Original Message----- > > From: Jonathan Cameron > > Sent: Sunday 29 November 2020 1:15 PM > > To: Michal Simek > > Cc: Lars-Peter Clausen ; Peter Meerwald-Stadler > > ; linux-iio@vger.kernel.org; Anand Ashok Dumbre > > > > Subject: Re: [PATCH 2/2] iio: xilinx-xadc: Add basic support for Ultrascale > > System Monitor > > > > On Thu, 24 Sep 2020 08:08:31 +0200 > > Michal Simek wrote: > > > > > Hi, +Anand, > > > > > > On 23. 09. 20 22:47, Jonathan Cameron wrote: > > > > On Tue, 22 Sep 2020 15:46:24 +0200 > > > > Lars-Peter Clausen wrote: > > > > > > > >> The xilinx-xadc IIO driver currently has support for the XADC in > > > >> the Xilinx > > > >> 7 series FPGAs. The system-monitor is the equivalent to the XADC in > > > >> the Xilinx UltraScale and UltraScale+ FPGAs. > > > >> > > > >> The IP designers did a good job at maintaining backwards > > > >> compatibility and only minor changes are required to add basic > > > >> support for the system-monitor core. > > > >> > > > >> The non backwards compatible changes are: > > > >> * Register map offset was moved from 0x200 to 0x400 > > > >> * Only one ADC compared to two in the XADC > > > >> * 10 bit ADC instead of 12 bit ADC > > > >> * Two of the channels monitor different supplies > > > >> > > > >> Add the necessary logic to accommodate these changes to support the > > > >> system-monitor in the XADC driver. > > > >> > > > >> Note that this patch does not include support for some new features > > > >> found in the system-monitor like additional alarms, user supply > > > >> monitoring and secondary system-monitor access. This might be added > > at a later time. > > > >> > > > >> Signed-off-by: Lars-Peter Clausen > > > > > > > > Very nice. Will let this sit on the mailing list a bit longer but > > > > looks good to me. > > > > > > FYI: I have asked Anand to take a look at it and retest. > > > > > > Thanks, > > > Michal > > > > > Hi Michal / Anand, > > > > Any feedback on this one? > > > > Thanks > > > > Jonathan > > > > p.s. I'm just ploughing through my older emails and had this one marked as > > waiting for a reply. > > > Hi Jonathan, > > Thanks for waiting. > The patch looks good. > > Reviewed-by: Anand Ashok Dumbre > Tested-by: Anand Ashok Dumbre Thanks. I've applied this but there was a bit of complexity around devm changes that went in whilst this was under review. I think I fixed it up correctly, but great if you can take a quick look to make sure I didn't mess it up. I'll try and put the other outstanding series in as well which will probably result in more fiddly corners during the merge. Applied to the togreg branch of iio.git and pushed out as testing or the autobuilders to see what we missed. Thanks, Jonathan >