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 89798CD98CE for ; Fri, 12 Jun 2026 21:49:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=bHHV9NqOsH/MKrWgOYYnftqX6AVLRHobyNtf2M1ZJpQ=; b=BJC0E7AHrRaVa2 X91olDHX7zyXgvbKXLP4H7B0xp5BEvxJX3gZJAcHAaUD6Y6qkdFCy5Ny5AzkXOEJFnxwiH8qGEFQT 5hh3Ery5NOHzSdT0dei+XE+b2d+MeVIHL8W4lTmgG4h9fVgbGEiUQK8vQ4IvlUKDpQqi7O68XOrmK mhORGk9g67so/Jtb+ScnV/Qx/yu/mfgjsgU4fB5Om4IUoQ+Km7Zh8jDFNbgmdMSiplVRwsu0vrkWg SVrcQ6ZQmD2kWtHz9dAbnkXJ6iU1LNbREwfjFz0xmggOwPzgoCd2Ht7GmczukRAqEYjay6P9MWvIE j0xV7bDUxWvl7cLhyvEQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wY9kV-0000000BcSl-11PQ; Fri, 12 Jun 2026 21:49:11 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wY9kU-0000000BcSU-0BuS; Fri, 12 Jun 2026 21:49:10 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 5576B42A88; Fri, 12 Jun 2026 21:49:09 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 09D061F000E9; Fri, 12 Jun 2026 21:49:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1781300949; bh=+cf6F8jctolB5ST6PmkJlXK4IfpFbZ885OFDt8xDlHY=; h=From:To:Cc:Subject:Date; b=NgvZfUYslzX69igJ98J3aKT5QpU/LvB+bcvR6IfHjp5Ak8lNc1d5l+rmPB39l6sFi 0aySnOnRT55a8pLRaXgSjbf69pomTKepuO3nuJOPtdUy2LlCye86QbEU+JGm4Q8Y6B 6dZHAmAdJ9ox92iyGvy2DnX0dz5g4FsW05nlfiBs42khlKDv6oiRsoRy1+hukQe2W6 cNuJVYoovR9QSSCcGWOuDiHylkFsw9cy+OUYo0m6BLWPeZ8bpiBMWQ28Rv0WhY86eN 8IgEcw/yT3MnT0CgeYKfU/IIInbT+g7N+YyrKzuReRINdkI49LLDyBe3d2CSafiXH8 1LirCTdSBWmvQ== From: "Rob Herring (Arm)" To: Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Peter Ujfalusi , Heiko Stuebner Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org Subject: [PATCH] ASoC: Match DT helper types Date: Fri, 12 Jun 2026 16:49:02 -0500 Message-ID: <20260612214904.1882991-1-robh@kernel.org> X-Mailer: git-send-email 2.53.0 MIME-Version: 1.0 X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org The affected ASoC drivers read properties whose bindings use boolean flags, normal uint32 cells, or phandle-style arrays. Using helpers for a different encoding makes those accesses disagree with the binding. Use helpers that match the documented encoding and keep the existing driver storage by copying through temporary values where needed. Assisted-by: Codex:gpt-5-5 Signed-off-by: Rob Herring (Arm) --- sound/soc/codecs/ak4642.c | 2 +- sound/soc/codecs/sta32x.c | 6 +++--- sound/soc/codecs/sta350.c | 6 +++--- sound/soc/codecs/twl4030.c | 6 ++---- sound/soc/rockchip/rockchip_pdm.c | 3 +-- 5 files changed, 10 insertions(+), 13 deletions(-) diff --git a/sound/soc/codecs/ak4642.c b/sound/soc/codecs/ak4642.c index fe035d2fc913..b68555c8fbaf 100644 --- a/sound/soc/codecs/ak4642.c +++ b/sound/soc/codecs/ak4642.c @@ -613,7 +613,7 @@ static struct clk *ak4642_of_parse_mcko(struct device *dev) if (of_property_read_u32(np, "clock-frequency", &rate)) return NULL; - if (of_property_read_bool(np, "clocks")) + if (of_property_present(np, "clocks")) parent_clk_name = of_clk_get_parent_name(np, 0); of_property_read_string(np, "clock-output-names", &clk_name); diff --git a/sound/soc/codecs/sta32x.c b/sound/soc/codecs/sta32x.c index b9f9784f5164..857b729e6978 100644 --- a/sound/soc/codecs/sta32x.c +++ b/sound/soc/codecs/sta32x.c @@ -1038,7 +1038,7 @@ static int sta32x_probe_dt(struct device *dev, struct sta32x_priv *sta32x) { struct device_node *np = dev->of_node; struct sta32x_platform_data *pdata; - u16 tmp; + u32 tmp; pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); if (!pdata) @@ -1063,8 +1063,8 @@ static int sta32x_probe_dt(struct device *dev, struct sta32x_priv *sta32x) of_property_read_bool(np, "st,needs_esd_watchdog"); tmp = 140; - of_property_read_u16(np, "st,drop-compensation-ns", &tmp); - pdata->drop_compensation_ns = clamp_t(u16, tmp, 0, 300) / 20; + of_property_read_u32(np, "st,drop-compensation-ns", &tmp); + pdata->drop_compensation_ns = clamp_t(u32, tmp, 0, 300) / 20; /* CONFE */ pdata->max_power_use_mpcc = diff --git a/sound/soc/codecs/sta350.c b/sound/soc/codecs/sta350.c index 71af82b099c0..367d60577906 100644 --- a/sound/soc/codecs/sta350.c +++ b/sound/soc/codecs/sta350.c @@ -1091,7 +1091,7 @@ static int sta350_probe_dt(struct device *dev, struct sta350_priv *sta350) struct device_node *np = dev->of_node; struct sta350_platform_data *pdata; const char *ffx_power_mode; - u16 tmp; + u32 tmp; u8 tmp8; pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); @@ -1131,8 +1131,8 @@ static int sta350_probe_dt(struct device *dev, struct sta350_priv *sta350) } tmp = 140; - of_property_read_u16(np, "st,drop-compensation-ns", &tmp); - pdata->drop_compensation_ns = clamp_t(u16, tmp, 0, 300) / 20; + of_property_read_u32(np, "st,drop-compensation-ns", &tmp); + pdata->drop_compensation_ns = clamp_t(u32, tmp, 0, 300) / 20; pdata->oc_warning_adjustment = of_property_read_bool(np, "st,overcurrent-warning-adjustment"); diff --git a/sound/soc/codecs/twl4030.c b/sound/soc/codecs/twl4030.c index 9476cdfd4dde..91f5f907951a 100644 --- a/sound/soc/codecs/twl4030.c +++ b/sound/soc/codecs/twl4030.c @@ -202,13 +202,11 @@ static void twl4030_get_board_param_values(struct twl4030_board_params *board_params, struct device_node *node) { - int value; - of_property_read_u32(node, "ti,digimic_delay", &board_params->digimic_delay); of_property_read_u32(node, "ti,ramp_delay_value", &board_params->ramp_delay_value); of_property_read_u32(node, "ti,offset_cncl_path", &board_params->offset_cncl_path); - if (!of_property_read_u32(node, "ti,hs_extmute", &value)) - board_params->hs_extmute = value; + if (of_property_read_bool(node, "ti,hs_extmute")) + board_params->hs_extmute = 1; if (of_property_present(node, "ti,hs_extmute_gpio")) board_params->hs_extmute = 1; diff --git a/sound/soc/rockchip/rockchip_pdm.c b/sound/soc/rockchip/rockchip_pdm.c index c69cdd6f2499..966105a544f5 100644 --- a/sound/soc/rockchip/rockchip_pdm.c +++ b/sound/soc/rockchip/rockchip_pdm.c @@ -546,8 +546,7 @@ static int rockchip_pdm_path_parse(struct rk_pdm_dev *pdm, struct device_node *n unsigned int path[PDM_PATH_MAX]; int cnt = 0, ret = 0, i = 0, val = 0, msk = 0; - cnt = of_count_phandle_with_args(node, "rockchip,path-map", - NULL); + cnt = of_property_count_u32_elems(node, "rockchip,path-map"); if (cnt != PDM_PATH_MAX) return cnt; -- 2.53.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip