From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f44.google.com (mail-ot1-f44.google.com [209.85.210.44]) (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 8758B26A0EB for ; Wed, 2 Jul 2025 12:38:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751459883; cv=none; b=NMI4Dg6bi10/drTZ/osqvwrr1MmYS9D6sTQn8kgkspsddT+LT5OKi/woGaumX5EstZfMY578IfvfUb1hqrqEfhGVr6wMKhDYxHeQXMkTuaXRFq+4bd1E7VlFHrzwSfgqWrxFj7IP3C/Wv2NcopdHKQUTqP1pwlIxTYzY/37Ym5M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751459883; c=relaxed/simple; bh=uU+JKWZlOSfzPKyq1XI/5n7j3jfPISn/GknIBu+754Q=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=A/DM0Jlag1CI0sGLUwXEiZ9ch1sY4QkPCcS/xnpP+I11Yx1hnhKflnA2IKerQXqRq/XcbW1jf0Fce1DlnSMFncvuaH0t4riI8unn7/tAPcXKVdyD0wR2HiyxYldWWHZI+pIAbXVYJrOQkWKmT37sXrwMtDHEPpM3AGRIQa8Vl0w= 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=sORhhNBR; arc=none smtp.client-ip=209.85.210.44 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="sORhhNBR" Received: by mail-ot1-f44.google.com with SMTP id 46e09a7af769-73b1b1b3cb8so812040a34.2 for ; Wed, 02 Jul 2025 05:38:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1751459879; x=1752064679; 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=Ojsw+AkNzV7erO26jokTtJOtY0VwWEiinzqI7J34IJE=; b=sORhhNBRPV0VFe2D0Ezy7r9MXnXH/YUd1/ktE9rGR+A08PN4c5H8EY3vH4CSUEchUF vJ7C5zPswNnwJql4JNvDf7+IDB/2pZ4kc32caOrxtDGjJ+eeWIRKrwK+XOXgPNCuNLoL aKheB+2bNYVq44FJea5TN4gSKTUWGBALNhxL+CBOsTisftoqNAo7jiFLF1KjN22TpVzw dV+2cooMGiZxGK1xRjU4cWPMDwW/rnnfxvug0s5znwq4QaljkY2oLWeQbRQAu34/c7TQ LIpW0eCGxaNUdaPlRwMI7Ubr4pBug20rntMikarEuwAv2mASAKuMDs6DCMJiDNizWKVb r3+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751459879; x=1752064679; 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=Ojsw+AkNzV7erO26jokTtJOtY0VwWEiinzqI7J34IJE=; b=i/7lF5jnh9wqeXL30JH6MsT3r8IpD6mVSrIB7v7oa8KcmZpQEvBcFJPCYdTjFjlEVP j17yB9ZEhZOh135h6yQWwMZUaUYiTC0GYOSNb47LbvNHmhpak3CjpQ36yI4iBe8lnkuP QZmrKRX8SLL+03Ofro0M3UFbcb9RyCWHo9CzPjzvv0JSxPRjGbrv90/U+np6GV6y8OkM zNp2E2TPrsjAqb+6WgIIkUklOuInhJ0z+VY+JaYSgAjkZcHYjgDeWq5l8sqMviEvFXc+ /2oKwmPoTKAoo2ePylG8J5GgbUIFhijOSzgVFQ4YGTMXUJRyimfpPBtF4qxHQLhYxObu 1uLw== X-Gm-Message-State: AOJu0Yw7ZAs0ymoWSniVBeQMpbZmT5XyhRbacyXoAUB5KcS+LR4Sov1A 9QrlB/rvvIQM5pAuenmCYYYbSEuNQEoB0bFAJ26xTS7nf6/WEVi6dWttnQT8JDKHKSczPwk1KSZ 7Lf/DrBQ= X-Gm-Gg: ASbGnctI/4MK8FjuY7lugsPlhznpd5EX6EmrOmtwUfdkC3UVTSIWiL9JbtCggWAlHIm uZak6OprIWRZhWyraLSUi3Ab/uUI2m3bal5xIev3MhQIF0hFu7Qz3uxo6RAhiSqQEpwuXEqapKJ ULLmCqShU45ergXj6gkPALhlInKap5tDPP7U96ZO8g4j63tCcFkjxetruphdQaDQpg44h8Ed7pF 6Lk+LgsdPXSCcQP9XxUnitcMmOyXJcL1L5wlX3+Xad0DVUb5oDdxbytpZ8kEAMLKvFrBQDWSE78 W4M25wIy7g/pxFzKR8DWG892JYjdHKCaZ9f7Fj6HabW5R4wa0OKOBw87Yc9iot8TquTfXdG/Yln hewwxqeN/B7SlKTQXk4hTpPi7zzBNkrFI29udCAo= X-Google-Smtp-Source: AGHT+IEj5Dc1Netni0tqir/1xbnkQLxei6iTtHulTBtiYZFxi3bplPlig4+mWeKKumJKKpiHaH9WfQ== X-Received: by 2002:a05:6808:2383:b0:40b:3816:6637 with SMTP id 5614622812f47-40b887f2c1cmr2015465b6e.24.1751459879504; Wed, 02 Jul 2025 05:37:59 -0700 (PDT) Received: from ?IPV6:2600:8803:e7e4:1d00:70a7:ca49:a250:f1d5? ([2600:8803:e7e4:1d00:70a7:ca49:a250:f1d5]) by smtp.gmail.com with ESMTPSA id 5614622812f47-40b32420a76sm2554421b6e.38.2025.07.02.05.37.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 02 Jul 2025 05:37:58 -0700 (PDT) Message-ID: Date: Wed, 2 Jul 2025 07:37:56 -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 v3 10/12] spi: offload trigger: add ADI Util Sigma-Delta SPI driver To: =?UTF-8?Q?Nuno_S=C3=A1?= , Michael Hennerich , Jonathan Cameron , =?UTF-8?Q?Nuno_S=C3=A1?= , Andy Shevchenko , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Mark Brown Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-spi@vger.kernel.org References: <20250701-iio-adc-ad7173-add-spi-offload-support-v3-0-42abb83e3dac@baylibre.com> <20250701-iio-adc-ad7173-add-spi-offload-support-v3-10-42abb83e3dac@baylibre.com> <52c1fe276d16b68b955a00d0417b40902e2aa56e.camel@gmail.com> Content-Language: en-US From: David Lechner In-Reply-To: <52c1fe276d16b68b955a00d0417b40902e2aa56e.camel@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 7/2/25 2:57 AM, Nuno Sá wrote: > On Tue, 2025-07-01 at 16:37 -0500, David Lechner wrote: >> Add a new driver for the ADI Util Sigma-Delta SPI FPGA IP core. >> >> This is used to trigger a SPI offload based on a RDY signal from an ADC >> while masking out other signals on the same line. >> >> Signed-off-by: David Lechner >> --- --- >> +static int adi_util_sigma_delta_probe(struct platform_device *pdev) >> +{ >> + struct device *dev = &pdev->dev; >> + struct spi_offload_trigger_info info = { >> + .fwnode = dev_fwnode(dev), >> + .ops = &adi_util_sigma_delta_ops, >> + }; >> + struct clk *clk; >> + >> + clk = devm_clk_get_enabled(dev, NULL); >> + if (IS_ERR(clk)) >> + return dev_err_probe(dev, PTR_ERR(clk), "Failed to get >> clock\n"); >> + > > Small nit. Did you consider enabling/disabling the clock on the trigger > enable()/disable() callback? I guess the ref clk will be enabled anyways by > someone else but conceptually kind of makes sense to enable the resource only > when needed. > > Not a big deal (at least to me). > > - Nuno Sá > I thought about it, but we don't do that for any other FPGA IP cores so I didn't do it here either. Since they all use the same AXI clock anyway, disabling the clock here probably doesn't have any effect in practice since there will always be other users of the same clock keeping it enabled.