linux-kernel - Re: [PATCH v4 2/3] dt-bindings: soc: ti: am645-system-controller: add child nodes used by main domain

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]

Message-ID: <20240826195930.c7bylbgu6qma2v3c@siren>
Date: Mon, 26 Aug 2024 14:59:30 -0500
From: Nishanth Menon <nm@...com>
To: Jan Kiszka <jan.kiszka@...mens.com>
CC: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Vignesh
 Raghavendra <vigneshr@...com>,
        Tero Kristo <kristo@...nel.org>, Rob Herring
	<robh+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
        <devicetree@...r.kernel.org>, Bao Cheng Su <baocheng.su@...mens.com>,
        Diogo
 Ivo <diogo.ivo@...mens.com>
Subject: Re: [PATCH v4 2/3] dt-bindings: soc: ti: am645-system-controller:
 add child nodes used by main domain

On 14:01-20240826, Jan Kiszka wrote:
> On 26.08.24 13:42, Nishanth Menon wrote:
> > On 13:31-20240826, Jan Kiszka wrote:
> >> On 24.08.24 19:58, Nishanth Menon wrote:
> >>> On 18:37-20240814, Jan Kiszka wrote:
> >>>> From: Jan Kiszka <jan.kiszka@...mens.com>
> >>>>
> >>>> Expand bindings to cover both the MCU and the main usage of the AM654
> >>>> system controller.
> >>>>
> >>>> Signed-off-by: Jan Kiszka <jan.kiszka@...mens.com>
> >>>> ---
> >>>>  .../soc/ti/ti,am654-system-controller.yaml    | 19 +++++++++++++++++++
> >>>>  1 file changed, 19 insertions(+)
> >>>>
> >>>> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml
> >>>> index e79803e586ca..cb9da3ec39a8 100644
> >>>> --- a/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml
> >>>> +++ b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller.yaml
> >>>
> >>> As per linux-next, this file does not exist? looks like you might need
> >>> to rebase on latest next?
> >>>
> >>
> >> "This goes on top of
> >> https://patchwork.kernel.org/project/linux-arm-kernel/cover/20240518-dt-bindings-ti-soc-mfd-v1-0-b3952f104c9a@linaro.org/"
> >>
> >> Is that series obsolete by now?
> > 
> > This dependency information would have been useful when provided under the
> > diffstat section of the representative patch. :(
> > 
> 
> Yeah, who reads cover letters? ;) Will try to do that next time.

Looking at what we included in v6.11, looks like the above series from
krystoff will need to be re-thought through.
[1] makes the patches un-necessary to an extent by modelling the node as
a simple-bus - Looks like k3-am62-wakeup.dtsi was missed, but it should
be trivial enough fixup. Did you try applying the series on latest next?
it should not apply anymore (Doesn't apply on master either).

[1] https://lore.kernel.org/all/20240628151518.40100-1-afd@ti.com/

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D