[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8de13103-cd57-407c-839c-843dcedbac29@siemens.com>
Date: Mon, 26 Aug 2024 22:13:20 +0200
From: Jan Kiszka <jan.kiszka@...mens.com>
To: Nishanth Menon <nm@...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 26.08.24 21:59, Nishanth Menon wrote:
> 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/
>
Krystoff's and my patches still apply on top of master and next.
I don't mind via which patches we finally get to zero dtbs_check
warnings, just that we get there.
Jan
--
Siemens AG, Technology
Linux Expert Center
Powered by blists - more mailing lists