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=-7.0 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,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 BCB03C433EB for ; Thu, 13 Aug 2020 07:51:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A28472078B for ; Thu, 13 Aug 2020 07:51:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726816AbgHMHvK (ORCPT ); Thu, 13 Aug 2020 03:51:10 -0400 Received: from mail-ot1-f65.google.com ([209.85.210.65]:38760 "EHLO mail-ot1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726106AbgHMHvJ (ORCPT ); Thu, 13 Aug 2020 03:51:09 -0400 Received: by mail-ot1-f65.google.com with SMTP id q9so4110499oth.5; Thu, 13 Aug 2020 00:51:08 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=eU62SrftExeZEPFtSlHLsBRzbUkLx1KNOgZc6VKFeBQ=; b=RKVY0mqqakUrEXk4TORV8A5/ZSMY0dexhKfXyaOKBp8tYT6+lN9tMY1gCu59dMSnI/ 6Av5sXCLtCtfska6KBUfoUvPLN8QSlHGcwX+BKv8ZNWhNtirhMGElzBnWp56J7vtBCeA RTsTm+tpM0JqO6hs1Zx9oL0KiDPnHu2wr8nXIFBaGz9jqMUEyZCYMyj21d8ckpZmX8IH sIyh51oPoVvqSR3qDoGMDZJhbXuz4s9KHdaT9j4oPeIaqxws0OL9u53GcL8uZhE7hrov ikjJM0LMxR/OG4BC9tZAySXGXWwQpPbcG9j3oyJJCCrBhecUJhWEIB5ZPymwSBlP+imO EXuw== X-Gm-Message-State: AOAM531KpbVTNZA6DQGhlLakjE36ZGk11NQSWXrXhgb2m/BrbDAUDZAN GTNNlK5/qceLxjH+paJhJQLuqYhFPre+GsLWPoc= X-Google-Smtp-Source: ABdhPJzAAKS8g3t/W5mSJd2Vboc2FEvoNvVXoru7u4kNP7jabPCwVCki2JjTk/fEfMzVv8QAFC24k07bLAU2LYhSK+k= X-Received: by 2002:a05:6830:1b79:: with SMTP id d25mr2990956ote.107.1597305067462; Thu, 13 Aug 2020 00:51:07 -0700 (PDT) MIME-Version: 1.0 References: <20200812203618.2656699-1-robh@kernel.org> In-Reply-To: <20200812203618.2656699-1-robh@kernel.org> From: Geert Uytterhoeven Date: Thu, 13 Aug 2020 09:50:55 +0200 Message-ID: Subject: Re: [PATCH] dt-bindings: Whitespace clean-ups in schema files To: Rob Herring Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-hwmon@vger.kernel.org, linux-rtc@vger.kernel.org, Linux Fbdev development list , ALSA Development Mailing List , Linux PM list , linux-iio@vger.kernel.org, Linux MMC List , USB list , "open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM" , Linux Kernel Mailing List , DRI Development , linux-spi , "open list:GPIO SUBSYSTEM" , netdev , MTD Maling List , Linux I2C , "open list:SERIAL DRIVERS" , linux-input@vger.kernel.org, linux-clk , Linux ARM , Linux Media Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org Hi Rob, On Wed, Aug 12, 2020 at 10:36 PM Rob Herring wrote: > Clean-up incorrect indentation, extra spaces, long lines, and missing > EOF newline in schema files. Most of the clean-ups are for list > indentation which should always be 2 spaces more than the preceding > keyword. > > Found with yamllint (which I plan to integrate into the checks). > Signed-off-by: Rob Herring Thanks for your patch! > --- a/Documentation/devicetree/bindings/clock/renesas,cpg-clocks.yaml > +++ b/Documentation/devicetree/bindings/clock/renesas,cpg-clocks.yaml > @@ -24,9 +24,9 @@ properties: > - const: renesas,r8a7778-cpg-clocks # R-Car M1 > - const: renesas,r8a7779-cpg-clocks # R-Car H1 > - items: > - - enum: > - - renesas,r7s72100-cpg-clocks # RZ/A1H > - - const: renesas,rz-cpg-clocks # RZ/A1 > + - enum: > + - renesas,r7s72100-cpg-clocks # RZ/A1H > + - const: renesas,rz-cpg-clocks # RZ/A1 This change breaks alignment of the comments at the end of each line. > - const: renesas,sh73a0-cpg-clocks # SH-Mobile AG5 (I only checked the files I care about) If you don't update commit e0fe7fc6f2ca0781 ("dt-bindings: Whitespace clean-ups in schema files"), I can send a patch after v5.9-rc1. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds