Re: [PATCH 1/4] DNM: dts: arm64: rockchip: copy pf5 device tree from mainline Linux

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

Hi,

On 12.04.24 15:02, Michael Riesch wrote:
> Hi Marco,
> 
> Thanks for your response!
> 
> On 4/5/24 19:02, Marco Felsch wrote:
>> Hi Michael,
>>
>> thanks for your patches.
>>
>>> DNM: dts: arm64: rockchip: copy pf5 device tree from
>>
>> I suppose DNM means "do not merge", instead of marking it this way we
>> put the not yet upstream Linux dts files into arch as well but mark them
>> as upstream dts file, e.g.:
>>
>>   - arch/arm/dts/imx8mm-innocomm-wb15-evk-upstream.dts
>>   - arch/arm/dts/imx8mp-debix-model-a-upstream.dts
>>   - arch/arm/dts/imx8mp-debix-som-a-bmb-08-upstream.dts
>>   - arch/arm/dts/imx8mp-debix-som-a-upstream.dtsi

I have been wondering if we should add yet another device tree
directory for kernel DTs that are not yet in the kernel, e.g.
common/boards/dts. Then we can place full DTs there, remove them
once they are in dts/ and clean up arch/$ARCH/dts/ that way.

> OK, I'll do it that way!
> 
>> Once the files are usptream we can remove them.
> 
> Is this something I should keep in mind or may I expect that this
> happens automagically?

Usually, when someone notices. Having them in a single directory
as suggested above may make it easier to sport no longer needed DTs.

Thoughts?

Cheers,
Ahmad

> 
>> On 24-04-05, Michael Riesch wrote:
>>> [...]
> Best regards,
> Michael
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |