From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 460A733BBA3; Thu, 22 Jan 2026 16:46:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769100370; cv=none; b=T3v0l6NbjEyOw9uk9dSVlKcjRCnPCTHO/qr5QLKPVS5RoEYl7bLYa6ZjwdMbaT2jlsmDPa5o7Unea+MxiMiKIcOfcD+I/7Ur4XsNTmWuqFOyyiq7eYbbYXKop4L7yCjF6BXFoq/shrcFJWuQgD5bQYgVqcHetqXZLMLBN7VJlFY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769100370; c=relaxed/simple; bh=uPmc3PZ3Z09O12bVNITp9Iv/l+6xdUK6dBV30LVmlvg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=XUCgv7m8YOuGcFnW8MC31ke/8xYQpqKxUbkM7NHRuJ3ID7Tzbb8m2p8dpIP67BO2Cl4soq4zaA0+Z56NY3NA+iNQDZyKLVqiYyey4P1UC6Ckkdgc0SUmuTN84NvYMiCaVhAMOqKLl6rMxL8AN/mCVavl4Nkcwk2DU9ZV/CbGYCI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Ilcr6s08; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Ilcr6s08" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2B899C19422; Thu, 22 Jan 2026 16:46:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1769100369; bh=uPmc3PZ3Z09O12bVNITp9Iv/l+6xdUK6dBV30LVmlvg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Ilcr6s08HPute2XfdseF2UWovLayU+6lOFa1F0pOPNz/pJcyfLKkN+jo/3D06WajM zTEOiJwfliH9sltRoev9dGW5fPyKP5UXHzi6T3B/RSN/kbActZHvbQpxSeXJ1Id/3m xuo/vJBoHL9wTDi+1qPgIWt3uUZtGoAf0SNHRDZNt84ZctGS7O/N6ZJyTLA3Q/R2Az DzhbjMXftmvmb5/aXTGyPhPmx0NfFLcY5diMD5HHXmISKBXjgXCsYMxPuh8fNiMOJx STPrcsHjorejznDLHUHS+kWHwFwH29ZpKygScfiHyf09mQS/uguaczXbWQinDPenqq KyKh+qMvLJs4w== Date: Thu, 22 Jan 2026 10:46:08 -0600 From: "Rob Herring (Arm)" To: Alexander Dahl Cc: Michael Turquette , Philipp Zabel , Ryan Wanner , linux-kernel@vger.kernel.org, Krzysztof Kozlowski , linux-clk@vger.kernel.org, Stephen Boyd , linux-arm-kernel@lists.infradead.org, Alexandre Belloni , Conor Dooley , Claudiu Beznea , devicetree@vger.kernel.org, Nicolas Ferre Subject: Re: [PATCH v3 02/19] dt-bindings: clock: at91: Split up per SoC partially Message-ID: <176910036733.2542290.7853965507867142412.robh@kernel.org> References: <20260120143759.904013-1-ada@thorsis.com> <20260120143759.904013-3-ada@thorsis.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260120143759.904013-3-ada@thorsis.com> On Tue, 20 Jan 2026 15:37:23 +0100, Alexander Dahl wrote: > Before adding even more new indexes creating more holes in the > clk at91 drivers pmc_data->chws arrays, split this up. > > This is a partial split up only for some of the old PMC definitions. > > Binding splitup was proposed for several reasons: > > 1) keep the driver code simple, readable, and efficient > 2) avoid accidental array index duplication > 3) avoid memory waste by creating more and more unused array members. > > Old values are kept to not break dts, and to maintain dt ABI. > > Link: https://lore.kernel.org/linux-devicetree/20250207-jailbird-circus-bcc04ee90e05@thorsis.com/T/#u > Signed-off-by: Alexander Dahl > --- > > Notes: > v3: > - full splitup for all at91 family SoCs instead of just a few > > v2: > - new patch, not present in v1 > > .../reset/atmel,at91sam9260-reset.yaml | 3 ++- > .../dt-bindings/clock/atmel,at91rm9200-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/atmel,at91sam9260-pmc.h | 16 +++++++++++++ > .../dt-bindings/clock/atmel,at91sam9g45-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/atmel,at91sam9n12-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/atmel,at91sam9rl-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/atmel,at91sam9x5-pmc.h | 15 ++++++++++++ > include/dt-bindings/clock/atmel,sama5d2-pmc.h | 20 ++++++++++++++++ > include/dt-bindings/clock/atmel,sama5d3-pmc.h | 14 +++++++++++ > include/dt-bindings/clock/atmel,sama5d4-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/microchip,sam9x60-pmc.h | 15 ++++++++++++ > .../dt-bindings/clock/microchip,sam9x7-pmc.h | 19 +++++++++++++++ > .../clock/microchip,sama7d65-pmc.h | 24 +++++++++++++++++++ > .../dt-bindings/clock/microchip,sama7g5-pmc.h | 19 +++++++++++++++ > 14 files changed, 219 insertions(+), 1 deletion(-) > create mode 100644 include/dt-bindings/clock/atmel,at91rm9200-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,at91sam9260-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,at91sam9g45-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,at91sam9n12-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,at91sam9rl-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,at91sam9x5-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,sama5d2-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,sama5d3-pmc.h > create mode 100644 include/dt-bindings/clock/atmel,sama5d4-pmc.h > create mode 100644 include/dt-bindings/clock/microchip,sam9x60-pmc.h > create mode 100644 include/dt-bindings/clock/microchip,sam9x7-pmc.h > create mode 100644 include/dt-bindings/clock/microchip,sama7d65-pmc.h > create mode 100644 include/dt-bindings/clock/microchip,sama7g5-pmc.h > Acked-by: Rob Herring (Arm)