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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 69838C4167B for ; Fri, 1 Dec 2023 22:30:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229642AbjLAW35 convert rfc822-to-8bit (ORCPT ); Fri, 1 Dec 2023 17:29:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36524 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229456AbjLAW3z (ORCPT ); Fri, 1 Dec 2023 17:29:55 -0500 Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1C58CB2 for ; Fri, 1 Dec 2023 14:30:02 -0800 (PST) Received: from omf15.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 99887C0353; Fri, 1 Dec 2023 22:30:00 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf15.hostedemail.com (Postfix) with ESMTPA id 3232618; Fri, 1 Dec 2023 22:29:58 +0000 (UTC) Message-ID: <8d846c564461af99c27d81a69888d73f3622de8b.camel@perches.com> Subject: Re: [PATCH v1 1/1] checkpatch: Add dev_err_probe() to the list of Log Functions From: Joe Perches To: Andy Shevchenko , Guenter Roeck Cc: Andrew Morton , linux-kernel@vger.kernel.org, Andy Whitcroft , Dwaipayan Ray , Lukas Bulwahn Date: Fri, 01 Dec 2023 14:29:57 -0800 In-Reply-To: References: <20231201151446.1593472-1-andriy.shevchenko@linux.intel.com> <3c709cc1-0da8-4d23-9f75-8c18d4d18779@roeck-us.net> <8fff7625-69b7-4c32-a3bb-d3ca24f149bf@roeck-us.net> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 8BIT User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) MIME-Version: 1.0 X-Stat-Signature: cxkb6dt39n4ho47e1synzehy4yn6or5w X-Rspamd-Server: rspamout03 X-Rspamd-Queue-Id: 3232618 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX190xDXCP3i7iNh3gaKN1Esa/zfj00BlQ6c= X-HE-Tag: 1701469798-838526 X-HE-Meta: U2FsdGVkX18tO43hiZol7B+VzlMrlwOmQNyIq3vBQzI2fDWyZ+qN7bp62t+yPOgaZVEudqvFKwPOu+47vFhwwOtGwAUtStJ5+uPr8qnGudJ9LVMf0448VHDlFzbb2O5jxNsDmy/ElzVue4SLgpsE4CxGRM1N8OEksVuI7fzMdHGFfyungRV8kac3/y3UqFvj70GJNMwZu6LN33i6O/vXVicr5U6JvkqwsSvH3enH1omzXO7K/aiRklxvq3eHeJADFibynteUp/Uq0F1eqRrdGcqXmhbef+3ziIOAqa+3UvIyYYcM0PJcdFmz5h9DwTWH Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2023-12-01 at 19:07 +0200, Andy Shevchenko wrote: > Currently the [style] behaviour is > inconsistent independently on somebody's preferences... Which is IMO perfectly fine. The ratio of multiple line and single line uses of dev_err_probe is ~50:50 $ git grep -w dev_err_probe | grep -P ';\s*$' | wc -l 1532 $ git grep -w dev_err_probe | grep -P ',\s*$' | wc -l 1871