Received-SPF: pass (google.com: domain of linux-kernel+bounces-64141-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Tue, 13 Feb 2024 19:13:57 +0000
From: Conor Dooley <conor@kernel.org>
To: Dharma.B@microchip.com
Cc: robh@kernel.org, tglx@linutronix.de, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, Nicolas.Ferre@microchip.com,
	alexandre.belloni@bootlin.com, claudiu.beznea@tuxon.dev,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2] dt-bindings: interrupt-controller: Convert Atmel AIC
 to json-schema
Message-ID: <20240213-estranged-charger-bf0372f367e0@spud>
References: <20240209100122.61335-1-dharma.b@microchip.com>
 <20240212140824.GA107736-robh@kernel.org>
 <003d61c9-b914-4e1c-b3f8-1140ea640039@microchip.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="DP2O63EPE4l7uJmE"
Content-Disposition: inline
In-Reply-To: <003d61c9-b914-4e1c-b3f8-1140ea640039@microchip.com>


--DP2O63EPE4l7uJmE
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, Feb 13, 2024 at 04:23:36AM +0000, Dharma.B@microchip.com wrote:
> On 12/02/24 7:38 pm, Rob Herring wrote:
> > On Fri, Feb 09, 2024 at 03:31:22PM +0530, Dharma Balasubiramani wrote:
> >> +  atmel,external-irqs:
> >> +    $ref: /schemas/types.yaml#/definitions/uint32-array
> >> +    description: u32 array of external irqs.
> >=20
> > Constraints on the array size and/or entry values?
>=20
> The hardware's support for external IRQs may differ, which is why a u32=
=20
> array is utilized. This choice is based on the fact that IRQ numbers are=
=20
> commonly expressed as integers, and a 32-bit unsigned integer provides a=
=20
> standardized size capable of representing a broad range of numbers. This=
=20
> size is more than adequate for accommodating IRQ numbering.

I don't think Rob was questioning your use of u32s, but rather the fact
that you do not limit the values at all nor the number of values.

--DP2O63EPE4l7uJmE
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZcu/dAAKCRB4tDGHoIJi
0hRGAQDFmS0WZOIhtx+CxVE2or6F0Xb2Sy0lXRItnm9EtBauHgD+J50w1UOrdUOX
LsiY0HyUU5V9F0jwzQmdiaR/ntTSfAY=
=eCUq
-----END PGP SIGNATURE-----

--DP2O63EPE4l7uJmE--