From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oi1-f179.google.com (mail-oi1-f179.google.com [209.85.167.179]) (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 D26D125EFA6 for ; Mon, 24 Mar 2025 18:54:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742842469; cv=none; b=TFwdVNM8P+7L0/IjUg1k9ueDZsTrNpyd0L0E0kzXDFe8SouEfHMgMJwIb9npFESlk28Hh4CJx8AyM3jwuyb5EvyCnEhBxqcg12hhvDIiAs7yIh4oIe3oOJIhiQ9NJUi9MpAy3NGgcSTYZdBL405f2/SvrRjjhwFJ2bKqbfpv/fI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742842469; c=relaxed/simple; bh=bYepHZHGKJdzRa8ajrcVmkzK+TWNwXQ/53joqZg6Ssc=; h=Message-ID:Date:MIME-Version:Subject:To:References:From: In-Reply-To:Content-Type; b=aYPXA5x/ydT2oPXekLYTdf6k/Yn1NciW437zsPEYdRR7GDTk2RTosbScWbG/cvH0S/odkey5QDz6/jmsy3e4tGcETc/0W6VKF1U9e5BBgc/CY0912nAoGLEC8kUmuP7HH2hnAN+WDMDlLpUrf4uKBUNCQQS8PIQUZWXVEQS2W6c= 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=YzzueryM; arc=none smtp.client-ip=209.85.167.179 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="YzzueryM" Received: by mail-oi1-f179.google.com with SMTP id 5614622812f47-3f6a92f2328so2636587b6e.0 for ; Mon, 24 Mar 2025 11:54:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1742842467; x=1743447267; 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=06LT1E1RRNMfucwpIVrPW/y7BeeDpu7TG33qT6wY/6A=; b=YzzueryMFRgtaTCjzu4AQRYJG62LvLZm+8Z2RajfkODX3Z28nMmonjtXlDnz5ihGT1 qCK7StYu27iP5Lrvzvq1CzMfkAKyjYYO44cF/at0imjOdHZ7nUoPPlTWqxuh6aa2QUeX W6Cw+/IdbjmZQPw0Jc9FzxLEk8btrnApGtrRkdkAAQ5XQGayaL3siXUE7IjTaHWFsnLk XbXscwdJ8QsTnmotjX/J+tGUxNJlxvfZoAXr23fLsucnI9GP6xPMD5bgzB566uT2sfa0 HKs1mPLUGNXGiaFoyDQaWtyGnIY8QzjiLaV1uUjqzWiKrEUGBAch4hZjBnz+3a8vCeyY CNig== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742842467; x=1743447267; 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=06LT1E1RRNMfucwpIVrPW/y7BeeDpu7TG33qT6wY/6A=; b=Idhj9fzUPraKyhQfCcI7mjVEymh6Uh7JZVO5DWfBTga1uqyUs+HnTOTTha1BO6vJWD Eb4up4AD7Fk8Zav12LFEmXYOxQ00aiPNgW4ygcSJSRb2ivnwPFRB4cJm9O2QXEWxwTCA 7ZQKOZw3iy4VaO/zR3I646nDtaD6WRM2Mrw9jzhghjxB7OiMDXjTdwRuTHnqouL2A9oD CTkpItECgfzOqU63uuP/7DCgyFUNKDQXhFlBrSRkV/Y6IDNdH1FJWvRO7l/4LytB5vnZ yyojCg9TPke+gKZG8Wcwr8UZ2ngBIFTrTrpn13kvkjFW0nPvn7QCoRM23JpsLKr6hl43 4pPw== X-Forwarded-Encrypted: i=1; AJvYcCXDO8jIMp/t3DwBBPPLAdH/ddyRv8lNLcAlPM4uzU78L7heG4Mj51HiO0v4LFfnj7RpW2oZpi374Og=@vger.kernel.org X-Gm-Message-State: AOJu0YwB07JS57gf+QtX/S0T+NbbsCFjOXtukg0gAsDm6AzFDZcVAnIf z6zrQFq8uPxg0STtZramGY9ccZrO7Ty/nsx6VyV5lVUX7Dlvby7Sx+sd6IA60NA= X-Gm-Gg: ASbGnctb5NBmGZ/tEdK8eVc8ItwqXhZGQLhOZBA+jbqKaIbFviV2hNVg4mRkBEqOoVn IZOxUmFCRYfU2cAOQ9l6CkGzXv02WjT6n5W5VYH5ChnJqNuLLP9nugFHZz7DZFvzvdMbksJILVV DLN1RuRbBIm6jQ0iid/5QvIjToLfzBi+fht3GMOvC7FIwF6fT6UUiqmV4CB9QKivoXTyAw6pxR/ YTMHotraT60K0TMQv6WekHN/VR/l0xhXDBKe5vh7asEuT+ol4hDhHOHACZfYBMm+3IeS1Vtmnzx Ja4ZAkMQF4SPAh4wU8RgMUqqHtiL2MnfOPMPJOJlyQtMeD/hwkRK4WZnStEbh1H/KUJiNGOFnH+ kjht6pQ== X-Google-Smtp-Source: AGHT+IFyi/1W4BZWJx8USQ1sFUEuL/rB0lIMaIVBqUowj6ohQAXIoie0doJJgSvfsuYqIWjgfs5AJQ== X-Received: by 2002:a05:6808:9ab:b0:3fa:1d22:6d28 with SMTP id 5614622812f47-3febeeaa67fmr6698335b6e.18.1742842466714; Mon, 24 Mar 2025 11:54:26 -0700 (PDT) Received: from [192.168.0.113] (ip98-183-112-25.ok.ok.cox.net. [98.183.112.25]) by smtp.gmail.com with ESMTPSA id 5614622812f47-3febf6bd0a7sm1682759b6e.2.2025.03.24.11.54.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 24 Mar 2025 11:54:26 -0700 (PDT) Message-ID: <705a307d-32dc-4edc-bb71-0e201859264f@baylibre.com> Date: Mon, 24 Mar 2025 13:54:25 -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 2/5] iio: adc: adi-axi-adc: add set decimation rate To: Pop Ioan Daniel , Lars-Peter Clausen , Michael Hennerich , Jonathan Cameron , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nuno Sa , Olivier Moysan , Javier Carrasco , Andy Shevchenko , Guillaume Stols , Trevor Gamblin , Dumitru Ceclan , Matteo Martelli , =?UTF-8?Q?Jo=C3=A3o_Paulo_Gon=C3=A7alves?= , Alisa-Dariana Roman , Marcelo Schmitt , Herve Codina , Ramona Alexandra Nechita , Dragos Bogdan , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20250324090813.2775011-1-pop.ioan-daniel@analog.com> <20250324090813.2775011-3-pop.ioan-daniel@analog.com> From: David Lechner Content-Language: en-US In-Reply-To: <20250324090813.2775011-3-pop.ioan-daniel@analog.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 3/24/25 4:07 AM, Pop Ioan Daniel wrote: > Add support for setting decimation rate. > > Add separate compatible string for the custom AD7405 IP and implement > the necessary changes. > > Signed-off-by: Pop Ioan Daniel > --- > drivers/iio/adc/adi-axi-adc.c | 43 ++++++++++++++++++++++++++++++++++- > 1 file changed, 42 insertions(+), 1 deletion(-) > > diff --git a/drivers/iio/adc/adi-axi-adc.c b/drivers/iio/adc/adi-axi-adc.c > index 766406f45396..0ed609e294ba 100644 > --- a/drivers/iio/adc/adi-axi-adc.c > +++ b/drivers/iio/adc/adi-axi-adc.c > @@ -70,6 +70,9 @@ > #define ADI_AXI_ADC_REG_CHAN_CTRL_3(c) (0x0418 + (c) * 0x40) > #define ADI_AXI_ADC_CHAN_PN_SEL_MASK GENMASK(19, 16) > > +#define ADI_AXI_ADC_REG_CHAN_USR_CTRL_2 0x0424 This is a per-channel register, so expect to have a c parameter times offset like the other channel registers. > +#define ADI_AXI_ADC_DEC_RATE_MASK GENMASK(15, 0) Usually, we try to include the register name in these as well even if the name does get a bit long. ADI_AXI_ADC_CHAN_USR_CTRL_2_DEC_RATE_N_MASK > + > /* IO Delays */ > #define ADI_AXI_ADC_REG_DELAY(l) (0x0800 + (l) * 0x4) > #define AXI_ADC_DELAY_CTRL_MASK GENMASK(4, 0) > @@ -232,6 +235,16 @@ static int axi_adc_test_pattern_set(struct iio_backend *back, > } > } > > +static int axi_adc_set_dec_rate(struct iio_backend *back, > + unsigned int rate) This needs to take a channel index parameter as well to keep it generic. > +{ > + struct adi_axi_adc_state *st = iio_backend_get_priv(back); > + Might be helpful for future users if we add a comment explaining why we are setting the denominator but not the numerator. > + return regmap_update_bits(st->regmap, (ADI_AXI_ADC_REG_CHAN_USR_CTRL_2), > + ADI_AXI_ADC_DEC_RATE_MASK, > + FIELD_PREP(ADI_AXI_ADC_DEC_RATE_MASK, rate)); > +} > + > static int axi_adc_read_chan_status(struct adi_axi_adc_state *st, unsigned int chan, > unsigned int *status) > { > @@ -465,6 +478,28 @@ static const struct iio_backend_info adi_axi_adc_generic = { > .ops = &adi_axi_adc_ops, > }; > > +static const struct iio_backend_ops adi_ad7405_ops = { > + .enable = axi_adc_enable, > + .disable = axi_adc_disable, > + .data_format_set = axi_adc_data_format_set, > + .chan_enable = axi_adc_chan_enable, > + .chan_disable = axi_adc_chan_disable, > + .request_buffer = axi_adc_request_buffer, > + .free_buffer = axi_adc_free_buffer, > + .data_sample_trigger = axi_adc_data_sample_trigger, > + .iodelay_set = axi_adc_iodelays_set, > + .test_pattern_set = axi_adc_test_pattern_set, > + .set_dec_rate = axi_adc_set_dec_rate, > + .chan_status = axi_adc_chan_status, > + .debugfs_reg_access = iio_backend_debugfs_ptr(axi_adc_reg_access), > + .debugfs_print_chan_status = iio_backend_debugfs_ptr(axi_adc_debugfs_print_chan_status), > +}; As mentioned in my DT bindings patch reply, it looks like this register is part of the generic AXI ADC IP block, so we shouldn't need to add a chip-specific struct in this case. > + > +static const struct iio_backend_info axi_ad7405 = { > + .name = "axi-ad7405", > + .ops = &adi_ad7405_ops, > +}; > + > static int adi_axi_adc_probe(struct platform_device *pdev) > { > struct adi_axi_adc_state *st; > @@ -522,7 +557,7 @@ static int adi_axi_adc_probe(struct platform_device *pdev) > return -ENODEV; > } > > - ret = devm_iio_backend_register(&pdev->dev, &adi_axi_adc_generic, st); > + ret = devm_iio_backend_register(&pdev->dev, st->info->backend_info, st); I'm guessing you haven't rebased on the iio tree recently. This line was already changed in [1]. [1]: https://web.git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git/commit/drivers/iio/adc/adi-axi-adc.c?h=testing&id=7a794e3a0dc5b1df525bf72260b641d70e337784 > if (ret) > return dev_err_probe(&pdev->dev, ret, > "failed to register iio backend\n"); > @@ -575,10 +610,16 @@ static const struct axi_adc_info adc_ad7606 = { > .has_child_nodes = true, > }; > > +static const struct axi_adc_info adi_axi_ad7405 = { > + .version = ADI_AXI_PCORE_VER(10, 0, 'a'), > + .backend_info = &axi_ad7405, > +}; > + > /* Match table for of_platform binding */ > static const struct of_device_id adi_axi_adc_of_match[] = { > { .compatible = "adi,axi-adc-10.0.a", .data = &adc_generic }, > { .compatible = "adi,axi-ad7606x", .data = &adc_ad7606 }, > + { .compatible = "adi,axi-ad7405", .data = &adi_axi_ad7405}, > { /* end of list */ } > }; > MODULE_DEVICE_TABLE(of, adi_axi_adc_of_match);