Re: [PATCH] dt-bindings: clock: agilex: mark mpu_l2ram_clk deprecated
From: Krzysztof Kozlowski
Date: Thu Aug 06 2026 - 04:03:25 EST
On 06/08/2026 09:43, Chan, Genevieve wrote:
>>> +/* 42 - reserved/unused */
>>
>> How IDs can be reserved? These are implemented by drivers (abstract IDs
>> not used by hardware), so how driver could reserve it?
>>
>> Best regards,
>> Krzysztof
>>
> You are right. These IDs are DT clock specifier indices (provider one-cell ABI), not hardware IDs. The clk driver registers clocks into hws[id]; unused indices remain -ENOENT.
>
> ID 42 was never defined in the original Agilex binding (EMAC0=41, EMAC1=43). The comment was placed to only documents that historical hole so we do not renumber later IDs. It is not a hardware reservation — it is for ABI hygiene.
>
> Happy to reword to something clearer, e.g. /* 42 never assigned; do not renumber */, or drop the comment if preferred.
>
Please wrap your responses.
Comment is simply not needed.
Best regards,
Krzysztof