From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-f49.google.com (mail-oa1-f49.google.com [209.85.160.49]) (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 75B0117CA17 for ; Thu, 20 Feb 2025 18:27:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740076059; cv=none; b=YJNPvlupwK2xazWjzn7ivhHW2I+D2bhw+0DmDZjpj7g2Ffrh8QK1jvqJd4WxW4krDG+PtPrrQKtX2LSvi97insxHC+ofp4dah8NRjL3jGOSUVH0KuVucHI0EOvCjxuBKbpXLSbE7Zu1tXSA8ZKB46ufV7Y6nPrwLJajmgqFGdms= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740076059; c=relaxed/simple; bh=VHgizFaYz1ce+SVHS71EReLg+1ZGhqjTgNtrNddwkIc=; h=Message-ID:Date:MIME-Version:Subject:To:References:From: In-Reply-To:Content-Type; b=MyIqlqSrlX7/NCcKZkTg8/r70pqilR7MG6z3dGIlDPrKFY4ZsvV1qzTh4oGnbZ2xiud3CHjpVfMrMeITdcFcSirNELSWqgcfnluDjGDRkurWlVvbQLfA7IxiYJYK0apTFxC3B2z9p+xR7Fg+dEmElobnBhZzBIivFvSR56gGm58= 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=1SSIrEjH; arc=none smtp.client-ip=209.85.160.49 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="1SSIrEjH" Received: by mail-oa1-f49.google.com with SMTP id 586e51a60fabf-2bd2218ba4fso296667fac.1 for ; Thu, 20 Feb 2025 10:27:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1740076055; x=1740680855; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=W7qk0FcOBmefqQ8ES313TjuenUiSsXs8dKbnSBmPA7Y=; b=1SSIrEjH90mSrpXDTaWr1//pNXAGMg37AmfHRR3j0KpGQehZOSENaI1jI0C9oZWCJd w68ay8tCsRevFlbC4QYrloGmYGh14zD7xu3uG63lBLDeXGYcJ3E/Hy7wnSVhvWRMzXFt dIWiixbtiyiFc8nz0ZAmfx2aVDXzuHbRYAqM+aUAHpH9RBOpJvi2RqZ+I5CHEP6b8w68 Kea+ce4o841ANlA/gvEKM/s6anhc7r3v7zTWRb8HXioFhzfuZNuSC7ASItHj5y4As+hG L0zmo0sZWsW2w1UpVY1fPVp3odAYHD7ESPQ8pamkUA3+Rtkd54Tm0/WBuXO7KZ79CcFh 0qng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740076055; x=1740680855; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=W7qk0FcOBmefqQ8ES313TjuenUiSsXs8dKbnSBmPA7Y=; b=vaFSYfgAp7rK9zi/3tV4WmkLHjxZYQUTv4wzzk+k0UqUdW+qiY1uFqdm0ZgY18TJGY p6pI6Hfxqt4PNMAhQKC6Pc6W+xLzlgZ6p0kKxUqEQ1Cm5U7cHcu9AZ+TXPq5iY0ZM/E7 CcA/2guS+iXF7IGwMEQ7BTiEpu3UG36GmgK36iWRQ2Ohq3RXhfVzzuuhGzx6bvEEHvRE LJ+O0MDWgQwFPobqaHN8dmpw7KS1MJ/ffh17INE5eYNVXwDSXmpv3Brl9wEESVDOxsC/ uWTMiUlpCPPeVQvnqrwzALsqbSsxO8OfE6krJoT/UrwG0/fniNuiJ9pY2QeMI9aP8bR7 MMVg== X-Forwarded-Encrypted: i=1; AJvYcCVbUT8pH8KV8spSJJWFOcT+G+9kVBCjxiUk+Cu/8a/gZox2X1kmOfQozhZbAJwNZ0GWzxBPlHhq5mU=@vger.kernel.org X-Gm-Message-State: AOJu0Yw/tmzB3+Cf5R6s4z9E9FDzm1HRTpqnkJ1wZTxlZuh3xFHNB6i9 KZ8uJxvAp4L3V5bMxQQZcnWWIc2Q7TRkSNU79NEKjgjsDIqBsLRXgHuwPTC1Wqg= X-Gm-Gg: ASbGncsybC0cdkSeaMgFbSNU9uMK+rSSFhAQRbBPAHQwnF+ZhRbnYMZnqsj14RxyY+o 1iPPIj8VuoygWx5hkLzpLS3AZbw36NBIC7i7G4muYt2z8lIIxLfvFLGwm8DdCqtWF/EX27EGlN2 JTEH3fMDOBSfueaFXesiDxQWmCOvXW8CDR6Sh7WOwYqFWzShLKZSP3Jm3t8cUYQ9vehFwIm9OSt l7pGYeUoinayfQxHUnDMju1MyOw4H9j7TjhM53CFesYUQ76Igx9MU6s7bsqKe1MAsZEjVX3CZxj msUoWhJ4Nm3aVtfLRD/O5H3xNDoxxQmloMUk+1p0Ya8vR/TcjmMB X-Google-Smtp-Source: AGHT+IGsm/MqvD+9VZNwdbbtbnUbjtDWYbJQ7F2+IksnXJpGnAqy8TY/Rb9MOR38Jj96OQB0zkHZQg== X-Received: by 2002:a05:6870:1705:b0:297:2763:18d4 with SMTP id 586e51a60fabf-2bd50cef076mr52222fac.15.1740076055473; Thu, 20 Feb 2025 10:27:35 -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 586e51a60fabf-2b963a16847sm6402045fac.49.2025.02.20.10.27.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 20 Feb 2025 10:27:34 -0800 (PST) Message-ID: Date: Thu, 20 Feb 2025 12:27:32 -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 14/14] Documentation: ABI: testing: ad4080 docs To: =?UTF-8?Q?Nuno_S=C3=A1?= , 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 References: <20250220135429.8615-1-antoniu.miclaus@analog.com> <20250220135429.8615-15-antoniu.miclaus@analog.com> <8f588f4b88d122815df694660d19672e8ccd3d70.camel@gmail.com> Content-Language: en-US From: David Lechner In-Reply-To: <8f588f4b88d122815df694660d19672e8ccd3d70.camel@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 2/20/25 8:53 AM, Nuno Sá wrote: > On Thu, 2025-02-20 at 15:54 +0200, Antoniu Miclaus wrote: >> Add documentation for the ad4080 attributes. >> >> Signed-off-by: Antoniu Miclaus >> --- >>  .../ABI/testing/sysfs-bus-iio-adc-ad4080      | 55 +++++++++++++++++++ >>  1 file changed, 55 insertions(+) >>  create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-adc-ad4080 >> >> diff --git a/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4080 >> b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4080 >> new file mode 100644 >> index 000000000000..e37bfba0e989 >> --- /dev/null >> +++ b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4080 >> @@ -0,0 +1,55 @@ >> +What: /sys/bus/iio/devices/iio:deviceX/lvds_sync >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + This attribute handles the data synchronization >> process.Because the CNV >> + signal is not taken into account by the FPGA when capturing >> the data, we >> + need a process that configures the ADC to output pattern >> data, writes the >> + SYNC bit in the axi_adc register map, waits until the custom >> HDL syncs the >> + data correctly, and then changes the output mode to analog >> data instead of >> + the fixed pattern. > > I'll comment this one in the driver. I have some questions on how this works... > >> + >> +What: /sys/bus/iio/devices/iio:deviceX/lvds_lvds >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + Configures the signal type of the CNV signal. The value can >> be either CMOS >> + (lvds_cnv=0) or LVDS (lvds_cnv=1). > > The name seems to be wrong with what you have implemented. From this > description, I would think of this as a DT property? Can the signal type really > change at runtime? > >> + >> +What: /sys/bus/iio/devices/iio:deviceX/filter_sel >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + This attribute enables the digital filter functionality of >> the AD4080.In >> + order to capture data correctly, the function must configure >> the ADC >> + through SPI to select the filter type and enable data capture >> in filter >> + mode through axi_adc(In this mode, data is gated by a signal >> generated by >> + the AD4080 (GPIO1 and is not continuous as it is when the >> filter is >> + disabled). >> + >> +What: /sys/bus/iio/devices/iio:deviceX/filter_sel_available >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + Return the available filter modes that can be set. > > There's a standard attr for this. I think we settled Yup. filter_type and filter_type_available. >> + >> +What: /sys/bus/iio/devices/iio:deviceX/sinc_dec_rate >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + Set the filter’s decimation rate. >> + >> +What: /sys/bus/iio/devices/iio:deviceX/sinc_dec_rate_available >> +Date: February 2025 >> +KernelVersion: >> +Contact: linux-iio@vger.kernel.org >> +Description: >> + Return the available filter's decimation rates. >> + >> + > > I'm not yet convinced we need the dec_rate custom attr. I'll add more comments > in the driver. If we do need it, in another driver recently we concluded that decimation rate is the same as oversampling ratio and there is already a standard attribute for oversampling ratio, so we used that. > > - Nuno Sá > >