From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 69A1435B136 for ; Mon, 7 Sep 2026 09:24:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788773089; cv=none; b=uUperASJ9eJTMS1PKQJMSeSmpH7lfbLmuXHYmO1q/UOwmMn8dAkQs5VlC0DZrwZ3LTI9PdhbtFdYum7+fVDXDDbxR+FMg3cdo+N+Gacyc/T3rM2FsLqhypuCY3U4LQ6NgVAfO6lhNleJdn5jb/JCZ7eNlHfyH5VbbPJu5Zs1Pao= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788773089; c=relaxed/simple; bh=Esx+dtM0TYwdO57j6xnTXvlu/TzjID1CmstR5RvhL0k=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=DcUVmp4p1Pg3RWS0QwglvUyHQSMlhMz7tQdmGiXzz2TEJ07k1D1nWLtYdhSHYXGPH5q4IyeNz8luIVlF4Z0cQrnxOD+wFWRVhpkUEQpqKMhTP2Jw+U8urwPUCGhay342JHDqgWlTNaFJBpLo9KTClh3dnNwd3UpYblr5v6KUhjY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=QyNOXyZx; arc=none smtp.client-ip=209.85.128.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QyNOXyZx" Received: by mail-wm1-f54.google.com with SMTP id 5b1f17b1804b1-49cca4ffdcfso27166655e9.0 for ; Mon, 07 Sep 2026 02:24:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788773085; x=1789377885; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:subject:cc:to:from:date:message-id:from:to:cc:subject :date:message-id:reply-to:content-type; bh=oRUYERj4W2ZSChyYrd8ZBUDzTcb+VBgjaS3zm4CIBsE=; b=QyNOXyZxk/VY3LypHInkRb2EunLm1ieDNALCC4W7lD56GpTbLVPHAeoyeBfqLUwLe+ de2c0gR5veh/0oXJal6zKnL2U9E+zhykrWILMAKzA3aSAhNOVDIjvs0vsXtjxhy8jOtg ozKndcZn2U9yIRZaV2pdkVbxQAYKGyGXBLHmIH5riH1nEQGUG/XjtWBRFYIGUgLCapRu Bq3RGYjcbgykIeqTlCxiFewhWFeEvPy4bDnVEV2w38u6tdncMBjjBKEkbSabex7gkbZG +jgrreiWu1FnGhy8c8GTVot9Cf+9oxgj4z5R2qsXVD+mf0Qypbkjih6C+IsEuLEKpfaN 2Jyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788773085; x=1789377885; h=in-reply-to:content-disposition:content-type:mime-version :references:subject:cc:to:from:date:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=oRUYERj4W2ZSChyYrd8ZBUDzTcb+VBgjaS3zm4CIBsE=; b=H1TYTrAycJ6NupGaJ1UgiXAEuzoMcA7dAd1odQBWLUeLRPAHikEppbNRIy82ASM1Yb Jpxngg2lXZcaA1iUyKOqYx8fGBUiY3U4vx7KT1gD9anvcJ/F8k+zblO5YBV7IxfnuwdB zanDXRnSYAb06bcMQ7IRbMCxpcg5M/NafAW9MvwBc55Hi1pIuqmQDokRg0bl6t2OsyRD HU8lk5T5xlIwRke3pErxbAN7O2o/vLpxs25AGg3YpJqdCppSI6DJFxHnqsoY1SytuEhk 7K0JjZnfYMmf0HZ3/Wps9dTg1fLwcs5y++1ufPVgO/dOCgY1KSXnv4JVleWMkKWJSaUQ Yjbg== X-Forwarded-Encrypted: i=1; AKwUvBwqqt0BbVs5KYim+CMtiPpvwJ/+C1VYj3EPnMqYxKirLExZWED7kxGxGI2W6Djlbzq4VUr1jHd/QnpjbLA=@vger.kernel.org X-Gm-Message-State: AFuF++l9RMM2axzSNwEoh1r5vqqRGSrzfPGjkZ9q7mqnW//IbNb3F3+w 0hUg0AfcycPS+ZEg1Au/JzWpU+b5jVzbZ2nUpY0u9HFWysInBTErABsu X-Gm-Gg: AYBFou3ndu42lHGJsbcU4ogVF6SQ6L3KK4sXv6u2aatRjYxqRNwtyQg3Dh4pAbCnvio 6dKRlLV/7x6OQnQPpaBYUXUu0ejmNIrWzQ2Ef/8iadT6ckuhqdrvfNRAN4S4Zqd0P6EmO8mmNNa MgyQxGLMjr/TrGZxeXHQ5t+Gr667ZspvciKzmrpfRDtc6HjSZYG79+QewEp0tHTBgQ7ZTSWeLGX 64j7F+QU2zfB7BJ+e4RIavcp11ocpwl1ddBURIWp/fDAzs/RylExm6u5cwpN7AWxgG3fzZvPLJP Tkz1vVw5dbLrE9L1l31dvWdygZqzp3TxThyBEIto7L+7ax1PQtQZcgaoEhx1sCZzc7jrBHq7IY3 0u8OBTBg9kYBK7r2iG10pPqZUFe3Y5YCMp6Jtb7KMgiQqqEUBYtP8rp1At9Rpsel0W8mXn4db/6 0z/AKayQS/L71lkP/YzJlumBHOf7a+k0o/+AG9dzVVyp/9Er/tvwxDZYyKUseIReL/R0tvVu3X6 doxqyHWavRO/QywMp9vIYbAIXCZF1Z5NLnU X-Received: by 2002:a05:600c:310d:b0:49c:dc14:d681 with SMTP id 5b1f17b1804b1-49cf81e5449mr213057725e9.3.1788773085428; Mon, 07 Sep 2026 02:24:45 -0700 (PDT) Received: from Ansuel-XPS. (host-79-26-252-140.retail.telecomitalia.it. [79.26.252.140]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49cf755c22esm294849975e9.0.2026.09.07.02.24.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Sep 2026 02:24:44 -0700 (PDT) Message-ID: <6a9e82dc.8fb0a6ce.a0d71.1d0a@mx.google.com> X-Google-Original-Message-ID: Date: Mon, 7 Sep 2026 11:24:41 +0200 From: Christian Marangi To: Krzysztof Kozlowski Cc: Stephen Boyd , Brian Masney , Jerome Brunet , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Felix Fietkau , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 1/3] dt-bindings: clock: airoha: Document support for AN7583 clock References: <20260819221458.30040-1-ansuelsmth@gmail.com> <20260819221458.30040-2-ansuelsmth@gmail.com> <20260827-expert-ruby-grasshopper-954aa2@quoll> 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: <20260827-expert-ruby-grasshopper-954aa2@quoll> On Thu, Aug 27, 2026 at 11:52:13AM +0200, Krzysztof Kozlowski wrote: > On Thu, Aug 20, 2026 at 12:14:54AM +0200, Christian Marangi wrote: > > Document support for Airoha AN7583 clock. This is based on the EN7523 > > clock schema with the new requirement of the "airoha,chip-scu". > > > > Add additional binding for additional clock and reset lines. > > > > Signed-off-by: Christian Marangi > > --- > > .../bindings/clock/airoha,en7523-scu.yaml | 20 ++++++ > > include/dt-bindings/clock/en7523-clk.h | 3 + > > .../dt-bindings/reset/airoha,an7583-reset.h | 65 +++++++++++++++++++ > > 3 files changed, 88 insertions(+) > > create mode 100644 include/dt-bindings/reset/airoha,an7583-reset.h > > > > diff --git a/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > index eb24a5687639..edecc635807b 100644 > > --- a/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > +++ b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > @@ -30,6 +30,7 @@ properties: > > compatible: > > items: > > - enum: > > + - airoha,an7583-scu > > - airoha,en7523-scu > > - airoha,en7581-scu > > - econet,en751221-scu > > @@ -50,12 +51,30 @@ properties: > > description: ID of the controller reset line > > const: 1 > > > > + airoha,chip-scu: > > + $ref: /schemas/types.yaml#/definitions/phandle > > + description: phandle to the Chip SCU providing the registers required > > + for configuring the PCIe related clocks and resets. > > This is the clock provider. Clock provider should not be accessing > registers of other device to configure its clocks. Either you > misrepresented clock hierarchy or devices. > > I did search for DTS to try to understand the big pictuer - nothing, no > results, no upstream submission to Linux kernel. > Hi, you can use en7581 as an example as the implementation is exactly the same register wise. The register for clock and reset for normal system and PCIe are scattered between the "SCU" and "chip SCU" registers (they are 2 different register block, the name is taken from the programming guide) - Chip SCU provide register access to some clk gate and clk rate - SCU provide register for reset, PCIe clock, other clock gate and PHY SERDES. Upcoming and current airoha clock driver all follow this pattern of declaring one of the 2 register and use a sysconf for the other as the 2 register block are tighlty coupled. There was a similar phandle in another series adding support for pinctrl for econet EN7528. This phandle is not present for en7581 just because it's hidden by a direct call to the syscon with syscon_regmap_lookup_by_compatible API. With AN7583 I'm trying to fix this by making it more explicit. Does this makes the situation more clear? -- Ansuel