From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f50.google.com (mail-ot1-f50.google.com [209.85.210.50]) (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 EB9F471747 for ; Fri, 7 Feb 2025 20:51:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738961505; cv=none; b=LF9V2yKIrvBauJz44jbsGx9OFegyWoQGHLzOtY3ik2LJiq0dYMNV/G9tmbsm/J1x13ggK47xr8aOnKR9nfWCATpsyvS9tLmlqdBlwAuRH7fq9bmO637vKJEyFFbX9zxH39FjbzhSB4T2H9ZJnY2Bd+wOJMpW3Z0LN7I6w0+H9ms= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738961505; c=relaxed/simple; bh=j0K3veYgNs9ITuGH6rfEbzVZGYKqSl9G/AaRQDM6RO4=; h=Message-ID:Date:MIME-Version:Subject:To:References:From: In-Reply-To:Content-Type; b=BcDl43PYcrAh4ji6yjrkwT7RsLZ90xNAR2ZhSR79ZyT/z19XaxT3lU4OV+lRQTV3sRiVWjZBKqzClElgenCGlPqpMbzglpPtMJ9+X0Uob81M6zM++iEjhEwFWR9sPf73moK93Sjb+osfKfXx0MaZY1meSr41r8w2lrjBCIt+lIQ= 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=HHF9C7Tv; arc=none smtp.client-ip=209.85.210.50 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="HHF9C7Tv" Received: by mail-ot1-f50.google.com with SMTP id 46e09a7af769-71e3005916aso770770a34.2 for ; Fri, 07 Feb 2025 12:51:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1738961502; x=1739566302; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=VQGQcdGFinvFOpfGuSYGng0xGWMSfOVhvommmb2mfPo=; b=HHF9C7TvC5v3O5qwMl077YKeKLExv/7juTO/sfK/IO9KN6nXmt935uSSPkBheIBR0a xGQv9RnuYT509ER2xjN4t5gmuGU0dNEebvg6BB4H0SuP4nRo1Rl8OUYHopu4TbiPH8rk 2tXRGmIKGMu95eCN7gX+XJ9LMoHr0x2Fc/WCdAx348jQifRZz8GRkKE1re0iUyoS7tJ/ WrzIhMCoW17SIGvj1g/0n5Zf0aZzpAdCRIIE+5TFVrFdFN32kXobFFbf2Djb/dYHapHn A6YMgBSRnmz7S0yFO7ZMeuQir1XEGSm1iyruTzkyUSkV98Lit1494ANl2exYhiNK8dYQ lzUQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1738961502; x=1739566302; h=content-transfer-encoding:in-reply-to:content-language:from :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=VQGQcdGFinvFOpfGuSYGng0xGWMSfOVhvommmb2mfPo=; b=TLjsw+d980Gy/QnD+sogN9AtjAu4NS7adINpe2F2euQnLHigK/40aplTvZenEsZ5Zo O6BjC8uVZPW0w8UiNxux2EmZchv8PAJxiOlHiPIBwJFgN83uHMDFYiaA9UvjD2LDnREI dYMxSUF2bdkj4o090JFauD87ynYYhjWJLqs9SGLqWrcHO8TRIrUEvBOIfueGu9jzmhZq /2yyDj7rAQCZeewxrg2jNSzo4c1V9TNz1kToIKvEqtcJqgP/K4NbWxzE9mBOYBTCZ6C+ 1V6m9Wy5A0vo1Yc/xCYJYyjLf3qp+LUGteFF/xNevVTBnInpBEEczQYA4FHCNkq9CSDV KMlg== X-Forwarded-Encrypted: i=1; AJvYcCX19LhOYOq2cRawf/FsFkFv/kw/WipyE3sFhUMurzzV3sVOp5bF5/3rfUoNAoKXKYt1UqgR4oM2EVI=@vger.kernel.org X-Gm-Message-State: AOJu0Yxvrf5Eaj+Lg82KFSMgeUXfvfxmfvGWxKAUUAz5HsExeJ5y6hrv LpSw1ttInNdvmmSh+BpS591XIQiGiLcYY7Wf+WZ/FWN07jwp0APppk1JOGhMdLY= X-Gm-Gg: ASbGncsb9tdZfpQ6sF66+8pjnVYomBEju0tGs3ZAQ38oDiylkfeVo5PgCgmONIC5G4o xegtB7C2JKbyGi2joh3c6LeNkZC7JVFCxQmetTYXQLUl61ERo7OWzgLsJZiQ2x2QGYXk55xsQkx QWxsfbirsRAlZsCXgAq2bhLGMt/C1hlb4z4Ej579r9RVOlXL0u9E+MxK8kLnSkjcSe4X8pzaHbW lNWrYVqtfCPZU4OmCgVq0aAOU1qTwxLwPurSf6EMFVQyuS4Cm0Qa9AHPaQAaqEerWwvOB6fqzbU 4Wij+7Mnb3fDNRtrvXJs5sTbpn1po1lih+e0x0PdDkB2vguxpVFL X-Google-Smtp-Source: AGHT+IG7PscmiUU5iaoZJFTzcARWpAGODDHZRUMCO99df9PiCQJBs76r9w1AoVJ5tFHypQESPlOSMA== X-Received: by 2002:a05:6830:438a:b0:71d:f343:5f5b with SMTP id 46e09a7af769-726b8829061mr3412971a34.12.1738961501979; Fri, 07 Feb 2025 12:51:41 -0800 (PST) Received: from [192.168.0.142] (ip98-183-112-25.ok.ok.cox.net. [98.183.112.25]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-726afa4d2b7sm1006099a34.63.2025.02.07.12.51.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 07 Feb 2025 12:51:41 -0800 (PST) Message-ID: Date: Fri, 7 Feb 2025 14:51:40 -0600 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 v12 9/9] iio: adc: ad4851: add ad485x driver To: Antoniu Miclaus , jic23@kernel.org, robh@kernel.org, conor+dt@kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pwm@vger.kernel.org References: <20250207140918.7814-1-antoniu.miclaus@analog.com> <20250207140918.7814-10-antoniu.miclaus@analog.com> From: David Lechner Content-Language: en-US In-Reply-To: <20250207140918.7814-10-antoniu.miclaus@analog.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 2/7/25 8:09 AM, Antoniu Miclaus wrote: > Add support for the AD485X a fully buffered, 8-channel simultaneous > sampling, 16/20-bit, 1 MSPS data acquisition system (DAS) with > differential, wide common-mode range inputs. > > Signed-off-by: Antoniu Miclaus > --- ... > +static int ad4851_parse_channels_common(struct iio_dev *indio_dev, > + struct iio_chan_spec **chans, > + const struct iio_chan_spec ad4851_chan) > +{ > + struct ad4851_state *st = iio_priv(indio_dev); > + struct device *dev = &st->spi->dev; > + struct iio_chan_spec *channels, *chan_start; > + unsigned int num_channels, reg; > + unsigned int index = 0; > + int ret; > + > + num_channels = device_get_child_node_count(dev); > + if (num_channels > AD4851_MAX_CH_NR) > + return dev_err_probe(dev, -EINVAL, "Too many channels: %u\n", > + num_channels); > + > + channels = devm_kcalloc(dev, num_channels, sizeof(*channels), GFP_KERNEL); > + if (!channels) > + return -ENOMEM; > + > + chan_start = channels; > + > + device_for_each_child_node_scoped(dev, child) { > + ret = fwnode_property_read_u32(child, "reg", ®); > + if (ret) > + return dev_err_probe(dev, ret, > + "Missing channel number\n"); > + if (reg >= AD4851_MAX_CH_NR) > + return dev_err_probe(dev, ret, ret will be == 0 here, so need to replace with -EINVAL. > + "Invalid channel number\n");