linux-kernel - Re: [GIT PULL] ARM: SoC fixes (and a cross-arch dt-include fix)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 22 May 2017 13:44:35 +0200
From:   Linus Walleij <linus.walleij@...aro.org>
To:     Olof Johansson <olof@...om.net>, Rob Herring <robh+dt@...nel.org>
Cc:     Linus Torvalds <torvalds@...ux-foundation.org>,
        "linux-arch@...r.kernel.org" <linux-arch@...r.kernel.org>,
        "arm@...nel.org" <arm@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [GIT PULL] ARM: SoC fixes (and a cross-arch dt-include fix)

On Fri, May 19, 2017 at 9:34 PM, Olof Johansson <olof@...om.net> wrote:

>  - We've started telling people to avoid cross-tree shared branches if all
>    they're doing is picking up one or two DT-used constants from a
>    shared include file, and instead to use the numeric values on first
>    submission. Follow-up moving over to symbolic names are sent in right
>    after -rc1, i.e. here. It's only a few minor patches of this type.

OK it seems like a reasonable process.

It's not like I can think about anything better.

I was more opting for doing things this way:

- Create bindings and <dt-bindings/foo/bar.h> merge it into the
  foo subsystem.

- Merge driver into drivers/foo/bar.c that use <dt-bindings/foo/bar.h>

- Submit DTS patch to ARM SoC (or whetever) using only numerical
  values.

- After the merge window, follow up with a patch replacing the
  numerical constants with #defines from <dt-bindings/foo/bar.h>

An alternative would obviously be to wait for the next merge window
after merging the driver patch but I guess people are too impatient
to do that (including me).

We discussed cross-tree dependencies a bit on ksummit-discuss
but this solution was not mentioned back then.

Yours,
Linus Walleij