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.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS 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 1BC9AC433E0 for ; Fri, 1 Jan 2021 01:56:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id CB4BA20735 for ; Fri, 1 Jan 2021 01:56:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726984AbhAABzo (ORCPT ); Thu, 31 Dec 2020 20:55:44 -0500 Received: from smtprelay0239.hostedemail.com ([216.40.44.239]:49046 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726485AbhAABzo (ORCPT ); Thu, 31 Dec 2020 20:55:44 -0500 Received: from filter.hostedemail.com (clb03-v110.bra.tucows.net [216.40.38.60]) by smtprelay01.hostedemail.com (Postfix) with ESMTP id 12236100E7B40; Fri, 1 Jan 2021 01:55:03 +0000 (UTC) X-Session-Marker: 6A6F6540706572636865732E636F6D X-HE-Tag: north20_20090f4274b2 X-Filterd-Recvd-Size: 2313 Received: from [192.168.1.159] (unknown [47.151.137.21]) (Authenticated sender: joe@perches.com) by omf03.hostedemail.com (Postfix) with ESMTPA; Fri, 1 Jan 2021 01:55:02 +0000 (UTC) Message-ID: <4abaac7b4b792dd44880d8acf6ef51d9db89e512.camel@perches.com> Subject: Re: checkpatch.pl: Bogus case of DT_SPLIT_BINDING_PATCH From: Joe Perches To: Jonathan =?ISO-8859-1?Q?Neusch=E4fer?= , Andy Whitcroft Cc: Rob Herring , linux-kernel@vger.kernel.org Date: Thu, 31 Dec 2020 17:55:00 -0800 In-Reply-To: References: Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.38.1-1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2020-12-31 at 23:04 +0100, Jonathan Neuschäfer wrote: > Hi, > > I've encountered a case where the DT_SPLIT_BINDING_PATCH warning was > emitted even though I didn't change anything outside of Documentation/ > devicetree/bindings. I just converted a binding from plain text to YAML. Rob? Your code... > Here's a transcript of checkpatch (from Linux 5.11-rc1)'s output: > >   $ scripts/checkpatch.pl --strict patches-wpcm/0001-dt-bindings-arm-Convert-nuvoton-npcm750-binding-to-Y.patch >   WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? >   #31: >   deleted file mode 100644 >    > >   WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst >    > >   WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst >    > >   total: 0 errors, 3 warnings, 0 checks, 21 lines checked >    > >   NOTE: For some of the reported defects, checkpatch may be able to >         mechanically convert to the typical style using --fix or --fix-inplace. >    > >   patches-wpcm/0001-dt-bindings-arm-Convert-nuvoton-npcm750-binding-to-Y.patch has style problems, please review. >    > >   NOTE: If any of the errors are false positives, please report >         them to the maintainer, see CHECKPATCH in MAINTAINERS. > > > I attached the patch, for reference. > > > Best regards, > Jonathan Neuschäfer