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 DD7F1CE7CE7 for ; Tue, 1 Oct 2024 09:08:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References: Subject:Cc:To:From:Message-Id:Date:Content-Type:Content-Transfer-Encoding: Mime-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=4Zdv10SJzsReJ9cdAvKhNMK+4CSxsDOH1wiW7DwcRJU=; b=bhRFJcaQSudvJ7f5lZ6EzKgZTI 2VLem8kRpxQXc3+m9SxygpZzev2ypvVZ7e810j+ZsXNpyxtVhD8b6Jqp5D91SeOPdzMOffiTDGxGa rJEYQPWilJ655ZKUKd71kLAUbozfRW6CZc3GY0a+9fLV3EOSunfs1EotuPivMFNtx05q3mNHwdN/j hTVLyNPGDROdBYDqggnELZ2ZeKHaEQTAZszSrecUbbYmaMUBUX0pSZIS99p3Yf7S9wiQ/oxvoL8zR PDuoa/YizrFFnitxUCMcm+5ne6YtZ463T/FO/Vf+fnVStcxHHQWnzsuF4DpMduEuIrGsrCuQbT/MA Q5L4PZbQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1svYra-00000002BVS-39XG; Tue, 01 Oct 2024 09:08:10 +0000 Received: from out-187.mta1.migadu.com ([2001:41d0:203:375::bb]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1svYqM-00000002BDO-1aja for linux-arm-kernel@lists.infradead.org; Tue, 01 Oct 2024 09:06:55 +0000 Mime-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=helene.moe; s=key1; t=1727773612; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=4Zdv10SJzsReJ9cdAvKhNMK+4CSxsDOH1wiW7DwcRJU=; b=O6MiPbSL6N+4wvOtq6fDYRD5nneMzkq5R3ODanXsqIkxfje7mW1umGyEwwqXSzaSPW9ahp Ui9KT57n3axkifB34GHdZ0xQtIppAE/fdhT9+tEqQNBNwzXYW7JYqSBAggSRcoTGoWcxVV THchCpw4/K5KKKW2A6iHpB0l3MAZxV9D+Hytc17WHRIcFemIhsCwJelHHu1KCWV2seyMI3 7NKiPjrrBM8tMFKvAYpWiDLY7o/sp56oFm54n5O3PjgF7IFr7SdQu4P21mKsIBKqHoiw1O zbn3ISnh/iGJ9arOSbHivu4elN4NbcVbUbFKsAgjcon6l4JFg/iGM0PJyKByaA== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 01 Oct 2024 11:06:49 +0200 Message-Id: X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: =?utf-8?q?H=C3=A9l=C3=A8ne_Vulquin?= To: "Dragan Simic" , , Cc: , , , , , Subject: Re: [PATCH v3 5/5] spi: rockchip: Use dev_{err,warn}_probe() in the probe path References: <5b6bd142dab3ab93d7039db3e2fdcfea6bee2217.1727601608.git.dsimic@manjaro.org> In-Reply-To: <5b6bd142dab3ab93d7039db3e2fdcfea6bee2217.1727601608.git.dsimic@manjaro.org> X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241001_020654_561836_8473B4B2 X-CRM114-Status: GOOD ( 10.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun Sep 29, 2024 at 11:21 AM CEST, Dragan Simic wrote: > Use function dev_err_probe() in the probe path instead of dev_err() where > appropriate, to make the code a bit more uniform and compact. Use the ne= w > function dev_warn_probe() to improve error handling for the TX and RX DMA > channel requests, which are actually optional, and tweak the logged warni= ngs > a bit to additionally describe their optional nature. > > Previously, deferred requests for the TX and RX DMA channels produced no > debug messages, and the final error messages didn't include the error cod= es, > which are all highly useful when debugging permanently failed DMA channel > requests, such as when the required drivers aren't enabled. > > Suggested-by: H=C3=A9lene Vulquin > Signed-off-by: Dragan Simic I can now see appropriate messages for pending probes on Rockchip SPI devices, instead of "(reason unknown)", which is incredibly helpful for debugging DeviceTrees and kernel configs. Tested on 6.11 and looks like it works as intended. Thank you for this patch! Tested-by: H=C3=A9l=C3=A8ne Vulquin