From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (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 746D7448B84 for ; Mon, 7 Sep 2026 09:24:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788773089; cv=none; b=HZzDa1/vYcey/eVrbAvkCEqU3QF69qnkDOoZBYJQSHqPmerF391tVWpK5fVmDl2swsRxECeuroe4zMWzJp0/gGes6L1Flw+QnIHvBgMrjqu6qe3hB8FTfQKn0ZoELyrOjB9nwq/ShG2kedHte9gv57uJ2Zu3bYugdHG42fIo2fk= 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.43 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-f43.google.com with SMTP id 5b1f17b1804b1-49d05d51553so13385225e9.2 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=KSz55xpdKaa1ErlmqFVbErsBLmRWUZYy70kl0uh2FFK8xRRt1ezpin8Jgav5Jpgb2t DxgF6W3yrC8Fh7zceu8iqmfkvtA/pebbozZfY3BKDH5DW4mqmhjn0wNG8U1ONvoDcihF sHC2koxEkfylkfCFiRX5NXNHy9WiTc/CEiATmFcQURK+gPjlojNZ9DLramflN8FWVEuu Epmas4NbaMSjHIxB1onY/rwHy+QvFxFtq8n214e5QfOLZPp0xYJVcQ33ExP6+sTtL3AL pXa7CJbTiExq+Ltdgc9OqAtLx/81t9OiTi7nVbVqSLutm/W0vtfa1u10BlL11tHG3UVT PWKQ== X-Forwarded-Encrypted: i=1; AKwUvByGyuxcgJg0sym1YjEMXlf0NX75WmeQ/qqglvYWXylOGas/jsEnGBdJfWzkFgeHl8lX2HugPoraCh4=@vger.kernel.org X-Gm-Message-State: AFuF++kF3tSApboQnIX0HSSPBlj8YGq6AywlNpjeeFQc8TKcPNV+ndTm QOz2TJlMdTbpqs+J45+uneqf9tHxjjv4EHG0XTiOdC63BVF9lyYYZafG X-Gm-Gg: AYBFou3pgQ3MkhUl2WJNF1jCf7+SLtLrmBzJJ7adXgFtTJOnjLVByu9Jiz+fyp/hNpJ P4hLv+FHdsNuJfv0Kex0l36gS4tFSJ6hmfe1GTMyMousAsBAQRTqnO1Vms3e4nYSerUm8W8kY4M zd+sLYl1mskZrHJDZLOCeMWtTMNfAEzMnIlpb05Yo4FdOYxa2MyAU5g4kCn8C/z73zj+8a2plSk vsXZ9jbClsOlIPRimiigAAbCb4XWmiRtX5JH6Xq4+75ZbMbQx2fYfqFHy1MHV71EBdWkbI7Jrjs 0j2v1q2VPmhzhWgZA/JR4qXiZD7NoVeHq8PiQ4f8JPG/9mOPXE1irN2a6Tgho6IV4jgWxZ19Sp8 9Asc26mpkyvz0TixPNIvAPOLJn2zWfQLdV96sRqYsP/MSRmXc7vW6SXy4OMWghaR0vfc0V/FG1J KNmLdlMnZldcYYyIcwsppZQav/WYtN6QTzPR8AQ1K0LVZb8TXw8CGiVJC7pLQL56IMdUJlVOOsE 5/5SpVH9B8aBIheBEPnQasvZu8bqADDxhbV 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-clk@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