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 5BC7D407CDE for ; Fri, 10 Jul 2026 10:22:06 +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=1783678927; cv=none; b=YN+EUcxkwDGuBYLY67DVQxfZi81ujDSUCfYoIZBvHSl1fSeGLv1tKIXnFhantZjdWRipw4ON6kD73eB8ANMXqYaZ15uxw/k6yj07h5F0t3xRUyS34inGNg5wyl7V+NKVJREpi7NjdOfzDv+p6EulmWT55ND5bG1RNSZ440trahI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783678927; c=relaxed/simple; bh=1FZVoSjzKMyVRnA+73pI8+UhFeytlac2ubKrBiaTzGQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=nCQMZPWQb02444Lshd+CVBTeQiOJrol7tmlCx8zji6ZI/hnw1Rzs6GJlZaBJ7HNsq62edAshykiLqZxBZ+PxBkiL0cN8YYEFXkpd5qXv3LNGKcsMh1ALOTB0ksfmfTFkV7XXD6IV1M/omWyPQ374Ukg0eXzE0ovrWCjXLcVfG6A= 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=XPpil9Cs; 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="XPpil9Cs" Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-37e0a189b0bso737566a91.1 for ; Fri, 10 Jul 2026 03:22:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783678926; x=1784283726; 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=4RFOqWcsEwd6ePMQdZZ8QPUfPlo/GmYQwqUASim0itw=; b=XPpil9Csi1AwGks8HRKtC+TgroO0KPOlEAZWyrrzaO3oF2LRHvFnErRiR3iAy90o72 b3QAwLBgBwOj9uocxs3bzDJ3skZLZPF5KTmbXbGeQj1P7yuaJ6H9BIbVK+OojaU6yLL7 AbZ84dAC/f2ACv8RNpBEyqFve6S7sOC+iKBSznbqqfCrQ37wn2Ny0p8o7gr5WeSL/1mT xb+TOqJDntXLID29dmT9rinccz7+UzfURibFKSqyS1MKKdNs/ZCcb6jqg/+CXGYUkmBH oUN9uM7WxvqNG0jkhTVWWr+etR0L0GeqhxH3aXQBH2oDjMuK61feIBwWeTdGb2EN+6Pc CBrQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783678926; x=1784283726; 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=4RFOqWcsEwd6ePMQdZZ8QPUfPlo/GmYQwqUASim0itw=; b=gem13IhMo8mQa5SChXoibY7ecylE/OHRbo7TGuOYXBAjuMD2e8geVEoObXJwnU3wUn yvVRQRj3NSEsl0ND0MBe4QizA0eKKjmWIngNXkCExWtqnkpxN+UzSG8uCtYLZed0p71e 2FayQlshAc+JgUoXfbFTReY4uwEy4GFfT+NxWKxvxgHu6C8HEtXxwf1lUD3DR3ol8qF7 7Dpj1CxcY/15V971b+Ekfzhjq1JvVFrRHSpKLSlGSgI19Xz0bkRYl+oiDil0698mb9Dn 9X7ymI3An2MflieIj/Rk4b8wLw2Og510KN8VHbeAWlS/4B2BSy3rJx4NuSbt4gONEiGU bGZg== X-Forwarded-Encrypted: i=1; AHgh+RqyhtG7O8qct2nzEow9wF5S3bMvAGumUueaSvqut1z4M8/nNZ1p5cmBli8vwTtF3X8B2yLXt+TjIBijBFo=@vger.kernel.org X-Gm-Message-State: AOJu0YzKNQFdGzr5mCptn1Zki3YzK0AY3aDmmoAkvnQ8GQvq01Qzesdi WOr8TQDl3ju1prLM81IH/C3UcM7qqrMm7sMxe/IOw7NmbraQfpisBxpa X-Gm-Gg: AfdE7cn1QJzeHHvIcgIb7ZCzcBmducYhJ1GJGW6szGUzUZn2PXIgQr6jpiPSHiqIJiS vTIf/ikfUXLT9yZWG9I8k4egB2duQV5hJsrLI8UQHEXH/oc60QBSeKV+pmg0quSzTBfLFI8Hywb beZF8Sk1xATYTr2bAihPU62XL3KiScPdhIFstBLVPLFCnoWVWmr2+ut6Sfw+Qmy73ANFuz6hoMj vEtE/mifafixoyfqdG2kX3nsbGunoeTWukKgIjis/6TOEpEfkPKJhV/rC/jTfUq/PDK8Haq8cnf OZFdiYvxXDAA8ATO0sXvCSSoeNdm0aKtX6jOswFacWf2yDH1Xnkgw4cekCBaPyNADi6Bndv1Tr7 ueqfpCtdIo/82SU3d+2bN405jQ/M+1ueqlit/H/cqMAmgh9WjEzwrMkBsI7eSiwlzvWBN/L7YkP FXMZ7hdRaLruPP7uhFhZWbl9d0CBr/EEgTtRQW9L9HwbbBXS8= X-Received: by 2002:a17:90b:2586:b0:387:e0db:bc30 with SMTP id 98e67ed59e1d1-38d6d1eb86cmr1754316a91.38.1783678925820; Fri, 10 Jul 2026 03:22:05 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-38a5516abffsm2490240a91.4.2026.07.10.03.22.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 10 Jul 2026 03:22:05 -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 v2 2/3] ASoC: xilinx: xlnx_spdif: Preserve devm_request_irq() error codes Date: Fri, 10 Jul 2026 17:21:37 +0700 Message-ID: <20260710102138.29347-3-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260710102138.29347-1-phucduc.bui@gmail.com> References: <20260710102138.29347-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 devm_request_irq() can return various error codes, such as -EINVAL, -ENOTCONN, -ENOMEM, -ENOSYS, and -EBUSY. However, the driver overwrites all of them with -ENODEV, which hides the actual cause of the failure. Also, devm_request_irq() already reports failures internally, so the additional dev_err() call is redundant. Return the original error code and remove the duplicate error message. Signed-off-by: bui duc phuc --- sound/soc/xilinx/xlnx_spdif.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sound/soc/xilinx/xlnx_spdif.c b/sound/soc/xilinx/xlnx_spdif.c index 017a64ab9f1e..deb7225c1b4b 100644 --- a/sound/soc/xilinx/xlnx_spdif.c +++ b/sound/soc/xilinx/xlnx_spdif.c @@ -274,10 +274,8 @@ 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; -- 2.43.0