From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-vk1-f170.google.com (mail-vk1-f170.google.com [209.85.221.170]) (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 83345363C63 for ; Sun, 6 Sep 2026 20:16:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788725772; cv=none; b=Df3jTb1o4ss4jSWltyHU+0kpLkww6/Q+hW5eL9mM3nTuXW0m0jDneoSKjp/ZUEqDYmA9WdBgycCg0sBOfVwQL/FMo3LTh8uLyNxzbu6DQFMnlBgXAuM0/2Btm8Mzi7cQrmT0Xw5CQ65jAwgA9YEawZH+SFJO0VcSGNTp1T7sN28= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788725772; c=relaxed/simple; bh=fO5Lwg7bVKd3xWFVdpuMa1iIDalrBwPzFNje6U0hlhY=; h=Content-Type:Date:Message-Id:To:Cc:Subject:From:Mime-Version: References:In-Reply-To; b=bW6rSVl9GeYGhYyFagK+FGkTr/sVVeG0WjuDmtI0PiZeDF95RvfzJBzr24NW8kGf4+sg3xW+l2U08F9MyQozQqnWXSpfcFBc84PJTzGCuG0mfg3SKc/3RnWdaL57s3wTIby9vBmgBIUPsA4icofxiaItFUOOtOiCVmszF25Li30= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=my1G1fBx; arc=none smtp.client-ip=209.85.221.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="my1G1fBx" Received: by mail-vk1-f170.google.com with SMTP id 71dfb90a1353d-5c7afda4515so1679115e0c.1 for ; Sun, 06 Sep 2026 13:16:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788725769; x=1789330569; darn=vger.kernel.org; h=in-reply-to:references:content-transfer-encoding:mime-version:from :subject:cc:to:message-id:date:content-type:from:to:cc:subject:date :message-id:reply-to:content-type; bh=l2LTI3znDIwovyervFr5DAc883+eC41JzvRn5/loQas=; b=my1G1fBx9KNjMqwZZ0Zw7kTohIIN9ia5e1dKoUkzIk8UC6UcxKfQU0PSiJ5gGmNtG9 cGfmBqgpjNSahdQvtbgU4IcIqjBwKsbKQlBao/ZrpRGtmxTfWK8Cz/ta1IHq0J7DyHdw QC2HAL6BuZdHcDX1wtMYen+KgAtXkx0iQTBAHS7vdek48jew2EY/dGPfy0ZCie5MKwUJ 0+N4Adi561hjPjDET8NtUQIX4v+Bu5p4AFGRb/4tT7ROZ0LvjaJ1xFLZKIOf+UWgHYM8 0lZgnwocjVBquXo2GR81Ca/37TbHCoMFS/tdlLsJYLDRbFvGanUwXwtTbLA4GzE0h/3/ La7w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788725769; x=1789330569; h=in-reply-to:references:content-transfer-encoding:mime-version:from :subject:cc:to:message-id:date:content-type:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=l2LTI3znDIwovyervFr5DAc883+eC41JzvRn5/loQas=; b=jLbe85FflrMTv2DvABO9BbIH6Q53nCmCvR4U5AiJqZnfoVT6EkeCziPcI+EgGmLXll tqI/RmSJple9zpKn8SrirhzE7Y+Gednfne5WiV/UuIzQj+lNkc0J+9Gq+0fxALPz3BV6 bTYBH4ri9gjFXfrvmbLMtPrYpReTyZy+9+WQ6I4rSkvMw8bXsFRm81D474FypoUwpBqI FbbTM32/Wzy03nUksNZfrHwq3XcxOmrp/fDYSatK5IX3+jPICEIth+TkaH4I4gW2FtVw +aazywm2AguLtikU/gKoUgfAywwksucs95GhbH9l+hZ0wcqlAPteZ9oljxhsP1w6mnX4 kwTg== X-Forwarded-Encrypted: i=1; AKwUvBxu7ciGlB99dednI2E3v9rYAk/xdjnTS6vMT5/EWBfks1HbUhEiKYAnNNh/TRBa3oa5gAplNpJBtc4=@vger.kernel.org X-Gm-Message-State: AFuF++ky/mnsetmHaSMYECqTVhDbF0hbhA0sO0NT+eM85a2K65G8v0jV hWzTA35LL8zQD+s8ntNF51g+eqxO6sHUB/xtw1YHhmo0HvGXvohVRQn2 X-Gm-Gg: AYBFou0z7yYzPdGu7DmqnT/hk/02C+2AvjiNV4Gqw+t3KAWZVzCVUm1ju6fMHjU8L/z bDpXjJzAvj68wHJh/vlBhlynI4+N5GTOXsDeALfdWdSlhPgk7mbX4AQOk2bZ1pCiPWOHJ0cDYuE oI1gvOIuMMJLFKbAIRk936bnjNaJ1YAUBkV3FjXMhEP8Z/kgoomFB3rnnBQw1skz85Yk/uM63mi nGR4G2pBiur17KAPQCivk1aAr5Os6OUUyxz5kp8aO8//AOLzCT2uneLMkbg1qv7CpaFJLdqMysJ 9Lkvudeo9loUN6YNxBx690uIqnIsVXkTTJnJo/j32dPW13R0dl5/q/iPJv5Zi4UOV8nuBk21VEG cESIzmpUGHC0dQd10CXvFLV8tHpEKfOZvLzAk7EBMp7/7QLvUs9NqQSr696RznQoUnBSpnbwM30 +90jkVYdPNJMBbR4dPSIlx1iL2oTW1peNhWaBevctg8rx+ZENE4w== X-Received: by 2002:a05:6123:2c8:b0:5bf:b3a0:388c with SMTP id 71dfb90a1353d-5c7ed3e2247mr8141061e0c.6.1788725769334; Sun, 06 Sep 2026 13:16:09 -0700 (PDT) Received: from localhost ([2800:40:44:f1f5:f400:4c49:91c1:9905]) by smtp.gmail.com with ESMTPSA id 71dfb90a1353d-5c7ec23d459sm6411242e0c.9.2026.09.06.13.16.06 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 06 Sep 2026 13:16:08 -0700 (PDT) Content-Type: text/plain; charset=UTF-8 Date: Sun, 06 Sep 2026 17:16:04 -0300 Message-Id: To: "Jonathan Cameron" , "Kurt Borja" Cc: "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "David Lechner" , =?utf-8?q?Nuno_S=C3=A1?= , "Andy Shevchenko" , , , Subject: Re: [PATCH v4 04/10] iio: adc: ti-ads1262: support per-channel sampling frequency From: "Kurt Borja" Precedence: bulk X-Mailing-List: linux-iio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Mailer: aerc 0.21.0-0-g5549850facc2 References: <20260828-ads126x-v4-0-1dc27e9c0260@gmail.com> <20260828-ads126x-v4-4-1dc27e9c0260@gmail.com> <20260830023619.654d1bbe@jic23-huawei> In-Reply-To: <20260830023619.654d1bbe@jic23-huawei> On Sat Aug 29, 2026 at 10:36 PM -03, Jonathan Cameron wrote: > On Fri, 28 Aug 2026 01:38:19 -0500 > Kurt Borja wrote: > >> Add per-channel sampling frequency support. The "available" attribute is >> assigned per-channel too, in order to eventually support per-filter >> availability. >>=20 >> Signed-off-by: Kurt Borja > A build issue below. > >> }; >> =20 >> +static const struct ads1262_channel ads1262_default_channel =3D { >> + .data_rate =3D ADS1262_DR_20_SPS, >> + .filter =3D ADS1262_FILTER_FIR, >> +}; > > See below - this probably wants to be a slightly more complex define. > > > >> @@ -763,8 +970,15 @@ static int ads1262_parse_channels(struct iio_dev *i= ndio_dev) >> if (!chan_specs) >> return -ENOMEM; >> =20 >> + st->num_channels =3D num_fw_channels + ADS1262_MON_CHANNEL_COUNT; >> + st->channels =3D devm_kcalloc(dev, st->num_channels, sizeof(*st->chann= els), >> + GFP_KERNEL); >> + if (!st->channels) >> + return -ENOMEM; >> + >> device_for_each_named_child_node_scoped(dev, node, "channel") { >> struct iio_chan_spec *spec =3D &chan_specs[i]; >> + struct ads1262_channel *chan =3D &st->channels[i]; >> =20 >> ret =3D fwnode_property_read_u32(node, "reg", ®); >> if (ret) >> @@ -772,6 +986,8 @@ static int ads1262_parse_channels(struct iio_dev *in= dio_dev) >> if (reg >=3D ADS1262_MONITOR_ADDR_OFFSET) >> return dev_err_probe(dev, -EINVAL, "%pfwP: reg out of range\n", node= ); >> =20 >> + *chan =3D (struct ads1262_channel)ads1262_default_channel; > > This is giving me a build warning: > > drivers/iio/adc/ti-ads1262.c:1913:26: warning: cast to non-scalar > drivers/iio/adc/ti-ads1262.c:1913:26: warning: cast from non-scalar > > So we can't do this... Hmm. It's pretty simply. Use a define for > the default channel with the (struct adds1262_channel) > and that should work for us. Thanks, I'll go for that. ... --=20 Thanks, ~ Kurt