Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Date:   Sat, 2 Dec 2023 18:06:24 +0200
From:   Petre Rodan <petre.rodan@subdimension.ro>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc:     linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
        Jonathan Cameron <jic23@kernel.org>,
        Lars-Peter Clausen <lars@metafoo.de>,
        Angel Iglesias <ang.iglesiasg@gmail.com>,
        Matti Vaittinen <mazziesaccount@gmail.com>,
        Andreas Klinger <ak@it-klinger.de>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Subject: Re: [PATCH v6 2/2] iio: pressure: driver for Honeywell HSC/SSC series
Message-ID: <ZWtWAPcJTNrD9wgv@sunspire>
References: <20231129170425.3562-1-petre.rodan@subdimension.ro>
 <20231129170425.3562-2-petre.rodan@subdimension.ro>
 <ZWdzz7VzCW5ctend@smile.fi.intel.com>
 <ZWeNNMfqKquDYI9X@sunspire>
 <ZWiPDlNJCbUAtIy8@smile.fi.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZWiPDlNJCbUAtIy8@smile.fi.intel.com>
Precedence: bulk


hello,

On Thu, Nov 30, 2023 at 03:33:02PM +0200, Andy Shevchenko wrote:
> On Wed, Nov 29, 2023 at 09:12:52PM +0200, Petre Rodan wrote:
> > On Wed, Nov 29, 2023 at 07:24:31PM +0200, Andy Shevchenko wrote:
> > 
> > 437:  ret = device_property_read_string(dev, "honeywell,pressure-triplet",
> > 					&triplet);
> > [..]
> > 455:	ret = match_string(hsc_triplet_variants, HSC_VARIANTS_MAX,
> > 						triplet);
> > 		if (ret < 0)
> > 			return dev_err_probe(dev, -EINVAL,
> > 				"honeywell,pressure-triplet is invalid\n");
> > 
> > 		hsc->pmin = hsc_range_config[ret].pmin;
> > 		hsc->pmax = hsc_range_config[ret].pmax;
> > 
> > triplet is got via device_property_read_string(), is there some other property
> > function I should be using?
> 
> I think I mentioned that API, but for your convenience
> device_property_match_property_string().

one of us is not sync-ed with 6.7.0-rc3 :)

cheers,
peter