From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A884ACCF9F8 for ; Sun, 9 Nov 2025 19:06:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=1Y3W5ywvn7drSnOuge5r3/YGn5rwH+Mp+hb7awIgl3s=; b=PL4Tbaf7seOv8+2hWF0chUkS/B vkkndJSE6nJRNbC35/VUZEO1KtW8f0EiPZrVChg06UXry1s4dVOyFmjwJ4YBlSSMR7ijachWehCGi 8wft2cKIg46xVqg9+eq5XtpjRj5I1lNKL7hS5Z51gk/VQaS18D15dFr6dKKqV37QiPaN17pcup8Xg XTJ8fKIzXLmiEodstE8xJq9gnPGXp7HWn/WxJQ3Ky7dYdjeWHYB7nzg4/L0dcDZ+6FgJ6mQf7E+Yf JIMdBEZ3Fof+jV/B+ediXSKmiYquM8sze/3Ro1KMG5B1yRkovMklwLNPa6GePSO5AKPL+uwwZsjGP 86cBSaWw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIAk7-00000004JmE-28Mk; Sun, 09 Nov 2025 19:06:27 +0000 Received: from mail-wr1-x42b.google.com ([2a00:1450:4864:20::42b]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vIAk4-00000004JlT-43ZN for linux-arm-kernel@lists.infradead.org; Sun, 09 Nov 2025 19:06:26 +0000 Received: by mail-wr1-x42b.google.com with SMTP id ffacd0b85a97d-42b2cff817aso178737f8f.0 for ; Sun, 09 Nov 2025 11:06:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1762715182; x=1763319982; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:autocrypt:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=1Y3W5ywvn7drSnOuge5r3/YGn5rwH+Mp+hb7awIgl3s=; b=Ajq6kKRRMrAMbvHWZHokkucNWOIrbGtpzuS7A1s7ifBQwKNZp5GSXfgxQkFljBlfrN /WZB/S923KNm+PMHyLmoCbH8HNiYPvAOQ/1XaDf0w8f83fnFf2Sw5xm2npiWOC/hrnK9 DiECMn1BhQBNrY+a41qWl/pcSINuDv8nMK6OcEcKg7U/pBUlhljzzFAErb70zmni4TX7 mOksJG/enmFEThl6R8XFTS3JxDJWzYMp5QmYgf2mTLberEZT+c9QoXVCCIkNcofNVhVa zBvz1nPrZg3FO09RpCtyGtQJ3jJgi2mCZjNG/5WK4DLvZDC2Rr2fG2Jx0RoEwSe7IOmy i68Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762715182; x=1763319982; h=content-transfer-encoding:in-reply-to:autocrypt:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=1Y3W5ywvn7drSnOuge5r3/YGn5rwH+Mp+hb7awIgl3s=; b=AZT2c9EwMp3W52cH6YjJzsnoLrJxhg35fKT0bHrqN0BvKeG/WBUFclzzOHXIgZAvc+ +LvJTyOC0jmMNjvE5SCSDfG2JOAXJTPSUW3FUq7jdwoD9XnyqUk/13jS1RvoTmmax7A4 wxanTUBOM4S+/5HJgzpoeNUxOauMps+5ZCLsZkXebhPgGTeB8uP3KcX2ox5dUj7DrU9o 9pnWmJYcw/9OLn5VLJAXzeoUiBkqoFYfuXg7euZ40ZJ1LHVZLlMv6F+Gtma/gRbfhL+q r+38baU/fUb5zRnni5RyXl6kHkH5gtRmU6F0JaIk19VhyHAeNt0/qx9pqxl7g492mJTa NrfQ== X-Forwarded-Encrypted: i=1; AJvYcCVCaBQVFP2jPIf+NFvbjTHfOoejCLYFnG5f+/0SJo7p9b4tphBssna2VfIA/bbgmojuVc9njpmw1S5FH2AxbIks@lists.infradead.org X-Gm-Message-State: AOJu0YzO0+/GfWWiqZnYpzZKHDqrLMQQGtGFQsBUwgwm6+O+ZE4lrInS MzAfuveRtIzkycUISBJx171+DHo9rDEO96h2ctV/RYllLDZ+cm0AKtmQtzu5rx/w8uI= X-Gm-Gg: ASbGnctLKrl3P9xDJVaSyMgMXlaW5skhEHuMn2IjzEMbptX6QbHkMA4aRJUTSuhU+oR k4XFHPVU3OqXWAX9LLU24WX0MkXaiJmvbEcjOnzqvEGlHVk2olMh9pfD2jDzd3XLnKqSefgJhiY HBHZKNMZ8AaxjpFhpLVMWMcu6eV3MSSH5AHzYeTxYgBZRYG5h6JFi63t+k4nQ0xx4ymMYJzYGfD NG+AeMkur40ebIwaU11aTkBiVku5GfwjfMB7QE8xJLFpojErzLwCuHhwzmGD+JejnhqV//DufLT exyt+f1ixe9FKMxydDCPue/CSH4yln2uQ4YHXJVqo8BxiFKgJuG/ArM7vZq9SV+/vnRXORDOQyo kb406va2JzIih9dnzOUKnYUXLd9axFs6JJ3bL4o8ZL6hAcqS1lAkBy+TG9Hs3pxZEnpt8rd949P 5zRtlsyXEOVFYidhyk7ZhN8SjS5fR1Hx0= X-Google-Smtp-Source: AGHT+IE/lbhJ0vCe2mId2P7awStumT/sI1M3BLEL1gPp+PFhAxOunSK8kwDpAn8OgL+MDe5CdFcyfg== X-Received: by 2002:a05:6000:2002:b0:426:fe0a:1bb3 with SMTP id ffacd0b85a97d-42b2dcab74dmr2388569f8f.7.1762715182253; Sun, 09 Nov 2025 11:06:22 -0800 (PST) Received: from [192.168.1.29] ([178.197.219.123]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b380a3a30sm4153513f8f.4.2025.11.09.11.06.19 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 09 Nov 2025 11:06:21 -0800 (PST) Message-ID: <86139020-d2bf-4dc8-8b38-d7faba838e46@linaro.org> Date: Sun, 9 Nov 2025 20:06:18 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 11/13] soc: qcom: Simplify with of_machine_get_match_data() To: Dmitry Baryshkov , Krzysztof Kozlowski Cc: Rob Herring , Saravana Kannan , "Rafael J. Wysocki" , Viresh Kumar , Matthias Brugger , AngeloGioacchino Del Regno , Yangtao Li , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Lorenzo Pieralisi , Daniel Lezcano , Bjorn Andersson , Konrad Dybcio , Thomas Gleixner , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Maximilian Luz , Hans de Goede , =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , Daniel Lezcano , Thierry Reding , Jonathan Hunter , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-arm-msm@vger.kernel.org, platform-driver-x86@vger.kernel.org, linux-tegra@vger.kernel.org References: <20251106-b4-of-match-matchine-data-v1-0-d780ea1780c2@linaro.org> <20251106-b4-of-match-matchine-data-v1-11-d780ea1780c2@linaro.org> <8fc8c945-ae67-4c58-837d-40bdf4d60035@kernel.org> <3q5bpkktogs3pxjboihynjduabqrcuayyexjqdv3cgp5krjaxo@afnknyguuzxl> From: Krzysztof Kozlowski Content-Language: en-US Autocrypt: addr=krzysztof.kozlowski@linaro.org; keydata= xsFNBFVDQq4BEAC6KeLOfFsAvFMBsrCrJ2bCalhPv5+KQF2PS2+iwZI8BpRZoV+Bd5kWvN79 cFgcqTTuNHjAvxtUG8pQgGTHAObYs6xeYJtjUH0ZX6ndJ33FJYf5V3yXqqjcZ30FgHzJCFUu JMp7PSyMPzpUXfU12yfcRYVEMQrmplNZssmYhiTeVicuOOypWugZKVLGNm0IweVCaZ/DJDIH gNbpvVwjcKYrx85m9cBVEBUGaQP6AT7qlVCkrf50v8bofSIyVa2xmubbAwwFA1oxoOusjPIE J3iadrwpFvsZjF5uHAKS+7wHLoW9hVzOnLbX6ajk5Hf8Pb1m+VH/E8bPBNNYKkfTtypTDUCj NYcd27tjnXfG+SDs/EXNUAIRefCyvaRG7oRYF3Ec+2RgQDRnmmjCjoQNbFrJvJkFHlPeHaeS BosGY+XWKydnmsfY7SSnjAzLUGAFhLd/XDVpb1Een2XucPpKvt9ORF+48gy12FA5GduRLhQU vK4tU7ojoem/G23PcowM1CwPurC8sAVsQb9KmwTGh7rVz3ks3w/zfGBy3+WmLg++C2Wct6nM Pd8/6CBVjEWqD06/RjI2AnjIq5fSEH/BIfXXfC68nMp9BZoy3So4ZsbOlBmtAPvMYX6U8VwD TNeBxJu5Ex0Izf1NV9CzC3nNaFUYOY8KfN01X5SExAoVTr09ewARAQABzTRLcnp5c3p0b2Yg S296bG93c2tpIDxrcnp5c3p0b2Yua296bG93c2tpQGxpbmFyby5vcmc+wsGUBBMBCgA+AhsD BQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAFiEEm9B+DgxR+NWWd7dUG5NDfTtBYpsFAmgXUEoF CRaWdJoACgkQG5NDfTtBYpudig/+Inb3Kjx1B7w2IpPKmpCT20QQQstx14Wi+rh2FcnV6+/9 tyHtYwdirraBGGerrNY1c14MX0Tsmzqu9NyZ43heQB2uJuQb35rmI4dn1G+ZH0BD7cwR+M9m lSV9YlF7z3Ycz2zHjxL1QXBVvwJRyE0sCIoe+0O9AW9Xj8L/dmvmRfDdtRhYVGyU7fze+lsH 1pXaq9fdef8QsAETCg5q0zxD+VS+OoZFx4ZtFqvzmhCs0eFvM7gNqiyczeVGUciVlO3+1ZUn eqQnxTXnqfJHptZTtK05uXGBwxjTHJrlSKnDslhZNkzv4JfTQhmERyx8BPHDkzpuPjfZ5Jp3 INcYsxgttyeDS4prv+XWlT7DUjIzcKih0tFDoW5/k6OZeFPba5PATHO78rcWFcduN8xB23B4 WFQAt5jpsP7/ngKQR9drMXfQGcEmqBq+aoVHobwOfEJTErdku05zjFmm1VnD55CzFJvG7Ll9 OsRfZD/1MKbl0k39NiRuf8IYFOxVCKrMSgnqED1eacLgj3AWnmfPlyB3Xka0FimVu5Q7r1H/ 9CCfHiOjjPsTAjE+Woh+/8Q0IyHzr+2sCe4g9w2tlsMQJhixykXC1KvzqMdUYKuE00CT+wdK nXj0hlNnThRfcA9VPYzKlx3W6GLlyB6umd6WBGGKyiOmOcPqUK3GIvnLzfTXR5DOwU0EVUNc NAEQAM2StBhJERQvgPcbCzjokShn0cRA4q2SvCOvOXD+0KapXMRFE+/PZeDyfv4dEKuCqeh0 hihSHlaxTzg3TcqUu54w2xYskG8Fq5tg3gm4kh1Gvh1LijIXX99ABA8eHxOGmLPRIBkXHqJY oHtCvPc6sYKNM9xbp6I4yF56xVLmHGJ61KaWKf5KKWYgA9kfHufbja7qR0c6H79LIsiYqf92 H1HNq1WlQpu/fh4/XAAaV1axHFt/dY/2kU05tLMj8GjeQDz1fHas7augL4argt4e+jum3Nwt yupodQBxncKAUbzwKcDrPqUFmfRbJ7ARw8491xQHZDsP82JRj4cOJX32sBg8nO2N5OsFJOcd 5IE9v6qfllkZDAh1Rb1h6DFYq9dcdPAHl4zOj9EHq99/CpyccOh7SrtWDNFFknCmLpowhct9 5ZnlavBrDbOV0W47gO33WkXMFI4il4y1+Bv89979rVYn8aBohEgET41SpyQz7fMkcaZU+ok/ +HYjC/qfDxT7tjKXqBQEscVODaFicsUkjheOD4BfWEcVUqa+XdUEciwG/SgNyxBZepj41oVq FPSVE+Ni2tNrW/e16b8mgXNngHSnbsr6pAIXZH3qFW+4TKPMGZ2rZ6zITrMip+12jgw4mGjy 5y06JZvA02rZT2k9aa7i9dUUFggaanI09jNGbRA/ABEBAAHCwXwEGAEKACYCGwwWIQSb0H4O DFH41ZZ3t1Qbk0N9O0FimwUCaBdQXwUJFpZbKgAKCRAbk0N9O0Fim07TD/92Vcmzn/jaEBcq yT48ODfDIQVvg2nIDW+qbHtJ8DOT0d/qVbBTU7oBuo0xuHo+MTBp0pSTWbThLsSN1AuyP8wF KChC0JPcwOZZRS0dl3lFgg+c+rdZUHjsa247r+7fvm2zGG1/u+33lBJgnAIH5lSCjhP4VXiG q5ngCxGRuBq+0jNCKyAOC/vq2cS/dgdXwmf2aL8G7QVREX7mSl0x+CjWyrpFc1D/9NV/zIWB G1NR1fFb+oeOVhRGubYfiS62htUQjGLK7qbTmrd715kH9Noww1U5HH7WQzePt/SvC0RhQXNj XKBB+lwwM+XulFigmMF1KybRm7MNoLBrGDa3yGpAkHMkJ7NM4iSMdSxYAr60RtThnhKc2kLI zd8GqyBh0nGPIL+1ZVMBDXw1Eu0/Du0rWt1zAKXQYVAfBLCTmkOnPU0fjR7qVT41xdJ6KqQM NGQeV+0o9X91X6VBeK6Na3zt5y4eWkve65DRlk1aoeBmhAteioLZlXkqu0pZv+PKIVf+zFKu h0At/TN/618e/QVlZPbMeNSp3S3ieMP9Q6y4gw5CfgiDRJ2K9g99m6Rvlx1qwom6QbU06ltb vJE2K9oKd9nPp1NrBfBdEhX8oOwdCLJXEq83vdtOEqE42RxfYta4P3by0BHpcwzYbmi/Et7T 2+47PN9NZAOyb771QoVr8A== In-Reply-To: <3q5bpkktogs3pxjboihynjduabqrcuayyexjqdv3cgp5krjaxo@afnknyguuzxl> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251109_110625_106685_B866626D X-CRM114-Status: GOOD ( 26.86 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 08/11/2025 17:31, Dmitry Baryshkov wrote: > On Fri, Nov 07, 2025 at 03:58:26PM +0100, Krzysztof Kozlowski wrote: >> On 07/11/2025 15:23, Dmitry Baryshkov wrote: >>> On Fri, Nov 07, 2025 at 08:08:28AM +0100, Krzysztof Kozlowski wrote: >>>> On 07/11/2025 08:02, Krzysztof Kozlowski wrote: >>>>> On 07/11/2025 04:19, Dmitry Baryshkov wrote: >>>>>> On Thu, Nov 06, 2025 at 08:07:18PM +0100, Krzysztof Kozlowski wrote: >>>>>>> Replace open-coded getting root OF node, matching against it and getting >>>>>>> the match data with new of_machine_get_match_data() helper. >>>>>>> >>>>>>> Signed-off-by: Krzysztof Kozlowski >>>>>>> >>>>>>> --- >>>>>>> >>>>>>> Depends on the first OF patch. >>>>>>> --- >>>>>>> drivers/soc/qcom/qcom_pd_mapper.c | 17 ++--------------- >>>>>>> 1 file changed, 2 insertions(+), 15 deletions(-) >>>>>>> >>>>>>> diff --git a/drivers/soc/qcom/qcom_pd_mapper.c b/drivers/soc/qcom/qcom_pd_mapper.c >>>>>>> index 1bcbe69688d2..07198d44b559 100644 >>>>>>> --- a/drivers/soc/qcom/qcom_pd_mapper.c >>>>>>> +++ b/drivers/soc/qcom/qcom_pd_mapper.c >>>>>>> @@ -613,25 +613,12 @@ static void qcom_pdm_stop(struct qcom_pdm_data *data) >>>>>>> static struct qcom_pdm_data *qcom_pdm_start(void) >>>>>>> { >>>>>>> const struct qcom_pdm_domain_data * const *domains; >>>>>>> - const struct of_device_id *match; >>>>>>> struct qcom_pdm_data *data; >>>>>>> - struct device_node *root; >>>>>>> int ret, i; >>>>>>> >>>>>>> - root = of_find_node_by_path("/"); >>>>>>> - if (!root) >>>>>>> - return ERR_PTR(-ENODEV); >>>>>>> - >>>>>>> - match = of_match_node(qcom_pdm_domains, root); >>>>>>> - of_node_put(root); >>>>>>> - if (!match) { >>>>>>> - pr_notice("PDM: no support for the platform, userspace daemon might be required.\n"); >>>>>>> - return ERR_PTR(-ENODEV); >>>>>>> - } >>>>>>> - >>>>>>> - domains = match->data; >>>>>>> + domains = of_machine_get_match_data(qcom_pdm_domains); >>>>>>> if (!domains) { >>>>>>> - pr_debug("PDM: no domains\n"); >>>>>>> + pr_notice("PDM: no support for the platform or no domains, userspace daemon might be required.\n"); >>>>>>> return ERR_PTR(-ENODEV); >>>>>>> } >>>>>> >>>>>> Here you are mixing two cases: >>>>>> - There is not match in the table (in which case the driver should print >>>>>> a notice) >>>>>> >>>>>> - There is a match in the table, but the data is NULL (the platform >>>>>> doesn't have PDM domains). In this case there should be no notice. >>>>> >>>>> >>>>> Why? Existing code printed notice in both cases. Why refactoring which >>>>> tries to keep code functionally equivalent should change it? >>>> >>>> Ah, you mean there was a debug before. Well, then I am a bit confused >>>> because table has entries without data (so expected condition) but old >>>> code returned ERRNO in such case - so unexpected condition. >>>> >>>> Wail failing the probe on expected condition? >>>> >>>> Unless it is not really expected and notice in second case is valid as well. >>> >>> If we know that there are no domains on the platform, then the notice >>> definitely doesn't apply. Failing the probe is a separate topic. The >>> rest of the code expects that _qcom_pdm_data is not NULL. >> >> I hoped that separate topic would be the reason, after commit msg >> adjustments, to keep this change, but if you insist that this must stay >> debug, then this patch should be just dropped because it is impossible >> to achieve with current helpers. > > Having the same pr_notice would be misleading: we point users to running > userspace daemon, while we _know_ that the daemon is useless because > there are no PDs. One of the ways to solve it would be to add extra > wrapping, so that the data in the match table is never NULL. The message does not matter. It still returns ENODEV which is failing the probe. This leaves error messages in the dmesg already, if pdm is ever probed for these devices, so my change of debug->notice here really, really does not matter. Failing of probe is always a failure of system, so it is not an expected behavior. Anyway, I will just drop this change as I said. Best regards, Krzysztof