Received: by 2002:ac8:5349:0:b0:427:7d78:cd45 with SMTP id d9csp266682qto; Tue, 19 Dec 2023 07:42:01 -0800 (PST) X-Google-Smtp-Source: AGHT+IGTeCV7+Z5LZ8+xBpCzx5EU+GEeB9ZowLx+J+1kJ+R1bknkXYGeJv3S2XJknmwUuol/aKDp X-Received: by 2002:a05:622a:1308:b0:423:d652:6afb with SMTP id v8-20020a05622a130800b00423d6526afbmr27102195qtk.33.1703000521327; Tue, 19 Dec 2023 07:42:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703000521; cv=none; d=google.com; s=arc-20160816; b=K+Mxdp4XC+ei5cSmXBqGx0Rk219hSgPDFfaA+t6dwJ3iJ7JFugAskxCE5d4p/TQwLL A+PuLFaF4A2a/UTPYPGEipmsyKliZbyT0Gc4XZnFiem5wZnO6pJrZ9C102uKBspz51yI NYN1E8RfttS8dnApRDJRkCH7ttwnhnWpBlTLO9Iqht7g1XR+q/9BvKRmXBKJIJH81fAO qVDKTs4Me0Ziy/z4hTTSARLZAyBiCO/zoYz1QvXF3iTfAJf//OSCSbB2FlM6kI26CPdD etAPZW+OUSfNgk6dRyZ0K2Y7umgDSILTbeVIGiRqWkMz4L6REF/Jlpqb/HV2bPl6Y7FX HLvg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=organization:in-reply-to:content-disposition:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:references :message-id:subject:cc:to:from:date:dkim-signature; bh=PfSxNhYlpowGcj6Fhn73kQvcgcRmIzhpwHNwlSC+dhk=; fh=NLsHMCkW9oUPGVAUwi4YjxY06qY4Y90bV6UHFXN6ohE=; b=RkMh/j8fRd/vmg/p3s3htM11EXplu2gOi9FRJubNbGk3kaHV2sUv9glMLlYq21yq51 v7YwhkO9XvojVjlgGpTbWAwJincmlTJ/YdfMGt/Y6e6qb0Dvt/+NyXiEdbmSvXPPu3Oe 4oEcnmAuwV8LcEeJSI2V2E1Ia2qUd/oDVPy58vqsZI8O/xjgfMn5B55POSNT+KOrkmUc 1lQIfVis2DNoBxw96GOvC0hLjhYO5X1mYHq2fz9qdmzhGFU7Pyw4R72yLNxRzzXhzB/b Gc71BrHIcC2oQbOlprT9LxvPm+w9hcSIwbK4xhuN5lp63OGwnPH4A7r5xAHQC2yMDiPP Oa7Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=PtMHchdS; spf=pass (google.com: domain of linux-kernel+bounces-5503-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-5503-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id fy15-20020a05622a5a0f00b00425896218f7si29383526qtb.566.2023.12.19.07.42.01 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 Dec 2023 07:42:01 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-5503-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=PtMHchdS; spf=pass (google.com: domain of linux-kernel+bounces-5503-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-5503-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 2E9C51C24742 for ; Tue, 19 Dec 2023 15:42:01 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 103C91CA98; Tue, 19 Dec 2023 15:41:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="PtMHchdS" X-Original-To: linux-kernel@vger.kernel.org Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4840D3456C; Tue, 19 Dec 2023 15:41:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1703000492; x=1734536492; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=UZ7LvyivSueEAM7MAiZ7KtzVMUOrn3bCVp4/8Q3XqY0=; b=PtMHchdSroO3KSPwoy+wBmfLLxTB6f29QYFcNtg1SK5suTgukN4izx6i XNFhnCarBQprQhCluAXxpSXj/gvZmv6tH/hOD68a5sbb/vz/syCgVDFko tqs7piAJ3tVxgD+m0oEGYdlVqnDYeQZ/XJuLxxG/3HUbCf7AfyRRNm1OF yXzuVyd3Pwvp24/7DiNG4s30RuQNNFXHVBZJo6FIeJ5ZNCRHWFzAVf1kD LijlrzM20LRwBiTkAaA6XUTICnu/+lI/Ehd57W66JAMlL3poeB8zzXD3r 8adjulOStGMpQ3mjg11lPUOXyntGwtHEPfQfdVIozWQTBIRcKHCB0XkRa A==; X-IronPort-AV: E=McAfee;i="6600,9927,10929"; a="380660571" X-IronPort-AV: E=Sophos;i="6.04,288,1695711600"; d="scan'208";a="380660571" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Dec 2023 07:41:31 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10929"; a="949206515" X-IronPort-AV: E=Sophos;i="6.04,288,1695711600"; d="scan'208";a="949206515" Received: from smile.fi.intel.com ([10.237.72.54]) by orsmga005.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Dec 2023 07:41:29 -0800 Received: from andy by smile.fi.intel.com with local (Exim 4.97) (envelope-from ) id 1rFcDl-00000007Ibg-3TvI; Tue, 19 Dec 2023 17:41:25 +0200 Date: Tue, 19 Dec 2023 17:41:25 +0200 From: Andy Shevchenko To: Petre Rodan Cc: linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, Jonathan Cameron , Lars-Peter Clausen , Angel Iglesias , Matti Vaittinen , Andreas Klinger , Rob Herring , Krzysztof Kozlowski Subject: Re: [PATCH 2/2] iio: pressure: change driver for Honeywell MPR series Message-ID: References: <20231219130230.32584-1-petre.rodan@subdimension.ro> <20231219130230.32584-3-petre.rodan@subdimension.ro> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231219130230.32584-3-petre.rodan@subdimension.ro> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo On Tue, Dec 19, 2023 at 03:02:21PM +0200, Petre Rodan wrote: > ChangeLog > - rewrite flow so that driver can use either i2c or spi as communication bus > - add spi driver (tested on MPRLS0015PA0000SA) > - add pressure-triplet property that automatically sets pmin, pmax > - fix transfer-function enum typo based on previous review [1] > - fix interrupt example in binding file (FALLING -> RISING edge) > - indentation changes based on previous code reviews > - renamed mpr_read_pressure to mpr_read_conversion since the sensor is > supposed to also provide temperature measuremets > (but I think mine is broken since the raw temperature value is always > 0x800000 - so temp reading not currently implemented) Changelog... > I've been told in the past that the use of mutexes is redundant in these cases > so please assess if the guard() from the driver code is required or should be > removed. A comment... > patch uses device_property_match_property_string() from the 'togreg' branch Another comment... > [1]: https://lore.kernel.org/lkml/20231116-grudge-hankering-b7a71d831b93@squawk/T/ Some Link: (should it be a tag?)... Where is the commit message? Or is this just an intermediate change to show what will be in the next version? I'm confused. -- With Best Regards, Andy Shevchenko