Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp6680035rdb; Fri, 15 Dec 2023 05:43:10 -0800 (PST) X-Google-Smtp-Source: AGHT+IGTNepACWAe+F6QQtmof1oWAJEAvdj1KVP4rk3P/VzT/ZVUYqh8ufJwF1Q5pWYfwUqzJtQL X-Received: by 2002:ac8:57ce:0:b0:425:4043:8d4a with SMTP id w14-20020ac857ce000000b0042540438d4amr11311977qta.101.1702647790528; Fri, 15 Dec 2023 05:43:10 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702647790; cv=none; d=google.com; s=arc-20160816; b=06qPl4t7AvrSXwZNUiEt+3byknP+bf1q042fYap1qN7S6iH+5kSFJEHMBJYd4Kep5q SCIjW6XwDoSJ55WBUIrWtB3R8ADMaLiLHTSVDJL96PBj2fJI1HbhpsHjsYPKmYU0MAko BfnV5pxsgK0wQOKkfpDLEoPzYH1N82I0SP8jDVQUM89E6/zZ1uU4U0LaqjSA2YUsAUK9 HC9r0DRxyPEEX9fhfg1YeCQlXq/tW73TrPNDsKSP+zuvxJkh+CI3GvqXJbtMYiQoWztS wcFYkh1RaTykZ4ed66OT7l+Z7h86f92m9DnfsStyont/Bl+iXAcAE2cQqs0CPdKfVAVN Gi5A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:in-reply-to:content-disposition:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:references :message-id:subject:cc:to:from:date:dkim-signature; bh=pv0AaHgjkAy6u95Pj8OOH2/guQb3Aq8ffCZaWiu1IRg=; fh=nw1ELX/P8ZH49zD/kHYIlgDJTgOkkNJ2IBlIz9NE+Pw=; b=apgpnr/DPPSgjFOGXr73hQoIzrByq9UC9syWUrUjHy5+wmxxFZMIKchGk4V7BFkq1s 8TeBQvvQgm4Nq8zrJErvR1cz11UiKyV8TWBjp2NkN38YDYfmVGnX5mo0WIjQWwMtOjcQ itPzbhfHo6jLpegWujaEyGCI3pjAd+hZxF/K+u31zrQ7xI1xcC3eZE2ayS5q+oBkSNAl fjNlzWawEjdcBo3Ts+bZFWkGREbqV49qDhEXJP80/0OrhmR/b9Oajg/8aBgGScmLlt8P xO+v1PB3UPDTJymAaJ7x+pWkjDlrb3iiwhzMwjRESpjpZcYXgo+E2Y0Dr9xd17JSUowI NTnQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail (test mode) header.i=@armlinux.org.uk header.s=pandora-2019 header.b=lG7viCNz; spf=pass (google.com: domain of linux-kernel+bounces-1033-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-1033-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=armlinux.org.uk Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id w15-20020a05622a134f00b00425e75b5a54si8286572qtk.343.2023.12.15.05.43.10 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Dec 2023 05:43:10 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-1033-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=fail (test mode) header.i=@armlinux.org.uk header.s=pandora-2019 header.b=lG7viCNz; spf=pass (google.com: domain of linux-kernel+bounces-1033-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-1033-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=armlinux.org.uk Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 379591C23D59 for ; Fri, 15 Dec 2023 13:43:10 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5697A2E40A; Fri, 15 Dec 2023 13:42:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="lG7viCNz" X-Original-To: linux-kernel@vger.kernel.org Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [78.32.30.218]) (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 808ED30321; Fri, 15 Dec 2023 13:42:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=armlinux.org.uk DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=pv0AaHgjkAy6u95Pj8OOH2/guQb3Aq8ffCZaWiu1IRg=; b=lG7viCNzqAjzgDF7gJwFPmDEJ4 LjsCYZjzFLR2bZTBYmuCcgLYwjT2eW9VGwCO+J8AgkzFFivO5Y7jx6u5khrCR4Kkl4tlxWeuARR2b zMIyXm8Iw1VWdmpe0o5eNSF6ZWnuBYXaG/cztXM6PEZZy1ymzgxCUOZO8jZncwaeoMOr4oanxJXn0 ll8PMs9VoKw1NBUkvId0gijqpyLdeSttPDx6Hl4oRPmhTqnfP67dt0/H46tVkz+qs4X+J+xnZk4Bs 4Ai6ePEieplGWO8h2S/Ebeqd8RRMMEukMDHCztZcjftHBa+R5107awrpcm5Sl6NPywXQdXVA6PY8f /E2uAr8g==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:51368) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rE8Sd-0002hd-1H; Fri, 15 Dec 2023 13:42:39 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1rE8Sc-0003em-Gu; Fri, 15 Dec 2023 13:42:38 +0000 Date: Fri, 15 Dec 2023 13:42:38 +0000 From: "Russell King (Oracle)" To: Jie Luo Cc: Andrew Lunn , Krzysztof Kozlowski , davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, hkallweit1@gmail.com, corbet@lwn.net, p.zabel@pengutronix.de, f.fainelli@gmail.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org Subject: Re: [PATCH v8 14/14] dt-bindings: net: ar803x: add qca8084 PHY properties Message-ID: References: <20231215074005.26976-1-quic_luoj@quicinc.com> <20231215074005.26976-15-quic_luoj@quicinc.com> <4cb2bd57-f3d3-49f9-9c02-a922fd270572@lunn.ch> 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: Sender: Russell King (Oracle) On Fri, Dec 15, 2023 at 08:16:53PM +0800, Jie Luo wrote: > On 12/15/2023 7:25 PM, Andrew Lunn wrote: > > > The "maxItems: 1" of the property resets is defined in ethernet-phy.yaml > > > that is referenced by qca,ar803x.yaml, but i have 11 reset instances > > > used for qca8084 PHY > > > > 11!?!?? Really? Why? > > > > I assume the order and timer matters, otherwise why would you need > > 11? So the PHY driver needs to handle this, not phylib framework. So > > you will be adding vendor properties to describe all 11 of them. So > > ethernet-phy.yaml does not matter. > > > > Andrew > > Since these resets need to be configured in the special sequence, and > these clocks need to be configured with different clock rate. > > But the clock instance get, the property name is fixed to "clock-names" > according to the function of_parse_clkspec, and the reset property name > is also fixed to "reset-names" from function __of_reset_control_get. I think you need to give more details about this. Where are these 11 resets located? What is the sequence? Why does the PHY driver need to deal with each individual reset? IMHO, a PHY driver should _not_ be dealing with the resets outside of the PHY device itself, and I find it hard to imagine that qca8084 would have 11 external resets. If these are 11 internal resets (to qca8084) then why are you using the reset subsystem, and why do you need to describe them in DT? Surely if they are internal to the PHY, that can be encapsulated within the PHY driver? This is an example of why it is useful to have an _example_ of the use of this binding, because it would answer some of the above questions. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!