From mboxrd@z Thu Jan 1 00:00:00 1970 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: 8bit 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 *pdev) return -ENOMEM; ctx->axi_clk = devm_clk_get_enabled(dev, "s_axi_aclk"); - if (IS_ERR(ctx->axi_clk)) { - ret = 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_aclk\n"); ctx->base = devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(ctx->base)) return PTR_ERR(ctx->base); ret = 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 = &xlnx_spdif_tx_dai; } else { @@ -274,29 +270,23 @@ static int xlnx_spdif_probe(struct platform_device *pdev) ret = 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; init_waitqueue_head(&ctx->chsts_q); dai_drv = &xlnx_spdif_rx_dai; } ret = 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"); dev_set_drvdata(dev, ctx); ret = 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; - } writel(XSPDIF_SOFT_RESET_VALUE, ctx->base + XSPDIF_SOFT_RESET_REG); dev_info(dev, "%s DAI registered\n", dai_drv->name); -- 2.43.0