From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f172.google.com (mail-pg1-f172.google.com [209.85.215.172]) (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 DD7B335E94F for ; Tue, 25 Aug 2026 08:22:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787646131; cv=none; b=J8m8OAPA/Do8+WYCtmTCu/MXZaBJZdDkYOfpiH60ce1brgMdNznJL7M2+oMCiOzIGMV2bHey/xjDz9RWszSfafFra3gyhgCFbbS6T339mgEUvIM3D1OlBE2rB4Zk+3SM3zgZhzA3Vcv2u/80arny1fPXE2icgXQnqbLXsUX39Zw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787646131; c=relaxed/simple; bh=mRJy0c72RRvKN67bg2xlInX3NFWUh2s+PzjaJOfoRxQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=klRwyD8b/GG50GnxxX8iTbgw/TKA5zS0J7c6ZdbiiGSAXBtCYqJHEXkM5pbMys0xAfjPktLSzKHLecfO/iTg8WaJ6Rhu3g2fXIspH+DARl1DtjGut7NnC8BYozAW5TUG0cYDSUAe5MFT2ppZ7GaB2eH2lvPUyraqPL39A+4nNT0= 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=OMcs6I1T; arc=none smtp.client-ip=209.85.215.172 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="OMcs6I1T" Received: by mail-pg1-f172.google.com with SMTP id 41be03b00d2f7-ca766c1c9ccso3066328a12.0 for ; Tue, 25 Aug 2026 01:22:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787646129; x=1788250929; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=fyutcEcBNhVLGX2cE6hT7o6hdT197Wb3v5dZIF2RSuM=; b=OMcs6I1T2mxjnqtMnkfzE6RR0OUw/UJ/2GJkaxoW5DeTY7dIOFFaiVB9IvROphSfYj 0o5KAMRJUXp8mcPQmYCjt/EqrOjoirEcEV/XFtEjgeMsp2ZdB//lDdaJUebD2+v8WDFP DxQ3uX0W6nDopKst+JQvVvCEF8ruTnmAxhcv4iQWvaT+ffgAcQhrLOH6cW60BCev1Y6L DMrDQsNQsezCm5cu8TDQcYa6Q6XPxdDtTDGHMADaX2Lujop/kqqpjIOq0iOzBTLFRFez zqFWxQd3X85/O2nCCYy2xAmvFg7+ohr+NidPcPkfZ22DuhObMgXBy76Z6NMZPKja3IVL UGTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787646129; x=1788250929; h=content-transfer-encoding:mime-version: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=fyutcEcBNhVLGX2cE6hT7o6hdT197Wb3v5dZIF2RSuM=; b=BYKw08pfGe05w9W+sbzMB+oxElUi7r4QnjSoye/bSt6BFhx9hRJafaPRXpT+fbswWR Ra4c80YDx+KOe5EJ2z/vzwlt4CCHOApDjK8JtIZq6Pip9VbpOVf5YdgbH9buJSmWqwbj dYqrujKuxAs0gvvINYYxoqAE7ozjkYgnDPTky/aBYYxAWxNX6/Qn8PQRifceri7iwgLn yrVE12PJ2gKa81lkh8yuOEv+cyhN+GzxqxyzFXrjUDskDMIgbb/2q9SsJyeyBTfkJILM +By5zFt5jMrsxPt6KqsyxtmNhi26YLwiVN5S+uRQd781pCqOmV5tdWCZVsqedFUtLeJH uU4Q== X-Forwarded-Encrypted: i=1; AHgh+RrMRWDxzkhWJnmItcl+xvZBulsF6TdrGPvFzEQ8k6v1TrcNQRYDMAopP/zPBMdTswlZrKptN8lk39JB7Xs=@vger.kernel.org X-Gm-Message-State: AFuF++kMThgoxMhEiMVkS5vyB8MEf42Jh4fpHWWCmzVB9L2AHN9dWNxs GIyPsHfbSUCoyQgLR2RspNScEy3TLMnrN5kdBRDkMt0aIbEvdKh+0g6u X-Gm-Gg: AR+sD12m1tHJcyZaqZM1KPm8ul33xFE+Ehu64SQ/wItyhyNSYTP5X7SfXzUsPM/2ajR k5Wuq0DNawrC6+Nmh49Rv0Qrzhi43Vn2jEbnQpnBffX7LofpVxq804cj56ZoHFvgDZwOkxLGdTq DN1wOmW5oqlC065cPflW15Hqtiz3EwTgWlG36qzdoOgVu5iwSuJwsEFhtrJD1jccCkmxUG/k+e2 PVg+phW3fN4EkdWoIBRqbtV47omWlwhVRoOmY1WGlO05AmnJHdty1ZufEzfSK/1NGIND/3XpLEL GK95pUJDpynwjQvNRHZsk6gL+ZTHZAKhhwK0HcuuqvOfdQxv8MD6b7BSTvCItTg6z8p3JHIo0oI KgXtCJwNazMV7pQ6f817W7dEXK/xpPJrQ51b3LGlyleE7ISHwRRtl2yOjrEj3vVCkmYt95tbcf0 BVIAVJxUPI30NH4wSWSj7thakoFw6Fyb5NVQyj9ynzmsqBS5TaLRFKkBLRacYmWGt3yzBmr4lOR nlbo1YipLRN9d8DzmQ81CyflNUNHagmwuRdzFdseSzQzqnan050cp0ms2oRKHbiz4mbR0REtAGJ rLdh X-Received: by 2002:a05:6a00:2383:b0:835:6388:655d with SMTP id d2e1a72fcca58-8520ba30a62mr39852008b3a.14.1787646128973; Tue, 25 Aug 2026 01:22:08 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-8520f151c7esm2749509b3a.48.2026.08.25.01.22.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2026 01:22:08 -0700 (PDT) From: phucduc.bui@gmail.com To: Lars-Peter Clausen , Liam Girdwood , Mark Brown Cc: Jaroslav Kysela , Takashi Iwai , Sebastian Reichel , linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH] ASoC: soc-generic-dmaengine: Fix DMA channel request warning Date: Tue, 25 Aug 2026 15:19:49 +0700 Message-ID: <20260825081949.55537-1-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 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 The DMA channel request code currently warns about legacy DMA failures when the channel name is not present in dma-names. This can report a firmware lookup failure as a legacy DMA failure. Furthermore, failures from the legacy DMA path are already reported by find_candidate(), making these warnings redundant. Only warn when the channel name is present in dma-names but the request fails, avoiding misleading and duplicate error messages. Fixes: 9167f260477b ("ASoC: soc-generic-dmaengine: Handle DMA channel request failures correctly") Reported-by: Sebastian Reichel Link: https://lore.kernel.org/all/aoyBuho270dTWYBL@jupiter.universe/ Signed-off-by: bui duc phuc --- sound/soc/soc-generic-dmaengine-pcm.c | 20 ++------------------ 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/sound/soc/soc-generic-dmaengine-pcm.c b/sound/soc/soc-generic-dmaengine-pcm.c index 3b18d90e81c3..65b7ec52a054 100644 --- a/sound/soc/soc-generic-dmaengine-pcm.c +++ b/sound/soc/soc-generic-dmaengine-pcm.c @@ -397,24 +397,8 @@ static int dmaengine_pcm_request_chan_of(struct dmaengine_pcm *pcm, if (PTR_ERR(chan) == -EPROBE_DEFER) return -EPROBE_DEFER; - bool has_fw_node = dev->of_node || is_acpi_device_node(dev->fwnode); - bool name_exists_in_fw = false; - - if (has_fw_node) - name_exists_in_fw = device_property_match_string(dev, - "dma-names", - name) >= 0; - - if (has_fw_node && name_exists_in_fw) - dev_warn(dev, "DTS/ACPI DMA channel '%s' request failed (%ld)\n", - name, PTR_ERR(chan)); - - if (has_fw_node && !name_exists_in_fw) - dev_warn(dev, "DTS/ACPI name '%s' not found, legacy failed (%ld)\n", - name, PTR_ERR(chan)); - - if (!has_fw_node) - dev_warn(dev, "Legacy DMA channel '%s' request failed (%ld)\n", + if (device_property_match_string(dev, "dma-names", name) >= 0) + dev_warn(dev, "dma-names has '%s' but request failed (%ld)\n", name, PTR_ERR(chan)); pcm->chan[i] = NULL; -- 2.43.0