From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-f47.google.com (mail-oa1-f47.google.com [209.85.160.47]) (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 1B7F117B418 for ; Mon, 22 Sep 2025 22:15:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758579333; cv=none; b=MiMXIR5UbCiJFHklecSqeRn9CMwk4deYIOGdqw+mJ/bb2C1jTq+OVgGuE5+PGDDQuKDnccbBp1tTLUf+8ipH9yDb+ve5TAoDqZMCaVIMIdk4jFRObQ8au7Ia8qaPzy0bdglfPO3l74Lif81+VPohpULTXkcjbRNikKHneiDzxSw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758579333; c=relaxed/simple; bh=RzkyAiWGPIV6p+AvKmAXHMrRF6qrXdOWtTLyoFs77fs=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=C1S/6ImTKMujGJ774sBoujVhDBN3acIyM394DbTktUOFQI+EiSn1fHsIpMcpwiMI7sTUKaNJdoy68xdac/D6Sz9BWsxES6u7QFHIJ3Q4heOchkoF40/Otfi0lwh5TjOVfOFGE+pnUZMOrv5LRrAchtQnMActxcW+KFzo8mLYcRE= 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=by9vcJQe; arc=none smtp.client-ip=209.85.160.47 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="by9vcJQe" Received: by mail-oa1-f47.google.com with SMTP id 586e51a60fabf-341f608661aso1349556fac.3 for ; Mon, 22 Sep 2025 15:15:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1758579331; x=1759184131; 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=3df8lNuWhvDC2rJQvD/ziJ2M1JBls7/FB1ABGRRG/BM=; b=by9vcJQeD6EPColGhUSB5oVuNQ6LxvyKFIFOtCrXS3hghVNVGMZBje/BSgC0SoCcdv 2vEziLwRBaH99SuIKVGaSoSJC1LfTmGFleUKMidl2UjmhoXPspUkdhY97XBhskUpuv6X Wta1/0yOHyC9RIjbcYqTsXayHdelEWOObqGeVEfuIvD5/INnUe9KdHL88n3RHyfXolmJ /hRzd7C1MZTTuyWdkXhZBcZt1rmp0bo/g7PhIu2CC1EA+6PL3O/D1T3Ch4gYFUqlbql4 5/GVHDUBgUMToILGL6tulvnJ08Tlah6No47KoR0SNlGFOKNV891wkX3XuX8tpyXjRXfY 7zPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758579331; x=1759184131; 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=3df8lNuWhvDC2rJQvD/ziJ2M1JBls7/FB1ABGRRG/BM=; b=tR6lWmyEeeVEraiNvaDMNyATSwlDdjp4+f0pjkbdhzoJFUW96/N5asy984AJKXNlaK iAK3g73WSAkUnWcXizc0X9+IU4w8HipiR3AsXiEhWQD3nM1+jh5QsnpoBkpHcaKjkBeC P4SRlE22NWeMxSpPcMSwfbgrmSt4l9wpLrxDGm62wNrjMa82wKo9QQQ9A/7kMQFcxVx9 YWcrsYAR32hAS0+TdA9Z2d86YCOYtUAkt/uneb4s5oH7uKJlt2kLkcTyu762Tko+oDVp oUG/ljdDrJrRhSqfeD++8JaJdM39yvovylGXPZYewu01KilIcAYV6nsGwiqUd4mOsj6N 80sg== X-Forwarded-Encrypted: i=1; AJvYcCVHWj2a7zqIOc91fFM1S6JFAfrCIfVuKWVw8kMmcpPx8kS4xQ4gwSzCohdhHodUajNelnl3qZQUuSBZ@vger.kernel.org X-Gm-Message-State: AOJu0YwWkM6gjsOUVCd2P77JxoWdq7qkm60tYA8hY3EcqUrWeopvMkBZ jfeC3xWW+NhNEInSZcZcxeeE7neMkqV7R94d1qJ6JX6p8U9INT+uwD0+OLDvOJged50= X-Gm-Gg: ASbGncvGdyM9mgl2vp6D8Jvvlrvho6LX5glgEnCXJRkvDEdCsC1NYn1J/b8yNeDHIyc DYcI/v9lMq4joMB7P3cu7aYBik8XRfoRKyel7kpoiAMwJotlYVQmFDAecKuhN8XWXaSn8A+enfk 5TKRhUJMr1xXNmMuaWcnrnLpF+dvDQPrWYyecX8eEWPZllBi3nbtoAOvEvRBHr/lwGo2cxwTH09 j/Td0KxYKY6tQiYnmPK9cASTwIkXBBBGKRMpDCvgrsMflZeVo7Ui2imxz1Z4A/IkG+c+nyTTahd 6czEq0/KLJVYE0F7GjAbqadk8UxxVuxdFAfjIwbD1McaDej5qzsoHv+15bRRmVwcNmBJBxhOUlG 0TFHCSNu8p0e2h1m1udRDG/0yMWovnjzHrPIM7llTCiR2+7RwJqjpWZWoW43ksKMPRuPKvRNtpy w= X-Google-Smtp-Source: AGHT+IFTQfgxCGrhqdxhdussfgDYYkEXHNBEZrXgWgIug9QWcQ3pADFUjI5JaX+cZ8bP6//iw+MOdg== X-Received: by 2002:a05:6871:4b13:b0:319:625d:319f with SMTP id 586e51a60fabf-34c83b0869bmr247841fac.39.1758579331095; Mon, 22 Sep 2025 15:15:31 -0700 (PDT) Received: from ?IPV6:2600:8803:e7e4:1d00:505f:96cd:1359:fff4? ([2600:8803:e7e4:1d00:505f:96cd:1359:fff4]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-336e0fbe706sm8402073fac.0.2025.09.22.15.15.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 22 Sep 2025 15:15:30 -0700 (PDT) Message-ID: <3457c119-2f49-43a3-b96b-736b8f5de99b@baylibre.com> Date: Mon, 22 Sep 2025 17:15:29 -0500 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/2] iio: dac: adding support for Microchip MCP47FEB02 To: =?UTF-8?Q?Nuno_S=C3=A1?= , Ariana Lazar , Jonathan Cameron , =?UTF-8?Q?Nuno_S=C3=A1?= , Andy Shevchenko , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20250922-mcp47feb02-v1-0-06cb4acaa347@microchip.com> <20250922-mcp47feb02-v1-2-06cb4acaa347@microchip.com> <859d8472a8f9e8d28b890ad565f9d3ce11e162d5.camel@gmail.com> Content-Language: en-US From: David Lechner In-Reply-To: <859d8472a8f9e8d28b890ad565f9d3ce11e162d5.camel@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 9/22/25 3:10 PM, Nuno Sá wrote: > Hi Ariana, > > Thanks for your patches. Some initial comments from me... > > On Mon, 2025-09-22 at 14:30 +0300, Ariana Lazar wrote: ... >> +static IIO_DEVICE_ATTR(store_eeprom, 0200, NULL, mcp47feb02_store_eeprom, 0); >> +static struct attribute *mcp47feb02_attributes[] = { >> + &iio_dev_attr_store_eeprom.dev_attr.attr, >> + NULL, >> +}; >> + > > Not going to argue about the ABI for now but I don't think this is a standard one? So > if acceptable you need an ABI doc. > Here's a random idea. (I would wait for Jonathan to weigh in first before assuming it is an acceptable idea though :-p) The config registers are pretty much going to be a one-time deal. So those could be written to only if they need it during probe. For the voltage output registers, we could add extra out_voltageY channels that are the power-on output state channels. So writing to out_voltageY_raw wouldn't change any real output but would just be written to EEPROM. This way these voltages could be controlled independently from the real outputs and it uses existing ABI. In any case, it would be interesting to hear more about how this chips are actually used to better understand this EEPROM feature.