From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oi1-f180.google.com (mail-oi1-f180.google.com [209.85.167.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4F09722F389 for ; Wed, 13 Aug 2025 21:19:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755119955; cv=none; b=XPtI4fdsM5oHp6jwA0bepRGifXDS7U1ScsXCuR3TiwrCVVM0ad3KboTmlRrtol7di2vI5FkzRjmrQwpFnizQKLGbcdEOEXPMfdkeJCbaUPz3rb9alukFNHUoFMj3lvaK2uSL1An8FKtVA/Wu2ixmVaefAxc603Zrexa6ds9y0HU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755119955; c=relaxed/simple; bh=5Dtc8i2j9SZ7V4cNxuQ2cMweRkAia6s3wQYOCEF0+k8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=FzJHE05xPe2cXYEaDQ8lkDaxLJOmdlq8SMQD3dhtrZPmZAk6z71eB84LT30Pvd2qoS16GTFl3II0ZKrU588hmmESNB1koi4GSlP4Yb6ZFREkbdtISbwQD9TTQfCz3CWG+P/P4USjInXbODdk1+MWyrqPZuYmB3O+xPKuTavTutI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=dlEyGOU6; arc=none smtp.client-ip=209.85.167.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="dlEyGOU6" Received: by mail-oi1-f180.google.com with SMTP id 5614622812f47-435de86378eso230678b6e.3 for ; Wed, 13 Aug 2025 14:19:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1755119952; x=1755724752; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=RAZbSGzMF2aCOavRS9phDDmLw+wZHgg5nRlBuGSNPYY=; b=dlEyGOU67qauKmm0S91jJ3I/RELB+Sl7XLjlf6FWkpZciybF3rko5cU1YIFkApFSvu HxuWRBZhDDBdOPJPy6qiEkF3hODoeL4xUer+5AL/Sdg8j4xiDs1JFqEc6S8fY/n7adbo vwbL+6yR9csELGEVJoKiS9CYu7Btz2GeaNwUylqoQOhT73WbTD0mEb57ZxpczkUu2ww7 GE9F2axaXlIWpTTY28czSWeUWI8gBWPmQnygRKHylf46wzr6tUyMuG/6cC+nEDCUBQuA Yp5d07RJYqv49JEcZtL0OP0ub+vMh0MmEXf0J8Cre9TaNCghGqFVSvgQ40Jx6tZksfWN jotw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1755119952; x=1755724752; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=RAZbSGzMF2aCOavRS9phDDmLw+wZHgg5nRlBuGSNPYY=; b=fzaxNUeQ2Xe9b5HqaPCROJc+2RaRKgzE65exTSe6aHstzCm4ojY9mtoAAdd4Y5kxVx mfA21BaoqcfVLIthr1gWpGGU+eBjxQIy/pKsupaWu3Qu8cadfbFnHDC/l/kY5sKtUfxl dTgEfjmDxvZnM61017nKkqKGty0DbLvHW4nVVL4qGhq+tbwZdsTzkiks+JahbAqeQ9vb g6WWbXvo2YnzpSFYQANw6/TCzN/pkgtuIxMMPmXIU7lnqMshvmHwu3EaWzSPo61KBauA IHBa937V7yW1WEAh7UadlU/JR8rx5f+/hDdFSlrObQYo2ytjIyg8pqDI2e8Xn2yPEOZI Gt2Q== X-Gm-Message-State: AOJu0Yy83KRC9vXZ9P/zmHvZ44GoHBR6/rfkoA/8ZqGeRIYiGqCstdFx G0KX39kchR8TB9LdKfMzYJM+yuF5Kkl1Q/SRWeMu/P6pDs1bKNbw1i4E8txGC2sAf5s= X-Gm-Gg: ASbGnctQLLoJZt20odve/ACDpA7JJa3GqXpntegNam811NcfTm5p0LJMif4dh2z9HXL MIlqX7MSvZ4eJKHUpy3yi+zhzYu1QRytZHvjC6vu7SIIvgjcifhsaXCB46iL38yd1nfE2nRhNux Em3ghLLY5C9U4OaDMNHmxdlc32u/zRprPoMDpWH+ZH2JewNqs69IO/84zO0lQxBdZpq1VAeXukS Vmj0sKsVaV4jHsON6UzB5wqTFaFObmy+1/MsKLajYgiPMXuQ+D31psdbS8M7ieOJp0CwQUuX4Dp T1K2FaOoniWM8cTbte7rnXKA6UcI9YuVqTigj6hxI11UzgYyYy2zIZogqJClXWiCBhpxZ8U8bpt kk0fPgTsAyGA6nwSI+awWfMDFSadMfIx8ULaUimATMarkXry39AGPNWRDeXlPT4J5fCqg7YYD X-Google-Smtp-Source: AGHT+IGHPcA55SVa24yWW3URShXaLfqVASGeLPjXlijGLEddyw8GnVhiSSMBPgkVFwlHujOhRbTPHQ== X-Received: by 2002:a05:6808:4a41:10b0:435:8506:2263 with SMTP id 5614622812f47-435df7a7f40mr289564b6e.24.1755119952173; Wed, 13 Aug 2025 14:19:12 -0700 (PDT) Received: from ?IPV6:2600:8803:e7e4:1d00:ae46:dfe2:81c8:dde? ([2600:8803:e7e4:1d00:ae46:dfe2:81c8:dde]) by smtp.gmail.com with ESMTPSA id 5614622812f47-435ce856832sm844327b6e.23.2025.08.13.14.19.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 13 Aug 2025 14:19:11 -0700 (PDT) Message-ID: <00a3c27f-34a4-4e56-8f7d-4eeaa3c19556@baylibre.com> Date: Wed, 13 Aug 2025 16:19:10 -0500 Precedence: bulk X-Mailing-List: linux-iio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 4/5] iio: mcp9600: Add support for dtbinding of thermocouple-type To: Ben Collins , Jonathan Cameron , =?UTF-8?Q?Nuno_S=C3=A1?= , Andy Shevchenko Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org References: <20250813151614.12098-1-bcollins@watter.com> <20250813151614.12098-5-bcollins@watter.com> Content-Language: en-US From: David Lechner In-Reply-To: <20250813151614.12098-5-bcollins@watter.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 8/13/25 10:15 AM, Ben Collins wrote: > Adds dtbinding check for thermocouple-type and sets sensor config > to match. Add iio info attribute to show state as well. > > Signed-off-by: Ben Collins > --- ... > @@ -447,6 +492,20 @@ static int mcp9600_probe(struct i2c_client *client) > data = iio_priv(indio_dev); > data->client = client; > > + /* Accept type from dt with default of Type-K. */ > + data->thermocouple_type = THERMOCOUPLE_TYPE_K; > + ret = device_property_read_u32(&client->dev, "thermocouple-type", > + &data->thermocouple_type); ret is not checked. We should either check it or drop it and add a comment explaining why it is OK to ignore the return value. Typically, for optional properties, we would ignore only -EINVAL meaning the property is not present and fail on other errors. We also need another dt-bindings patch to add the default in the bindings. > + if (data->thermocouple_type >= ARRAY_SIZE(mcp9600_type_map)) > + return dev_err_probe(&client->dev, -EINVAL, > + "Invalid thermocouple-type property %d.\n", > + data->thermocouple_type); > + > + /* Set initial config. */ > + ret = mcp9600_config(data); > + if (ret < 0) > + return ret; > + > ch_sel = mcp9600_probe_alerts(indio_dev); > if (ch_sel < 0) > return ch_sel;