[PATCH 10/12] dt-bindings: ras: document estatus provider
Krzysztof Kozlowski
krzk at kernel.org
Fri Dec 19 01:53:23 PST 2025
On 18/12/2025 11:31, Ahmed Tiba wrote:
> On 17/12/2025 12:41, Krzysztof Kozlowski wrote:
>>> +properties:
>>> + compatible:
>>> + const: arm,ras-ffh
>>
>> Again ras - what's that? Your patch or binding must explain that.
>
> That updated description will explicitly expand the Arm RAS acronym so the
> compatible string is self-explanatory.
>
>>> +
>>> + reg:
>>> + minItems: 1
>>
>> Why is this flexible?
>
> I'll keep `reg` describing the CPER status buffer, cap it at two entries, and
> document the second entry as the optional doorbell register that some firmware
> requires before reusing the buffer.
I still do not understand why this is flexible or in other words - why
second address space appears and disappears.
>
>>> + items:
>>> + - description: CPER status block exposed by firmware
>>> + - description:
>>> + Optional 32- or 64-bit acknowledgment register. Firmware watches this
>>> + register and expects bit 0 to be written to 1 once the OS consumes the
>>> + status buffer so it can reuse the record.
>>> +
Best regards,
Krzysztof
More information about the linux-arm-kernel
mailing list