From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH 0/2] Remove support for deprecated %pf and %pF in vsprintf Date: Fri, 22 Mar 2019 19:05:50 +0200 Message-ID: <20190322170550.GX9224@smile.fi.intel.com> References: <20190322132108.25501-1-sakari.ailus@linux.intel.com> <20190322135350.2btpno7vspvewxvk@paasikivi.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190322135350.2btpno7vspvewxvk@paasikivi.fi.intel.com> Sender: linux-kernel-owner@vger.kernel.org To: Sakari Ailus Cc: Geert Uytterhoeven , Petr Mladek , Linux Kernel Mailing List , scsi , Linux PM list , Linux MMC List , "linux-nvdimm@lists.01.org" , linux-um@lists.infradead.org, linux-f2fs-devel@lists.sourceforge.net, linux-block@vger.kernel.org, ACPI Devel Maling List , netdev , linux-btrfs , linux-pci , sparclinux , xen-devel@lists.xenproject.org, ceph-devel , Linux MM , Linux ARM List-Id: linux-pm@vger.kernel.org On Fri, Mar 22, 2019 at 03:53:50PM +0200, Sakari Ailus wrote: > Porting a patch > forward should have no issues either as checkpatch.pl has been complaining > of the use of %pf and %pF for a while now. And that's exactly the reason why I think instead of removing warning on checkpatch, it makes sense to convert to an error for a while. People are tending read documentation on internet and thus might have outdated one. And yes, the compiler doesn't tell a thing about it. P.S. Though, if majority of people will tell that I'm wrong, then it's okay to remove. -- With Best Regards, Andy Shevchenko