From nobody Sun Jul 26 01:44:31 2026 Received: from mail-pj1-f46.google.com (mail-pj1-f46.google.com [209.85.216.46]) (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 D97FE377566 for ; Fri, 10 Jul 2026 05:02:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783659768; cv=none; b=Fgw0jdFAsbKFWZvxVLlQhW3OZZHN8zhhbb1OjwK5X42k4HlfFurXdrRFjRACNrizUAig69hnXEQzOCHqVztKV06g7GE2eBR3gPsIn/ELyuKPu6az4TStGN4xG08pSds4e/dMkr3J1ZcZ48165A7FN9b6zrlcgncguWiM/dIGyfY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783659768; c=relaxed/simple; bh=Xa8yF/IuLRLtCtMkxppwZASR9ScQ7OKF5bveI9XEJgY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QpSJ8dm9ekCeqfkh+x+DWSnW3YwaRk3+s/pDI7DwN+vzBVx90Iaca9ZcSdNxXkJwV9F2faACpsML5SckXr45uttc7GEdS5SJVCcveIlEXO/llqSKW0sPPnVT2p4pRcKpHS+bJHA3sg/AO4wUL5CaJzWYqgozG0i0S7DQSt83A1I= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=n3kJAbxa; arc=none smtp.client-ip=209.85.216.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="n3kJAbxa" Received: by mail-pj1-f46.google.com with SMTP id 98e67ed59e1d1-3817f800c8bso589066a91.1 for ; Thu, 09 Jul 2026 22:02:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783659766; x=1784264566; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=FpEEKAJp3BqSZSJR3hfnA3eBxaF2LazIySDJM1KKJg8=; b=n3kJAbxakCVF2Z8SjKvfn0tZ06s6Ve8WcedYlaDfIkfEAe3YPTVOhj95n0I6+LXlrP K92Q/xSe8wGTLDN5QP+10eN04y+RqQLDk0bLM4Xn0209chV0TTZqXmaGLzjcrF4eiZzh qBBqim1nfhnzO/78wWXrtinqHt/tRO7xIyu7iKVnHS5wvhqO+VWKt0ShJ9niSvPQ1DxI OvvtAp6GEW/WJ6Ud2Rkcj65xCzg8LopZb71zy0RXvSVfk3N2K2fVgIaM2KPxJRIpCkrn UCybQq6kpVl8wcMzNw1s+JbZPhE7RQP1Bu9w2DwB4///B0weeh20Td3jsaSSpeNs7NQg mjtA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783659766; x=1784264566; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=FpEEKAJp3BqSZSJR3hfnA3eBxaF2LazIySDJM1KKJg8=; b=VvHWBBZTVhGZHuOBENWVJ20eB8RdaXztVCJquB/7tTswDVmW1U9H7Q9ACFnc//R5rL /f8UnzaMdKiwB/f7VXPIRUEpaLW2FhY9cMJo+RRKGjVO5lFkGSIBYmR2ykehueWrk38a W4970RPrMBzg8X2FNR6m0Z1yVO91il4zcXVhiP/+3fm1U9IE6+NWDP2Nk9QvXknfdwa8 nd0LNKjibZzAG5nIHdEU1fi/RvhvxwxbJK5LhE4xw1H1CEw3vvzfWfHGhKoFXoRJkk6u OzIQDvX7wzYOthjrRs/JHDMDR5UBaHCxnXRcfa7kD9VmLqZf7rotSJ8LORCEndWc8CLJ BJRw== X-Forwarded-Encrypted: i=1; AHgh+RpmuWzsFUgvmznWA2iUq524mwL5GDTe4HtFEytDnqYJeOafCfWeVBa5UvMS7pfxODWBxVG2LfQXiiYVTLQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yw5AgMCK42XTq78W2qR/+rgmhgvxS3zVaErfvZDwluWhYsTZxU/ 731dTRHRjnuDeE2u7eQDzWRea2zGitlJfP/dUykbiEuYMx0NZkhD4FLY X-Gm-Gg: AfdE7clwvEIp+dTPy0ceU6Pjx8KmekkVz/uCTR4G0ZtUcwxAIqW6mVIR8uKK6r10CHK S6lEj2+ucDFh0JYBg4cr/+9fOIPBjlVyDbNpmmCkrSGTDZf3sL5y1ncf/SOls7lDSM5+k1d2eue HPDzs093onSDArgN31AFP6YzqY8rKNgoQTvNiy1HQmsLxAh6nepjgZ9daxK4IU8eaanMClenR4b k5p+kZklKNMoWplaadJiR42Ebgvh8zjwHe8b4rRMLh+q+nq/hwYynSyDhLTu8Pl9XONoprFJY1Q D+00sDnxdtrq0ilqlvcfRH4dplk2tXghmopBX76peFUGkMbv5cqgaeLrWnIktpkxQ0zyyfPBJjf GrV5J4TiGicArai4DdxY27g3sld0azHTjWF0B2NirceLRUwSjec0dO3GvCvHbO6a3DRfJkf3zxG 0OCIuYI9RfwiKuelETPN0Lg57iqyxDH0ONUpvKhxA6Z1FtMQA= X-Received: by 2002:a17:90b:50:b0:36a:fcf5:64d2 with SMTP id 98e67ed59e1d1-389407321b5mr9590297a91.16.1783659765935; Thu, 09 Jul 2026 22:02:45 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-38a5516ad85sm2139647a91.2.2026.07.09.22.02.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 22:02:45 -0700 (PDT) From: phucduc.bui@gmail.com To: Vincenzo Frascino , Jaroslav Kysela , Takashi Iwai , Liam Girdwood , Mark Brown , Michal Simek Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bui duc phuc Subject: [PATCH 1/2] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling Date: Fri, 10 Jul 2026 12:02:22 +0700 Message-ID: <20260710050223.14266-2-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260710050223.14266-1-phucduc.bui@gmail.com> References: <20260710050223.14266-1-phucduc.bui@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: bui duc phuc Use dev_err_probe() for probe error handling where appropriate to simplify the code and properly handle deferred probe. Also remove redundant error messages when the called helper already reports failures, returning the error directly to avoid duplicate logging. Signed-off-by: bui duc phuc Reviewed-by: Michal Simek --- sound/soc/xilinx/xlnx_i2s.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/sound/soc/xilinx/xlnx_i2s.c b/sound/soc/xilinx/xlnx_i2s.c index ca915a001ad5..0676da122edd 100644 --- a/sound/soc/xilinx/xlnx_i2s.c +++ b/sound/soc/xilinx/xlnx_i2s.c @@ -185,17 +185,15 @@ static int xlnx_i2s_probe(struct platform_device *pde= v) return PTR_ERR(drv_data->base); =20 ret =3D of_property_read_u32(node, "xlnx,num-channels", &drv_data->channe= ls); - if (ret < 0) { - dev_err(dev, "cannot get supported channels\n"); - return ret; - } + if (ret < 0) + return dev_err_probe(dev, ret, "cannot get supported channels\n"); + drv_data->channels *=3D 2; =20 ret =3D of_property_read_u32(node, "xlnx,dwidth", &drv_data->data_width); - if (ret < 0) { - dev_err(dev, "cannot get data width\n"); - return ret; - } + if (ret < 0) + return dev_err_probe(dev, ret, "cannot get data width\n"); + switch (drv_data->data_width) { case 16: format =3D SNDRV_PCM_FMTBIT_S16_LE; @@ -233,10 +231,8 @@ static int xlnx_i2s_probe(struct platform_device *pdev) =20 ret =3D devm_snd_soc_register_component(&pdev->dev, &xlnx_i2s_component, &drv_data->dai_drv, 1); - if (ret) { - dev_err(&pdev->dev, "i2s component registration failed\n"); + if (ret) return ret; - } =20 dev_info(&pdev->dev, "%s DAI registered\n", drv_data->dai_drv.name); =20 --=20 2.43.0 From nobody Sun Jul 26 01:44:31 2026 Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) (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 83941374A0D for ; Fri, 10 Jul 2026 05:02:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.45 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783659771; cv=none; b=XF3OqrH1P1ZsTcRI4p84RYTfujITn3rC3aR58M+K6vvm0m7Iz/6yrQN32vYYpX8OriAGVKheDJWQweY4qF57Y5ptt5OxD470yFgRph90aIV947ppupU6arFfedwkJmOrjWVZJEU6Gln8Wx35V8f97kucGYKAWvqLsNPtO+WCHXg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783659771; c=relaxed/simple; bh=bBq3/syN7TJ3bwHiSJzar76LqfWy2MrbF09Mxs14XPQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=TLZ1J63oMHmEu7QfNc5jN6f18UEwhcMlerE/BCmUH+r6lMYOc8fPDTZLYm2sjzVNiQ4njmldU9nOGm1J+VafgDKcUJXJu+3HPIbm29fp81keGeFNx4cKcws/a2kE5jitRx/x7jjxhEx6FUXQ0vjAeQpV+2H78zM1lg+G0afNIWY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=JlRGHtFa; arc=none smtp.client-ip=209.85.216.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="JlRGHtFa" Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-381891a9525so705015a91.3 for ; Thu, 09 Jul 2026 22:02:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783659770; x=1784264570; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=ycIvlxwe0x12FPCgy021uD9ce3/L5Ep6ZuVevp5ozGk=; b=JlRGHtFaI+edGyWZA1PRSwXX4VVlpf9ztX1FkXIrb7MMDtIw9vwIyI7MFjMEO7FfpL LD7UkDpgBFqElv/4nAU7AFmzz34DWc3TfX+jFP7iOVXWeq6UFYusTFOxyCq1LiLQIM3c Q7ekbxVF7/2qVAEUFpC1Dy7AklNQIc518fLL4INDUCilKtnFxuimI4yBgCUf32jciR98 iIJSk9O2955MSbYB9C8j6MPWsBbBxamggYIHiIB0VdtUAbgNfc1B+gMrc+7DBM558hjd WsF2BpQ8Vrjb/W8MJExjkSHgmhJFyEGDknRZfvXA/Exc3pb3sA0VQ2O/Ij/DLdi0uXDG MXRQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783659770; x=1784264570; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=ycIvlxwe0x12FPCgy021uD9ce3/L5Ep6ZuVevp5ozGk=; b=WYkM6gvdD/OEBcuOSm1KvTGYfOvKQGnhtqm8Q530gCrj8rcmrn+Lh1oXrXX3qVeTb1 4u5otMqGymjopbwevlUlyqrRvfygPx0VzwfepMMbRXRteOPezGeYK39zWn1FzDvzEiF6 JBjQdIZwiwrmaSzmRXrzHC6fzaW/X6/K6qo2dFfWyeHeQS61BOhZYMOLgPhcMiqAe6Jt gEcvffi1tKVDQllM2ccip3oIa8Yx9mT5G6Rg+Sqvui0+amuSzYlnAgEUUFPEcgfhVCRD LJq8wha1SGDoJ4Vy6tlWTbWCZXVr96JKYc7s8GYcqSBHEkf+AsIEdDn7o1ggoiVulguY gAUw== X-Forwarded-Encrypted: i=1; AHgh+RrroXQPJDI4X+34MD3uQrL42ajjDez0ktzghHnXnkqtxdj3sBsZBTBxQbG4PBBfPPoiil3HnniRg26jUjI=@vger.kernel.org X-Gm-Message-State: AOJu0Ywmw6S6SFDD74dLEerXDizLAQ16k4bNzpfvUoyXum8oUR0T1cu0 KsbPuHQ6TngcLEowXTLeOOF7kA93IK8XQ1j7a9CZssuQPUnNtCWIQ/kNMc5JNw== X-Gm-Gg: AfdE7clDjOhvMNXzKrpjan5tq+GVbYTMuk/uFuTF5FP4bS0+GdCfJjObWokrKBsGskn 47/PljmaDY2hCnC3Uu1c5JYSkP5G+wRsGrumEhNt4hNGPF/Oy5qcJwL1o21EZNucl1jAFwAn1YK cUOEGtZzcca4PHE/IRZW5xHzTrEZofIP2xMAwXUqUEQDkCSpdUN6B4zTAdZv7S0iY0//fi3gROb bOWO99mpdfJuOszLsRUlrobZNmeqr7dyKdCSOc6XaRqEEa0C0+1WWeCxrxJ9PvmU/FtLIrp1Hr9 2zp3DqorL6O6X9/IIsD5rnYqDSwYuzZabSyFQVFCCGQp1YYZLrkEnOqthMWnOZ1R3WP7UtTDzZa jGBCeltF36IOxd8bW3cClMahthmHaKj0mDfg4844h9b2fWSleTN4Dv6VpFql5FT7pUep/7GrX7l +KvO05PP/1YoEE88vjO1VYGSx/0+ZfoTy2sOkLL/aztpRJTEs= X-Received: by 2002:a17:90b:3c4a:b0:381:21ea:912b with SMTP id 98e67ed59e1d1-3893f693923mr9181403a91.1.1783659769825; Thu, 09 Jul 2026 22:02:49 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-38a5516ad85sm2139647a91.2.2026.07.09.22.02.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 22:02:49 -0700 (PDT) From: phucduc.bui@gmail.com To: Vincenzo Frascino , Jaroslav Kysela , Takashi Iwai , Liam Girdwood , Mark Brown , Michal Simek Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bui duc phuc Subject: [PATCH 2/2] ASoC: xilinx: xlnx_spdif: Use dev_err_probe() and drop redundant error handling Date: Fri, 10 Jul 2026 12:02:23 +0700 Message-ID: <20260710050223.14266-3-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260710050223.14266-1-phucduc.bui@gmail.com> References: <20260710050223.14266-1-phucduc.bui@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: bui duc phuc Use dev_err_probe() for probe error handling where appropriate to simplify the code and properly handle deferred probe. Also remove redundant error messages when the called helper already reports failures, returning the error directly to avoid duplicate logging. Signed-off-by: bui duc phuc --- sound/soc/xilinx/xlnx_spdif.c | 30 ++++++++++-------------------- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/sound/soc/xilinx/xlnx_spdif.c b/sound/soc/xilinx/xlnx_spdif.c index 017a64ab9f1e..8f4d238be890 100644 --- a/sound/soc/xilinx/xlnx_spdif.c +++ b/sound/soc/xilinx/xlnx_spdif.c @@ -249,21 +249,17 @@ static int xlnx_spdif_probe(struct platform_device *p= dev) return -ENOMEM; =20 ctx->axi_clk =3D devm_clk_get_enabled(dev, "s_axi_aclk"); - if (IS_ERR(ctx->axi_clk)) { - ret =3D PTR_ERR(ctx->axi_clk); - dev_err(dev, "failed to get s_axi_aclk(%d)\n", ret); - return ret; - } + if (IS_ERR(ctx->axi_clk)) + return dev_err_probe(dev, PTR_ERR(ctx->axi_clk), "failed to get s_axi_ac= lk\n"); =20 ctx->base =3D devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(ctx->base)) return PTR_ERR(ctx->base); =20 ret =3D of_property_read_u32(node, "xlnx,spdif-mode", &ctx->mode); - if (ret < 0) { - dev_err(dev, "cannot get SPDIF mode\n"); - return ret; - } + if (ret < 0) + return dev_err_probe(dev, ret, "cannot get SPDIF mode\n"); + if (ctx->mode) { dai_drv =3D &xlnx_spdif_tx_dai; } else { @@ -274,29 +270,23 @@ static int xlnx_spdif_probe(struct platform_device *p= dev) ret =3D devm_request_irq(dev, ret, xlnx_spdifrx_irq_handler, 0, "XLNX_SPDIF_RX", ctx); - if (ret) { - dev_err(dev, "spdif rx irq request failed\n"); - return -ENODEV; - } + if (ret) + return ret; =20 init_waitqueue_head(&ctx->chsts_q); dai_drv =3D &xlnx_spdif_rx_dai; } =20 ret =3D of_property_read_u32(node, "xlnx,aud_clk_i", &ctx->aclk); - if (ret < 0) { - dev_err(dev, "cannot get aud_clk_i value\n"); - return ret; - } + if (ret < 0) + return dev_err_probe(dev, ret, "cannot get aud_clk_i value\n"); =20 dev_set_drvdata(dev, ctx); =20 ret =3D devm_snd_soc_register_component(dev, &xlnx_spdif_component, dai_drv, 1); - if (ret) { - dev_err(dev, "SPDIF component registration failed\n"); + if (ret) return ret; - } =20 writel(XSPDIF_SOFT_RESET_VALUE, ctx->base + XSPDIF_SOFT_RESET_REG); dev_info(dev, "%s DAI registered\n", dai_drv->name); --=20 2.43.0