On 24/02/26 07:30, Tom Rini wrote:
> On Tue, Feb 24, 2026 at 06:42:59AM +0530, Santhosh Kumar K wrote:
>> Hello Tom,
>>
>> On 23/02/26 20:54, Tom Rini wrote:
>>> On Mon, Feb 23, 2026 at 11:49:06AM +0530, Santhosh Kumar K wrote:
>>>> Add 'bootph-all' for 'main-bcdma' node to pass it to the FDT in early
>>>> stage to allow the BCDMA channel 0 to be allocated for OSPI NOR reads.
>>>>
NACK.
>>>> Signed-off-by: Santhosh Kumar K <[email protected]>
>>>> ---
>>>> arch/arm/dts/k3-am642-evm-u-boot.dtsi | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>>>> b/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>>>> index 6469f43fca81..5048432ef8ca 100644
>>>> --- a/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>>>> +++ b/arch/arm/dts/k3-am642-evm-u-boot.dtsi
>>>> @@ -42,6 +42,7 @@
>>>> <0x00 0x484c2000 0x00 0x2000>;
>>>> reg-names = "gcfg", "bchanrt", "rchanrt", "tchanrt", "ringrt",
>>>> "cfg", "tchan", "rchan";
>>>> + bootph-all;
>>>> };
>>>
>>> What's the status on upstreaming that whole node right there? Thanks.
>>
>> 'main_bcdma' node is already upstream [0] [1]
>>
>> [0]
>> https://elixir.bootlin.com/u-boot/v2026.04-rc2/source/dts/upstream/src/arm64/ti/k3-am64-main.dtsi#L141
>> [1]
>> https://elixir.bootlin.com/u-boot/v2026.04-rc2/source/arch/arm/dts/k3-am642-evm-u-boot.dtsi#L34
>
> Then we are we not upstreaming "bootph-all" and also everything else in
> our local node here? Thanks.
>
As Tom mentioned, Please required bootph-all in upstream (kernel) DT
first and let the nodes sync to U-Boot DT via regular DT sync flow. We
need to clean up -u-boot.dtsi over time and not add more code to it.
--
Regards
Vignesh
https://ti.com/opensource
