Re: [PATCH v3 0/2] Add capability to append to property

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



On 2024-11-11 10:54, Ayush Singh wrote:
Allow appending values to a property instead of overriding the previous
values of property.

Currently, we have /delete-node/ and /delete-property/, but lack
/append-property/. Hence we end up having to repeat all existing values
when appending to a property (e.g. see [1] appending to clocks from
[2]).

This functionality is also important for creating a device tree based
implementation to support different types of addon-boards such as
mikroBUS, Grove [3], etc.

In practice, it looks as follows:

```
dts-v1/;

/ {
	str-prop = "0";
};

/ {
	/append-property/ str-prop = "1";
};
```

If we add /append-property/, why not add /prepend-property/ as well? This is not only "nice for consistency", but it would also enable solving a problem where SoC compatible strings from dtsi [1] need to be repeated in board dts [2], because one cannot prepend to an existing property.

```
dts-v1/;
/ {
	compatible = "soc-vendor,soc1234";
};

/ {
	/prepend-property/ compatible = "board-vendor,board-xyz";
};
```

What do you think?

[1] https://elixir.bootlin.com/linux/v6.12.4/source/arch/arm64/boot/dts/renesas/r8a77951.dtsi#L18 [2] https://elixir.bootlin.com/linux/v6.12.4/source/arch/arm64/boot/dts/renesas/r8a77951-salvator-xs.dts#L14




[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux