From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on archive.lwn.net X-Spam-Level: X-Spam-Status: No, score=-5.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by archive.lwn.net (Postfix) with ESMTP id 51EAA7D918 for ; Thu, 24 Oct 2019 18:47:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2440154AbfJXSrh (ORCPT ); Thu, 24 Oct 2019 14:47:37 -0400 Received: from mail.kernel.org ([198.145.29.99]:38652 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2440168AbfJXSrh (ORCPT ); Thu, 24 Oct 2019 14:47:37 -0400 Received: from localhost (unknown [75.104.69.146]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4B01B20659; Thu, 24 Oct 2019 18:47:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1571942856; bh=P2j4bA86XgvOqRFtfCl5MtPYc2xM05N5w3nh3XKCbOU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JXFsM9yewwErnF5S3LK+7K6bicNse77w5ov+K8sMXE1WbXMnNUxJ53lRpClS1Y1CC 11fXyyi93K6RL+XYzVbm018cMUVbdftXTSBMyZqzEhGJR0OX5ENyHkk/LV+cr97wvX JQuzhFZ9aaA31ctCt2wdqqskCAv9tdJU2DNgNRus= Date: Thu, 24 Oct 2019 14:47:28 -0400 From: Greg Kroah-Hartman To: Mark Salyzyn Cc: Jarkko Sakkinen , linux-kernel@vger.kernel.org, kernel-team@android.com, "David S. Miller" , Jonathan Corbet , Ard Biesheuvel , Harry Wentland , Leo Li , Alex Deucher , Christian =?iso-8859-1?Q?K=F6nig?= , "David (ChunMing) Zhou" , David Airlie , Daniel Vetter , VMware Graphics , Thomas Hellstrom , Laurent Pinchart , Mauro Carvalho Chehab , Trond Myklebust , Anna Schumaker , Alexander Aring , Jukka Rissanen , Alexey Kuznetsov , Hideaki YOSHIFUJI , Ingo Molnar , Matthew Garrett , Hans de Goede , hersen wu , Roman Li , Maxim Martynov , David Ahern , Francesco Ruggeri , Linus =?iso-8859-1?Q?L=FCssing?= , Feng Tang , "Steven Rostedt (VMware)" , Andrew Morton , Rafael Aquini , netdev@vger.kernel.org, linux-doc@vger.kernel.org, linux-efi@vger.kernel.org, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, linux-nfs@vger.kernel.org, linux-bluetooth@vger.kernel.org, linux-wpan@vger.kernel.org Subject: Re: [PATCH] Cleanup: replace prefered with preferred Message-ID: <20191024184728.GC260560@kroah.com> References: <20191022214208.211448-1-salyzyn@android.com> <20191023115637.GA23733@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.2 (2019-09-21) Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org On Wed, Oct 23, 2019 at 08:40:59AM -0700, Mark Salyzyn wrote: > On 10/23/19 4:56 AM, Jarkko Sakkinen wrote: > > On Tue, Oct 22, 2019 at 02:41:45PM -0700, Mark Salyzyn wrote: > > > Replace all occurrences of prefered with preferred to make future > > > checkpatch.pl's happy. A few places the incorrect spelling is > > > matched with the correct spelling to preserve existing user space API. > > > > > > Signed-off-by: Mark Salyzyn > > I'd fix such things when the code is otherwise change and scope this > > patch only to Documentation/. There is no pragmatic benefit of doing > > this for the code. > > > > /Jarkko > > The pragmatic benefit comes with the use of an ABI/API checker (which is a > 'distro' thing, not a top of tree kernel thing) produces its map which is > typically required to be co-located in the same tree as the kernel > repository. Quite a few ABI/API update checkins result in a checkpatch.pl > complaint about the misspelled elements being (re-)recorded due to > proximity. We have a separate task to improve how it is tracked in Android > to reduce milepost marker changes that result in sweeping changes to the > database which would reduce the occurrences. Requiring checkpatch spelling warnings to be correct based on function names is crazy, you should fix your tools if you are requiring something as looney as that :) > I will split this between pure and inert documentation/comments for now, > with a followup later for the code portion which understandably is more > controversial. Please break up per subsystem, like all trivial patches, as this isn't anything special. thanks, greg k-h