Re: [PATCH v2 1/4] dt-bindings: arm: ras: Introduce bindings for ARM RAS error sources

Krzysztof Kozlowski <[email protected]> Tue, 28 Jul 2026 10:47:33 +0200

Newsgroups	org.kernel.vger.linux-edac,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-acpi,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID	<20260728-determined-lorikeet-from-tartarus-cf124e@quoll>

On Mon, Jul 27, 2026 at 01:51:08PM -0700, Trilok Soni wrote:
> On 7/20/2026 1:19 AM, Umang Chheda wrote:
> > new file mode 100644
> > index 000000000000..c2f4e1f8243e
> > --- /dev/null
> > +++ b/include/dt-bindings/arm/arm-ras.h
> > @@ -0,0 +1,11 @@
> > +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
> > +
> > +#ifndef _DT_BINDINGS_ARM_RAS_H
> > +#define _DT_BINDINGS_ARM_RAS_H
> > +
> > +/* arm,group-format - error record group register window page size */
> > +#define ARM_RAS_GROUP_4K		0	/* 4 KiB,  1 ERRGSR  */
> > +#define ARM_RAS_GROUP_16K		1	/* 16 KiB, 4 ERRGSRs */
> > +#define ARM_RAS_GROUP_64K		2	/* 64 KiB, 14 ERRGSRs */
> 
> Do you want me to have different DT for 4k and 16k and 64k? This is not a right approach. 
> 

Is this page size or register window page size? If the latter, why would
device registers be a binding?

Best regards,
Krzysztof