Re: [PATCH] dt-bindings: firmware: coreboot: Document optional device specific properties

[Krzysztof Kozlowski <krzk@kernel.org>]

On 19/11/2025 09:54, Chen-Yu Tsai wrote:
> On Wed, Nov 19, 2025 at 4:13 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 19/11/2025 08:32, Chen-Yu Tsai wrote:
>>> On Wed, Nov 19, 2025 at 3:13 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>>
>>>> On 19/11/2025 08:11, Chen-Yu Tsai wrote:
>>>>> Coreboot, or the ChromeOS second stage bootloader, depthcharge, will
>>>>> insert device specific properties into the coreboot firmware node when
>>>>> there are valid values.
>>>>>
>>>>> Document these properties in the binding.
>>>>>
>>>>> Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
>>>>> ---
>>>>>  Documentation/devicetree/bindings/firmware/coreboot.txt | 9 +++++++++
>>>>>  1 file changed, 9 insertions(+)
>>>>>
>>>>
>>>> TXT files cannot receive new properties. You need to first convert to DT
>>>> schema.
>>>
>>> OK. Let me look into this.
>>
>> After the conversion you will hit another problem - you need vendor
>> prefixes for these, because only generic properties can come without
>> them. Otherwise (without vendor prefix) these would define the type for
>> all other bindings, which probably is not what we want.
> 
> I understand the concern. But given it's specifically under the
> /firmware/coreboot node, which is inserted by coreboot, doesn't that
> already serve as a namespace or vendor prefix?

Unfortunately not or not completely. Properties have single type
globally (with exceptions). This means whatever you write here for
board-id will affect every possible future board-id property.

> 
> FWIW the ship has already sailed for naming. The first three properties
> were added to depthcharge [1] and coreboot [2] in 2018. The last property
> was added to depthcharge in 2023 [3]. That is what has shipped in immutable
> firmware on ARM-based Chromebooks since the RK3399 days. The coreboot
> change was presumably added for other devices.
> 
> This change only serves to document what the firmware already provides.
> Whether they should be grandfathered in or not doesn't change what the
> firmware already does; it just makes it more well known. It's not going
> to have any effect on validation either, as the properties are supposed
> to be inserted by the bootloader, not added statically to dts files.

If you convince Rob then it's fine, but I don't want to accept
downstream solutions just because they exist. This opens the door for
every vendor to implement what they want and later ask for forgiveness
("oh it is already there and we cannot change shipped devices"). I mean,
this was fine back 2010, but that was 15 years ago.

Best regards,
Krzysztof