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 X-Spam-Level: X-Spam-Status: No, score=-3.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2041C433B4 for ; Mon, 19 Apr 2021 11:02:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D539A61246 for ; Mon, 19 Apr 2021 11:02:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238299AbhDSLCt (ORCPT ); Mon, 19 Apr 2021 07:02:49 -0400 Received: from smtprelay0118.hostedemail.com ([216.40.44.118]:35932 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S237833AbhDSLCm (ORCPT ); Mon, 19 Apr 2021 07:02:42 -0400 Received: from omf11.hostedemail.com (clb03-v110.bra.tucows.net [216.40.38.60]) by smtprelay06.hostedemail.com (Postfix) with ESMTP id 17BD018020DA3; Mon, 19 Apr 2021 11:02:12 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf11.hostedemail.com (Postfix) with ESMTPA id 2016120A295; Mon, 19 Apr 2021 11:02:10 +0000 (UTC) Message-ID: <8cddfac598fb0ef3c10d583fc70a5ebf1c8aeddc.camel@perches.com> Subject: Re: [PATCH v5] printk: Userspace format enumeration support From: Joe Perches To: Greg Kroah-Hartman , Petr Mladek Cc: Rasmus Villemoes , Chris Down , linux-kernel@vger.kernel.org, Sergey Senozhatsky , John Ogness , Johannes Weiner , Andrew Morton , Steven Rostedt , Kees Cook , kernel-team@fb.com Date: Mon, 19 Apr 2021 04:02:08 -0700 In-Reply-To: References: <02c3b2f3-ff8e-ceb9-b30b-e533959c0491@rasmusvillemoes.dk> <5ea3b634-5467-35cf-dd08-1001f878b569@rasmusvillemoes.dk> Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.38.1-1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspamout02 X-Stat-Signature: f8tx41ti33ixk61gc7rgse56fbagbfns X-Rspamd-Queue-Id: 2016120A295 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX1+ecAu86kQ+6gE4zOHZP4JWVjjxPmJUBcc= X-HE-Tag: 1618830130-112912 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2021-04-19 at 11:53 +0200, Greg Kroah-Hartman wrote: > Hm, 12734 of the pr_err() calls do live in drivers/, so most of those > should be dev_err(). Might be something good to throw at interns... That depends on how much churn you want to have in old drivers that generally don't have any users because the hardware is ancient or no longer manufactured. I suggest not changing those. But I believe a coccinelle script was written quite awhile ago to convert pr_ to dev_ when a struct device * is available. http://btrlinux.inria.fr/staging-media-replace-pr_-with-dev_/