From nobody Sun Feb 8 20:23:34 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 8FDAAEB64DD for ; Sat, 24 Jun 2023 12:23:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233171AbjFXMXz (ORCPT ); Sat, 24 Jun 2023 08:23:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51712 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233132AbjFXMXv (ORCPT ); Sat, 24 Jun 2023 08:23:51 -0400 Received: from mail-lf1-x12d.google.com (mail-lf1-x12d.google.com [IPv6:2a00:1450:4864:20::12d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0C1F6E41 for ; Sat, 24 Jun 2023 05:23:50 -0700 (PDT) Received: by mail-lf1-x12d.google.com with SMTP id 2adb3069b0e04-4f96d680399so2020412e87.0 for ; Sat, 24 Jun 2023 05:23:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687609428; x=1690201428; 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=4mlusQIqHLc3L1kKJFIe3oY0E4pRPb3J1UORktAQD1k=; b=OZGC+9QI1M3ODN9W7x37gwtYeEEBZK1C4ZBcjZxWEjq+3+Csf55wDuNCEuAoUCMFA2 z7pg9Wg6+Xhr3fK5LtK/iPlPJq7TfMZC0OwOL+Z0ChzK13P/ldd6DrUokIboyx9QYnOO 08vGi/1T5NHk6NcnPBMUIzmbYsI5BpWi0cEIAhmTY8cLJbnQ6OjihjmIudftwqRiaGM5 xpN8EmGucaHcmnPQ6Ja5JNX30N1+W3xf0PriZf8eclootT9EEBYTwQvSAfMnsR7Bu00s VZYgwiIoCCJJbDuxHTARjplZEDBG0AQcwxjbcO8h42KRvK6nYwYXOotf0mgMWuh3TFxQ k1og== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687609428; x=1690201428; 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=4mlusQIqHLc3L1kKJFIe3oY0E4pRPb3J1UORktAQD1k=; b=P2ppbsmpEGvMr1E5KrosTxUAfNwkdviy4l8WULd3QeOrkl4a2vNAik/C3IvwSDei3n FXzpyQWBJL/h6G+EQOi4T1ihhQXM+FtElvbMqFpvt3zsbNgOeD9Y5G8fB3uPrzATG1cZ OVpFvBS6xYArfQhdGDvzodwy5kHqg/AR/thdoN+9Za/4RjbutWnCq73tAmHOy+sZ83mq qMXCO8A7zdGoR3rA/3sJ0QQHYWqmTRudmO/UnoJv90EoBDW2r4wAOaCEnMryGKqXjW9K nqSbk/8mbkSOdq0Me+xQKlZtB4GLqHvRiXtdO5+WHaneOXHmbkFHEn0bSovtaA4buOVG c/NQ== X-Gm-Message-State: AC+VfDzb0CsP2As3i2NnebO5tUdQorTaC1M2lXdzeuWkfkZBDN40M+sQ XaDwE54PvyQ0BHyscRbrdDTnJQ== X-Google-Smtp-Source: ACHHUZ5wvzrIOG2lC6PssO6R/HYy7Ys+3tceM2SMKPFaFQEPBrqqNy/ZeOtox/aofROgKnSiCgG8Qg== X-Received: by 2002:a05:6512:31c8:b0:4f9:5d34:44b1 with SMTP id j8-20020a05651231c800b004f95d3444b1mr8229225lfe.2.1687609428226; Sat, 24 Jun 2023 05:23:48 -0700 (PDT) Received: from [192.168.1.101] (abyk197.neoplus.adsl.tpnet.pl. [83.9.30.197]) by smtp.gmail.com with ESMTPSA id s13-20020a19ad4d000000b004f00189e1dcsm260200lfd.117.2023.06.24.05.23.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 24 Jun 2023 05:23:47 -0700 (PDT) From: Konrad Dybcio Date: Sat, 24 Jun 2023 14:23:45 +0200 Subject: [PATCH v2 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-v2-1-9db8c583138d@linaro.org> References: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@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=1687609426; l=5055; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=c6mXCk54PRq5wka3veC3/OftoitCZW2RQQCuecVwHSI=; b=CohtLcSjA2HX4NMnCMM24PR0MDttb7QJBE91IfGXR0mpfZQ66Toy6jR1mwMo2hrzJi23GRBNq 5PuMfr8LJhmDSUG+OpbyI+R7P41eN91mImDT+XPkMy9LQ+wIKsWP3Ge 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 | 73 ++++++++---------------------------------= ---- 1 file changed, 13 insertions(+), 60 deletions(-) diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c index fde33acd46b7..237d05d6208b 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; @@ -1405,7 +1401,6 @@ static irqreturn_t qcom_scm_irq_handler(int irq, void= *data) static int qcom_scm_probe(struct platform_device *pdev) { struct qcom_scm *scm; - unsigned long clks; int irq, ret; =20 scm =3D devm_kzalloc(&pdev->dev, sizeof(*scm), GFP_KERNEL); @@ -1418,50 +1413,27 @@ static int qcom_scm_probe(struct platform_device *p= dev) =20 mutex_init(&scm->scm_bw_lock); =20 - clks =3D (unsigned long)of_device_get_match_data(&pdev->dev); - scm->path =3D devm_of_icc_get(&pdev->dev, NULL); if (IS_ERR(scm->path)) 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 +1484,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 20:23:34 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 E5F18EB64DA for ; Sat, 24 Jun 2023 12:23:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233180AbjFXMX5 (ORCPT ); Sat, 24 Jun 2023 08:23:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51720 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233153AbjFXMXw (ORCPT ); Sat, 24 Jun 2023 08:23:52 -0400 Received: from mail-lj1-x22c.google.com (mail-lj1-x22c.google.com [IPv6:2a00:1450:4864:20::22c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C3461E45 for ; Sat, 24 Jun 2023 05:23:50 -0700 (PDT) Received: by mail-lj1-x22c.google.com with SMTP id 38308e7fff4ca-2b47a15ca10so24751091fa.1 for ; Sat, 24 Jun 2023 05:23:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687609429; x=1690201429; 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=SdwUD4ja8kl0C3si3v+OvYMyCtcQIhX5wG+ZPdgw56w=; b=hWJrUE+yTMBo33EWAQ3Fuij490p7Z6ilnJGULElb6XCef93PXynQsqdx65ZyA9hvMP w+j0Rm3iw7Anuz0b7KQ5oLmrSxm82yiCBW2dPS7ToTeJI/MONb1/+v+owUk5iSuUTy1F erDe/Ii8RS6H+eMWDHfzeWu1RzWmVh80oRQyplQLmXi2QHoCLwLAbxqUwsN26va8QJ+W NfRYg+rmlgHipqH8hXioYWNcI6I0tr9thcD4aYq7aPt0q7MQVR3CHnMEjWrSckPqYegX hEsWpGxbNyr5rKUw61VFdhSEIBrYE55sJuHwiHSUkdcys/fD0OV7ckiWTGxOTOwLOkya ekqg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687609429; x=1690201429; 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=SdwUD4ja8kl0C3si3v+OvYMyCtcQIhX5wG+ZPdgw56w=; b=FnxkYlYgncaIG2ghPLSQ3+vhCOkU55W+atumnkvXKAkSieZeEoJguvmbF0fEtyDCN5 u8Dix2oGw6XE/E5A4Rd4r85q9k+ZZPdi1g2uxj+RGLAVPnlOG92oOkQDmP0S2FM4GnmO j9UXSVX/kVGb032snD4tbog9A5lG9f4MxfEdyK9XE8rOqhfmpie9gfUyPZRcx9JOVEW2 mrhplyGL5WNPHrLvOPj9cV+M6X3liHBlvt2NMljc3TJj+n5tzbIXnQobv3pva1j5f4sB YRU7DOtXWSe/XqxR/6zHjYTyoMKXEdiwOuDQkBvOXUJOOeSWXjJbU1S8eu9bUN8oyB8G Is3Q== X-Gm-Message-State: AC+VfDzIJ3ACeif8CedNCabetMSOLDjRfsCB4Vvwe1esR5Wla6GLYSqz fC5TVqN8KQoEIsPjmdDiznH26w== X-Google-Smtp-Source: ACHHUZ7yuVQdFm4WKHzI+HOH8HyATj9c84QUrWlItmCnDxat+7yp+Tg9zXqsWsAlNAN1iTEDfufRVQ== X-Received: by 2002:a19:5050:0:b0:4f8:598b:e62d with SMTP id z16-20020a195050000000b004f8598be62dmr13826809lfj.56.1687609429133; Sat, 24 Jun 2023 05:23:49 -0700 (PDT) Received: from [192.168.1.101] (abyk197.neoplus.adsl.tpnet.pl. [83.9.30.197]) by smtp.gmail.com with ESMTPSA id s13-20020a19ad4d000000b004f00189e1dcsm260200lfd.117.2023.06.24.05.23.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 24 Jun 2023 05:23:48 -0700 (PDT) From: Konrad Dybcio Date: Sat, 24 Jun 2023 14:23:46 +0200 Subject: [PATCH v2 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-v2-2-9db8c583138d@linaro.org> References: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@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=1687609426; l=1572; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=LId2EVrQK5HfRRaxbRB0VnCSfdSPSyuZN4O6GtpVCuM=; b=TGAD5/2nSgKWjnQ1tOb5GK/LXL4aWbiquVPfQsfgiWQ9LZowwvQHrOdjJzXUbK3Q9yl5l/+ra qMQ6GwEnkJpDNQdO6nYIpMZmx15RuochA73XruUynPQoYXNWr7DlCmW 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 237d05d6208b..b8398002205d 100644 --- a/drivers/firmware/qcom_scm.c +++ b/drivers/firmware/qcom_scm.c @@ -1419,22 +1419,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 20:23:34 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 4B597EB64DA for ; Sat, 24 Jun 2023 12:24:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233196AbjFXMYA (ORCPT ); Sat, 24 Jun 2023 08:24:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51728 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233156AbjFXMXw (ORCPT ); Sat, 24 Jun 2023 08:23:52 -0400 Received: from mail-lj1-x22a.google.com (mail-lj1-x22a.google.com [IPv6:2a00:1450:4864:20::22a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BAD03BF for ; Sat, 24 Jun 2023 05:23:51 -0700 (PDT) Received: by mail-lj1-x22a.google.com with SMTP id 38308e7fff4ca-2b46cfde592so24899371fa.0 for ; Sat, 24 Jun 2023 05:23:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687609430; x=1690201430; 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=avKiWxoWYl1UXVnI1jhFRgj1Bd8Ey8Z1YlDdy76fbHw=; b=Y/r6vNeazTy5SqQSn7ySKBYS/EBpIMfd7saTZB52qjSq5nZEBXdxO4wWvT8zv6rMrO wZyWiR+0Tnfojf1i9AC5wqXMGQOyzw7Am2DdITaQAj/w2MeP2uY+GgDx6+i/bO2nCh7R ejJwJggxe/DpvTjOam5ljnvaB1OYct/bTIAqWtACIdNRUFA+gVMPv+meTW6eMeC+0Hod y5TgEkdnnF9RCFhINPanmzx6Lvc1A35m0fDUo8nVmyySkRLsLHOVbaWZhuKKlUqnZYnr gamLQAzCSH/X6JOQvqrQjebbafRTQY+fxE7qIjoFcqhMzKU5OzKE9C/mJRAcvuk+/pnB jUDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687609430; x=1690201430; 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=avKiWxoWYl1UXVnI1jhFRgj1Bd8Ey8Z1YlDdy76fbHw=; b=AaSaMVjdPyE5kxLonKYmU+/UMLIUv6j7j0f9SekUaOugoapnOSJLhZ0TsUpIsY3QuR 7zJmWvv5O7hYJmPUHVtyV/jGPExnZffgqO11Mzdf6eG0P1PxfZNFtygpx3t0MsMinyHh SoymiHphwDqfUMIErET4Mz4kJH6wRpJQFkvkVT0kZVwYPIX+1TC2caRaodKMZgkfRKx3 kl7vlUKgcT8Pxe8VCuDxTPvCtf3SVzTWilD60kNI0xiimAbVint33qlXVob1UZOiRH0x rWV74eY1ucl7JybEti5P0GHWGVb0vyElKqXdeNSAVUZJysRZmaMOfx1zNA+NE4eMK7eF pS8w== X-Gm-Message-State: AC+VfDxVElgtQ2qiFyPo1GaBWUGxRXq1jlzOK0NdNl077bilo/At4Zu1 VXT+hzAcImjY35PmI0yLd0DqyQ== X-Google-Smtp-Source: ACHHUZ7UTAx2/bEt+ZSl5QD0opwRKHGjqzeFROyYfCzAssepJG3s6vylbvwcndvYK/Kdd5NyeYH6kw== X-Received: by 2002:ac2:4992:0:b0:4f8:453f:58f3 with SMTP id f18-20020ac24992000000b004f8453f58f3mr16346770lfl.40.1687609430036; Sat, 24 Jun 2023 05:23:50 -0700 (PDT) Received: from [192.168.1.101] (abyk197.neoplus.adsl.tpnet.pl. [83.9.30.197]) by smtp.gmail.com with ESMTPSA id s13-20020a19ad4d000000b004f00189e1dcsm260200lfd.117.2023.06.24.05.23.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 24 Jun 2023 05:23:49 -0700 (PDT) From: Konrad Dybcio Date: Sat, 24 Jun 2023 14:23:47 +0200 Subject: [PATCH v2 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-v2-3-9db8c583138d@linaro.org> References: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@linaro.org> In-Reply-To: <20230623-topic-scm_cleanup-v2-0-9db8c583138d@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=1687609426; l=1502; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=2851rppnMfhTFcDTpFWS0mg9wSl32exm/NRmmxF+T/w=; b=Qy3D0ksutlS9UBuf6hqlAIbaAbyXSeHcDCXBDPBTZYPkD2ERbRol6/AF6Z8ZTspgmJZHgNBjx x6ELhXKung9BQemO4xI9iX3W25RZf4QdqPkxrMs4VF+BTUIB7z+cHss 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 b8398002205d..b6055b1c18d8 100644 --- a/drivers/firmware/qcom_scm.c +++ b/drivers/firmware/qcom_scm.c @@ -1478,20 +1478,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