From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B53C0379987; Sun, 26 Jul 2026 19:59:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785095995; cv=none; b=sHAQ7y+SAyxHytSLUshKbV7Kcj2B6yftMZsUmGDR/KA8ZOqWyi++qhR2RKGCYkN9AR4DtqRgm6VorWc5ua37B4o56MXIPk267pAc00tyqK0t8HZgO4Bp7cVyqAvKVVBFjcqLof6tgUDMrTmz+vFv3ekOqZqrMDMlty/GDd/97ew= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785095995; c=relaxed/simple; bh=hUZ5ps5AQ1e1xGcqJrHC0K7BmjIos5zDwd6o7fmPPlg=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=RfCabgM/5FEaPwD28keP0mCxj/Ibaj3SzBB1FT4bM5BxWod2Ozp9VQoQUHGUuKZoW+jbBpkft/Bl9lvoUZ5fouCev+3W9IWjVPqXUzSlVtiHMryJ8jGe42mGVqG1yiVxjExZhUDlcKmA8GWblEq9N3PwH/KB8RwhhzV6y7h0wK4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=bqpHW46m; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="bqpHW46m" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 84B531F000E9; Sun, 26 Jul 2026 19:59:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1785095993; bh=FLy+HtY0DJaJQCklCIN73S4dGy4dcZGkDMsgmHLYuH0=; h=Date:From:To:Cc:Subject:In-Reply-To:References; b=bqpHW46m6PRsQ9e/fAc/X2YDO8Ta6fGVDJH2aXH0P7UWMXEwRyRR050ohCF2DxuBR Ys1JV9ACONuenvvQ5mtS60/vyFG+UjhghkDhkEMFVeRwaRNBE62l2D2bnX05IBsQpG KX4uedObZFf7EE8KE17mKGdJfEJsVLMfj40xp33b3FmtuYPhkvUsC3bwsjn/9yFqVR 2o6ivqq5CJ998mNaJfn90CgLyd5+NHTZO90tePgsS58rxW1NWQv6TM2xS7/a1ZtkGO ZxpGiiUBcJfdaEmKQihgDX8uaEE0c0dr0ij9kzkY7ir6kDyA5gaRyNaYaziBtfbnHv vCYbXtNjgeL5w== Date: Sun, 26 Jul 2026 20:59:41 +0100 From: Jonathan Cameron To: Joshua Crofts Cc: Pan Chuang , Puranjay Mohan , Nuno =?UTF-8?B?U8Oh?= , Michael Hennerich , David Lechner , Andy Shevchenko , Marcelo Schmitt , Petre Rodan , Dan Robertson , Matti Vaittinen , Gustavo Silva , Maxwell Doose , Tomasz Duszynski , Nishant Malpani , Javier Carrasco , Li peiyu <579lpy@gmail.com>, Lorenzo Bianconi , Alex Lanzano , Jagath Jog J , Mikael Gonella-Bolduc , Subhajit Ghosh , Mudit Sharma , Kevin Tsai , Linus Walleij , Anshul Dalal , Song Qiang , Andreas Klinger , "Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= (The Capable Hub)" , Siratul Islam , Sebastian Andrzej Siewior , Bjorn Helgaas , Danilo Krummrich , Takashi Sakamoto , Xichao Zhao , Waqar Hameed , Sakari Ailus , Marcus Folkesson , Miao Li , Dixit Parmar , Andrew Davis , Kurt Borja , Shrikant Raskar , Chu Guangqing , Antoniu Miclaus , Srinivas Pandruvada , "Rafael G. Dias" , Svyatoslav Ryhel , Shi Hao , Colin Ian King , Salah Triki , Yury Norov , Greg Kroah-Hartman , Yash Suthar , Achim Gratz , Yasin Lee , linux-iio@vger.kernel.org (open list:ADXL355 THREE-AXIS DIGITAL ACCELEROMETER DRIVER), linux@analog.com (open list:ANALOG DEVICES INC IIO DRIVERS), linux-kernel@vger.kernel.org (open list) Subject: Re: [PATCH v2 00/13] iio: Remove redundant error messages on IRQ request failure Message-ID: <20260726205941.40f6eade@jic23-huawei> In-Reply-To: <20260724122701.0000156f@gmail.com> References: <20260724030524.607185-1-panchuang@vivo.com> <20260724122701.0000156f@gmail.com> X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 24 Jul 2026 12:27:01 +0200 Joshua Crofts wrote: > On Fri, 24 Jul 2026 11:03:35 +0800 > Pan Chuang wrote: > > > Commit 55b48e23f5c4 ("genirq/devres: Add error handling in devm_request_*_irq()") > > added automatic error logging to devm_request_threaded_irq() and > > devm_request_any_context_irq() via the new devm_request_result() helper, > > which prints device name, IRQ number, handler functions, and error code > > on failure. > > > > Since devm_request_irq() is a static inline wrapper around > > devm_request_threaded_irq(), it also benefits from this > > automatic logging. > > > > Remove the now-redundant dev_err() and dev_err_probe() calls > > in iio drivers that follow these devm_request_*_irq() > > functions, as the core now provides more detailed diagnostic > > information on failure. > > > > v2: > > - Rewrap commit description to one line. > > - Drop redundant "!= 0" and "< 0" comparisons. > > > > Okay, code-wise it's fine, but rewrap your commit messages to 72 > characters per line. That would involve breaking the commit reference across lines. I asked that the author not do that on the previous version because it'll make it harder to search for if anyone ever wants to do so. Also submitting patches suggests 75 chars. I'm not sure where the 72 comes from, though it is often quoted and another bit of submitting patches does say 70 to 75. > Feel free to apply my tag for the whole series: > > Reviewed-by: Joshua Crofts > > Also, I was scratching my head at how enormous the To: list is, given > these changes are only in IIO, but I assume you also added emails of > the people who authored the latest changes on a file that get_maintainer > outputs - there is no need to do this. Tools like b4 can automatically > add (correct) recipients based on get_maintainer.pl >