On Fri, Jan 14, 2022 at 09:21:52AM -0600, Rob Herring wrote: > On Thu, Jan 13, 2022 at 02:39:40PM +0100, Stanislav Jakubek wrote: > > Indent comments the same as the content they describe. > > Fixes yamllint warnings like: > > 32:13 warning comment not indented like content (comments-indentation) > > But that's not a warning we enable or is this something new in newer > yamllint? Don't know, I just noticed that the comment indentation is inconsistent and decided to fix that, but checkpatch wasn't complaining about it, so I ran it through yamllint to check which one is "correct". I guess the default yamllint config complains about it. It also gave me errors about some line lengths being > 80, but I decided to leave those alone. Not sure if it's worth it to break up the comments into multiple lines for that. Thanks, Stanislav > > In any case, a change like this can only be done at the end of the merge > window just before rc1. Otherwise, it is merge conflict hell. I would do > that, but I really prefer it the way it is. The reason is it makes the > '-' stand out more and those are easy to miss in yaml. For example, > these look the same but are a bit different: > > - items: > - const: foo > > - items: > const: foo > > > > > Signed-off-by: Stanislav Jakubek <stano.jakubek@xxxxxxxxx> > > --- > > .../devicetree/bindings/trivial-devices.yaml | 314 +++++++++--------- > > 1 file changed, 157 insertions(+), 157 deletions(-)
