From nobody Sun Feb 8 23:41:13 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 74ABCC001DE for ; Fri, 23 Jun 2023 21:51:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232896AbjFWVvA (ORCPT ); Fri, 23 Jun 2023 17:51:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40040 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232792AbjFWVtr (ORCPT ); Fri, 23 Jun 2023 17:49:47 -0400 Received: from mail-lj1-x232.google.com (mail-lj1-x232.google.com [IPv6:2a00:1450:4864:20::232]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8CE342701 for ; Fri, 23 Jun 2023 14:49:45 -0700 (PDT) Received: by mail-lj1-x232.google.com with SMTP id 38308e7fff4ca-2b46f1256bbso27374691fa.0 for ; Fri, 23 Jun 2023 14:49:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687556984; x=1690148984; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=RT+DMw9+JmEB+4blOGBEOQCCmDaLSKVa4tRJvUDZHYw=; b=lecwxWvFh2wABTwinbKbk2vXyK+vMqENR4o606JLOknjCo4R7wcHxvGUjmLWqT/pNp 6aZ/g+tzl+sRjGqw8AJ/A+09KAlA5CnoqeCh6VSOLFloP8o+/miSQhzVG8mrbivrSyA8 v0co5soOG49maSVuTMUkmwygSjbEDxl5Pe6JOQ5rKCiuNbboHHr4WiosZR73j+mXerQ4 D2BpLV1F6VNgc9eFxX4qs3UNBNPHaeOikR2B6jinv5Gwd0qurQJxmR+8vZEb+ebWM3K1 Zfw0LWBTNz5/ftSOHjrsWj8IzgdaiP6tqig9fgTtYqd2IoytDc2lNQGGfiVH+sKcU+oj UNwQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687556984; x=1690148984; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=RT+DMw9+JmEB+4blOGBEOQCCmDaLSKVa4tRJvUDZHYw=; b=EKG9Wzg4IomayfB8OBNNkFmgcDm2ZcpkCYqh1qydo9QY9Uw7bYExpif1UK641DN0wZ W318IfSXsIlQ9cvEiEgR5C7tHAnBF20olFppOvAGlRDPJDXhmGyETUUxS1l41snas1BV +yPDl4NaUAmwiz2LfPraCzivMUzwVAlgkClAwMhGdeULXHDoYCx1ijpbDtYHbQSGqXnp 1l7p2a4Jt3JNh3mMGo8OR0asUArgGG4XtGcBO+ydJA13uIm4gwSS+gf73zYHHm0xT5Jw YS5X3BsgvkeaJHBUH23mC4idCkZzuY84Kl5V8G0Q21FNiKoAQoZ5tbXWRcPzCRRzi2PY MxIg== X-Gm-Message-State: AC+VfDzdDYXMR2D/P8jEILSsEZ2NUeBeSVGh4HTOnaukI3JogJLN09Du lLfH2Uc+PArI3RN0sUKdcQy1ww== X-Google-Smtp-Source: ACHHUZ4Qsgh5BpEcXr0ex6p/I1xXp8oHnlGAZZ0gCcG1oLYqyghfqdPIB9b1i6aAsUWJGzKfw1KOiA== X-Received: by 2002:a2e:9d48:0:b0:2b4:50b8:5d3b with SMTP id y8-20020a2e9d48000000b002b450b85d3bmr7576901ljj.0.1687556983889; Fri, 23 Jun 2023 14:49:43 -0700 (PDT) Received: from [192.168.1.101] (abyk30.neoplus.adsl.tpnet.pl. [83.9.30.30]) by smtp.gmail.com with ESMTPSA id x20-20020a2e9c94000000b002b48437dcaasm1840919lji.103.2023.06.23.14.49.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Jun 2023 14:49:43 -0700 (PDT) From: Konrad Dybcio Date: Fri, 23 Jun 2023 23:49:42 +0200 Subject: [PATCH 1/3] firmware: qcom_scm: Always try to consume all three clocks MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20230623-topic-scm_cleanup-v1-1-383089eae98d@linaro.org> References: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> To: Andy Gross , Bjorn Andersson Cc: Marijn Suijten , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1687556981; l=4598; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=20MGaxR8SA/gitewjqJ05iW6t2qyGVNzk/M4vihgecc=; b=U+OJNRvlUVPfRdWvMndA3yI0+ew1J+eFh+Hvmxc5NRWEkkc7Hsdgqx5xIwmZmGGqlQmYtXvJ7 vCWfFMjPASBBOXevpw72vOUYRfsus9HEu+cBPvcoYYJFozf8XrBn6Ry X-Developer-Key: i=konrad.dybcio@linaro.org; a=ed25519; pk=iclgkYvtl2w05SSXO5EjjSYlhFKsJ+5OSZBjOkQuEms= Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The code for handling more than 1 clock is a bit messy and requires one to add new, SoC-specific compatibles if one wants to attach a clock. Switch devm_clk_get to devm_clk_get_optional to prevent throwing it from throwing errors when the clock is absent and defer checking the clock requirements to dt schema. This lets us get rid of compatibles that aren't necessary for backwards compatibility *and* will hopefully prevent the addition of meaningless new compatibles. Signed-off-by: Konrad Dybcio --- drivers/firmware/qcom_scm.c | 70 +++++++++--------------------------------= ---- 1 file changed, 13 insertions(+), 57 deletions(-) diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c index fde33acd46b7..dfa298347041 100644 --- a/drivers/firmware/qcom_scm.c +++ b/drivers/firmware/qcom_scm.c @@ -26,10 +26,6 @@ static bool download_mode =3D IS_ENABLED(CONFIG_QCOM_SCM_DOWNLOAD_MODE_DEF= AULT); module_param(download_mode, bool, 0); =20 -#define SCM_HAS_CORE_CLK BIT(0) -#define SCM_HAS_IFACE_CLK BIT(1) -#define SCM_HAS_BUS_CLK BIT(2) - struct qcom_scm { struct device *dev; struct clk *core_clk; @@ -1425,43 +1421,22 @@ static int qcom_scm_probe(struct platform_device *p= dev) return dev_err_probe(&pdev->dev, PTR_ERR(scm->path), "failed to acquire interconnect path\n"); =20 - scm->core_clk =3D devm_clk_get(&pdev->dev, "core"); + scm->core_clk =3D devm_clk_get_optional(&pdev->dev, "core"); if (IS_ERR(scm->core_clk)) { if (PTR_ERR(scm->core_clk) =3D=3D -EPROBE_DEFER) return PTR_ERR(scm->core_clk); - - if (clks & SCM_HAS_CORE_CLK) { - dev_err(&pdev->dev, "failed to acquire core clk\n"); - return PTR_ERR(scm->core_clk); - } - - scm->core_clk =3D NULL; } =20 - scm->iface_clk =3D devm_clk_get(&pdev->dev, "iface"); + scm->iface_clk =3D devm_clk_get_optional(&pdev->dev, "iface"); if (IS_ERR(scm->iface_clk)) { if (PTR_ERR(scm->iface_clk) =3D=3D -EPROBE_DEFER) return PTR_ERR(scm->iface_clk); - - if (clks & SCM_HAS_IFACE_CLK) { - dev_err(&pdev->dev, "failed to acquire iface clk\n"); - return PTR_ERR(scm->iface_clk); - } - - scm->iface_clk =3D NULL; } =20 - scm->bus_clk =3D devm_clk_get(&pdev->dev, "bus"); + scm->bus_clk =3D devm_clk_get_optional(&pdev->dev, "bus"); if (IS_ERR(scm->bus_clk)) { if (PTR_ERR(scm->bus_clk) =3D=3D -EPROBE_DEFER) return PTR_ERR(scm->bus_clk); - - if (clks & SCM_HAS_BUS_CLK) { - dev_err(&pdev->dev, "failed to acquire bus clk\n"); - return PTR_ERR(scm->bus_clk); - } - - scm->bus_clk =3D NULL; } =20 scm->reset.ops =3D &qcom_scm_pas_reset_ops; @@ -1512,38 +1487,19 @@ static void qcom_scm_shutdown(struct platform_devic= e *pdev) } =20 static const struct of_device_id qcom_scm_dt_match[] =3D { - { .compatible =3D "qcom,scm-apq8064", - /* FIXME: This should have .data =3D (void *) SCM_HAS_CORE_CLK */ - }, - { .compatible =3D "qcom,scm-apq8084", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) - }, + { .compatible =3D "qcom,scm-apq8064" }, + { .compatible =3D "qcom,scm-apq8084" }, { .compatible =3D "qcom,scm-ipq4019" }, - { .compatible =3D "qcom,scm-mdm9607", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) }, - { .compatible =3D "qcom,scm-msm8660", .data =3D (void *) SCM_HAS_CORE_CLK= }, - { .compatible =3D "qcom,scm-msm8960", .data =3D (void *) SCM_HAS_CORE_CLK= }, - { .compatible =3D "qcom,scm-msm8916", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) - }, - { .compatible =3D "qcom,scm-msm8953", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) - }, - { .compatible =3D "qcom,scm-msm8974", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) - }, - { .compatible =3D "qcom,scm-msm8976", .data =3D (void *)(SCM_HAS_CORE_CLK= | - SCM_HAS_IFACE_CLK | - SCM_HAS_BUS_CLK) - }, + { .compatible =3D "qcom,scm-mdm9607" }, + { .compatible =3D "qcom,scm-msm8660" }, + { .compatible =3D "qcom,scm-msm8960" }, + { .compatible =3D "qcom,scm-msm8916" }, + { .compatible =3D "qcom,scm-msm8953" }, + { .compatible =3D "qcom,scm-msm8974" }, + { .compatible =3D "qcom,scm-msm8976" }, { .compatible =3D "qcom,scm-msm8994" }, { .compatible =3D "qcom,scm-msm8996" }, - { .compatible =3D "qcom,scm-sm6375", .data =3D (void *)SCM_HAS_CORE_CLK }, + { .compatible =3D "qcom,scm-sm6375" }, { .compatible =3D "qcom,scm" }, {} }; --=20 2.41.0 From nobody Sun Feb 8 23:41:13 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 77625EB64DD for ; Fri, 23 Jun 2023 21:50:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230421AbjFWVup (ORCPT ); Fri, 23 Jun 2023 17:50:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39964 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232807AbjFWVtr (ORCPT ); Fri, 23 Jun 2023 17:49:47 -0400 Received: from mail-lf1-x12a.google.com (mail-lf1-x12a.google.com [IPv6:2a00:1450:4864:20::12a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8904E2707 for ; Fri, 23 Jun 2023 14:49:46 -0700 (PDT) Received: by mail-lf1-x12a.google.com with SMTP id 2adb3069b0e04-4f766777605so1504834e87.1 for ; Fri, 23 Jun 2023 14:49:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687556985; x=1690148985; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=5DBVmcbPM6O4YzyX5TBk3hTNZ7EUrOs7o/4qOremoNA=; b=DLTVi3Ko6yVpZslF+BPqKOP0yiJUwxq4mfFrffeByYiurppR0iv8Pwj9rR+Es5MgkR 44er5a2wZohpMA5HoiEVmEhLqWdlTC6wKk0YQPdzimLPeV2brVMgFqaa4zGJ167odeuk LCqrxyhKuHvZ0RyxLdqqJhJ7vTTloIe/lTJoXFGxYGAbE9QGBgJLquMbFpvmGJJIwi4F ZJjkmQyfH9LuBHRFr6XXwd9V6ryr57hVuFPku3arzNnkJXBKofaNdsrcyVVwnRp1QKT9 v8qObWZs/Jk2iRhMEAYkagEpLKzTmL2zvS4WKenl0DO9/FbXHm/RQ4yM53u4/h4p2epQ QFYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687556985; x=1690148985; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=5DBVmcbPM6O4YzyX5TBk3hTNZ7EUrOs7o/4qOremoNA=; b=XCJTvB+/oX0NpaaTS5dSdZDBTN3hKIg0glANEJ7/inY9SqsP2IUXxynPVc9YNQh8qS ue5P0CYR0ZrqnPyldGN0ihl36W2+TU8nl/V4YEIWdZRPZ8UZkV9MFvvyF3zalbRKajDz sD76bhAV5pxf9IBcO4tjE91yVKDF8iBxNy/eLDWDPNanu2iYmbl7nODcxehAYiR8Cjl7 PP/Rd1lAIXmMz55KBpDsASIReSRgW4M3eE3lEDw2dwY/LQRMlTTIGsS60rivllay+YTM ZgV2h3ZBcBpU82ZuiEAOd8gnf57RXHqhSU2g9nV6Afh0hPEkUVz48W4XcI9a1P6i+u1U yT0g== X-Gm-Message-State: AC+VfDyWp/Tv2NYwiblJXX+O+Fmg9DEa0T6cx3Jl7TY6HYx4bA9cYVi/ HtdEnOGfFRbuX2kDSE9omrrY8Q== X-Google-Smtp-Source: ACHHUZ7D7cOzbDYTujj5NSHsAVJZW3RnYMXWjzODK+/JexenXMCikUfaGFpHbg0lCpRpKgvfka7xVw== X-Received: by 2002:a2e:900a:0:b0:2b4:6195:bb26 with SMTP id h10-20020a2e900a000000b002b46195bb26mr13985888ljg.25.1687556984775; Fri, 23 Jun 2023 14:49:44 -0700 (PDT) Received: from [192.168.1.101] (abyk30.neoplus.adsl.tpnet.pl. [83.9.30.30]) by smtp.gmail.com with ESMTPSA id x20-20020a2e9c94000000b002b48437dcaasm1840919lji.103.2023.06.23.14.49.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Jun 2023 14:49:44 -0700 (PDT) From: Konrad Dybcio Date: Fri, 23 Jun 2023 23:49:43 +0200 Subject: [PATCH 2/3] firmware: qcom_scm: Always return devm_clk_get_optional errors MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20230623-topic-scm_cleanup-v1-2-383089eae98d@linaro.org> References: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> To: Andy Gross , Bjorn Andersson Cc: Marijn Suijten , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1687556981; l=1572; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=pHysGjB4oZJ+5xN5W/FUx19qYCDxjkWrQHiYcbKX6uQ=; b=F9TfQwe9lA2sfHfajdi4PRDRwx/kETtoE4+UfIsGvDLi7W2Ey/0Aj1XfKriTsLSO9cvh0+L5l Aiesih+NUnqC8nf3K7HoDCSDZhyNQmUFCUg0jMBQKusn2IFa34idJ0v X-Developer-Key: i=konrad.dybcio@linaro.org; a=ed25519; pk=iclgkYvtl2w05SSXO5EjjSYlhFKsJ+5OSZBjOkQuEms= Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If devm_clk_get_optional throws an error, something is really wrong. It may be a probe deferral, or it may be a problem with the clock provider. Regardless of what it may be, it should definitely not be ignored. Stop doing that. Signed-off-by: Konrad Dybcio --- drivers/firmware/qcom_scm.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c index dfa298347041..d7fc972b20ab 100644 --- a/drivers/firmware/qcom_scm.c +++ b/drivers/firmware/qcom_scm.c @@ -1422,22 +1422,16 @@ static int qcom_scm_probe(struct platform_device *p= dev) "failed to acquire interconnect path\n"); =20 scm->core_clk =3D devm_clk_get_optional(&pdev->dev, "core"); - if (IS_ERR(scm->core_clk)) { - if (PTR_ERR(scm->core_clk) =3D=3D -EPROBE_DEFER) - return PTR_ERR(scm->core_clk); - } + if (IS_ERR(scm->core_clk)) + return PTR_ERR(scm->core_clk); =20 scm->iface_clk =3D devm_clk_get_optional(&pdev->dev, "iface"); - if (IS_ERR(scm->iface_clk)) { - if (PTR_ERR(scm->iface_clk) =3D=3D -EPROBE_DEFER) - return PTR_ERR(scm->iface_clk); - } + if (IS_ERR(scm->iface_clk)) + return PTR_ERR(scm->iface_clk); =20 scm->bus_clk =3D devm_clk_get_optional(&pdev->dev, "bus"); - if (IS_ERR(scm->bus_clk)) { - if (PTR_ERR(scm->bus_clk) =3D=3D -EPROBE_DEFER) - return PTR_ERR(scm->bus_clk); - } + if (IS_ERR(scm->bus_clk)) + return PTR_ERR(scm->bus_clk); =20 scm->reset.ops =3D &qcom_scm_pas_reset_ops; scm->reset.nr_resets =3D 1; --=20 2.41.0 From nobody Sun Feb 8 23:41:13 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 45AE9EB64D7 for ; Fri, 23 Jun 2023 21:50:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231784AbjFWVug (ORCPT ); Fri, 23 Jun 2023 17:50:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40016 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232818AbjFWVts (ORCPT ); Fri, 23 Jun 2023 17:49:48 -0400 Received: from mail-lf1-x12f.google.com (mail-lf1-x12f.google.com [IPv6:2a00:1450:4864:20::12f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 85C8A2723 for ; Fri, 23 Jun 2023 14:49:47 -0700 (PDT) Received: by mail-lf1-x12f.google.com with SMTP id 2adb3069b0e04-4f86bc35f13so1565403e87.1 for ; Fri, 23 Jun 2023 14:49:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687556985; x=1690148985; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=SlONZ8TKyqT3PdFqD54Nj2aaobJzFqkktvSArbuqu70=; b=d14/Sqwli0wZo1aDlP1ix49NXOgH7Az2yJI2ZTAVgrTdZSAY7laI7V8/5jk5KRaB17 AkfudzVs4F6WQpi4MMr9O9Q9KJfSbkDlfrEjzJOZN6gcso065RCdVNJSLcgbgbP7Dl9n IvQw3hiDFbkZmIliWWNSV0d5wk+8/wOWjWBO2B14YUSvteqaWZuZKpEZXUo8k8f4R0Iv OqashMtNr449Lv4lT1EJkhZTe5gFcnXfivHmOWY/AOdYwFern7SpF6nTxcDMCVUmSJnJ J3SdsSxKBvwxgmKCINkEGS2BE+CkRW6QYSjjQTV6Ktf16AKbB14BZ7T8zoBkFAGJjhkG WCgA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687556985; x=1690148985; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=SlONZ8TKyqT3PdFqD54Nj2aaobJzFqkktvSArbuqu70=; b=Kv4XKXgFSdhK3sFQcpq0Q97JjKun5g80BeAA6E/pjiSNAj+iEITL5EVek2eFcdk8xe mI9Lkc6SHdy7HlVRJMH8pgGzbf5QGDmbjjKNB6+z1LZWCxRz247ls001UacspOPpA7ew GV3DVIWwAOWyZh84An/j8GJhrQaDVhb5WL75Zzdc3HfFGxmRijEZBeglqZuw780S1lMU Ay9DQEaeHcvZYybQZaxLFCQymIZEhk0pTQOqgKXS8BAULK7Kyxp2cbVctRz52kuMJQM4 UCSRceDBsiLOqfOJFHlIN5Rf80U+cM07M95ozmw67zu51dM8wGzR+ryiwg6OdOzLbO8P 240Q== X-Gm-Message-State: AC+VfDz7a+56V2j1uHhQFK9OGSd/lXCXcOG9Mh4rRWTddHRhgQFMIC6F X7pV5aHI1F2Uv8y8jBzObvemuw== X-Google-Smtp-Source: ACHHUZ4tA49jt4MntzmSZCt35XsZBv0Q4GrQYvthIP75Bx6oK9QUCwzfeEzGH+ZZRCY1jlN33JTxUg== X-Received: by 2002:a2e:9854:0:b0:2b4:756b:8e71 with SMTP id e20-20020a2e9854000000b002b4756b8e71mr12630714ljj.44.1687556985660; Fri, 23 Jun 2023 14:49:45 -0700 (PDT) Received: from [192.168.1.101] (abyk30.neoplus.adsl.tpnet.pl. [83.9.30.30]) by smtp.gmail.com with ESMTPSA id x20-20020a2e9c94000000b002b48437dcaasm1840919lji.103.2023.06.23.14.49.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Jun 2023 14:49:45 -0700 (PDT) From: Konrad Dybcio Date: Fri, 23 Jun 2023 23:49:44 +0200 Subject: [PATCH 3/3] firmware: qcom_scm: Drop useless compatibles MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20230623-topic-scm_cleanup-v1-3-383089eae98d@linaro.org> References: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v1-0-383089eae98d@linaro.org> To: Andy Gross , Bjorn Andersson Cc: Marijn Suijten , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1687556981; l=1502; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=YeauYl4ZQZY5xFS1G6VWl/VwUEAHSGaL4eh3BOpBnEE=; b=sxaKygGpuvRJhsRXCw9CfYWD83iHXn9UfSEtVsNIFx5potBlUyoyY1D2CYYobi0D7yey7a337 E0RwUnUXfZ8AGyqZCKQAgdFaZSKTNauXyrqi4F2T0Z3LFUmHT+N4uKo X-Developer-Key: i=konrad.dybcio@linaro.org; a=ed25519; pk=iclgkYvtl2w05SSXO5EjjSYlhFKsJ+5OSZBjOkQuEms= Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org There are three categories of compatibles within the driver: 1. Ones which were introduced without a qcom,scm fallback 2. Ones which were introduced with a qcom,scm fallback 3. Ones which were defined but never used Keep 1 for backwards compatibility and axe 2 & 3. Signed-off-by: Konrad Dybcio --- drivers/firmware/qcom_scm.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c index d7fc972b20ab..ff0e8911499c 100644 --- a/drivers/firmware/qcom_scm.c +++ b/drivers/firmware/qcom_scm.c @@ -1481,20 +1481,15 @@ static void qcom_scm_shutdown(struct platform_devic= e *pdev) } =20 static const struct of_device_id qcom_scm_dt_match[] =3D { + { .compatible =3D "qcom,scm" }, + + /* Legacy entries kept for backwards compatibility */ { .compatible =3D "qcom,scm-apq8064" }, { .compatible =3D "qcom,scm-apq8084" }, { .compatible =3D "qcom,scm-ipq4019" }, - { .compatible =3D "qcom,scm-mdm9607" }, - { .compatible =3D "qcom,scm-msm8660" }, - { .compatible =3D "qcom,scm-msm8960" }, - { .compatible =3D "qcom,scm-msm8916" }, { .compatible =3D "qcom,scm-msm8953" }, { .compatible =3D "qcom,scm-msm8974" }, - { .compatible =3D "qcom,scm-msm8976" }, - { .compatible =3D "qcom,scm-msm8994" }, { .compatible =3D "qcom,scm-msm8996" }, - { .compatible =3D "qcom,scm-sm6375" }, - { .compatible =3D "qcom,scm" }, {} }; MODULE_DEVICE_TABLE(of, qcom_scm_dt_match); --=20 2.41.0