From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 63DDAC83038 for ; Tue, 1 Jul 2025 19:00:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:References:Content-Type: Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject:In-Reply-To:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=NV08uxmKxQiLzKQgdc7Udco8g49OXPRD/TVseEZ79Sg=; b=eZKA9GV5dH1/c5pjnDXLH6MPJq ybCUKUMR6oQf5sM0fdNiFSATy9JpEaIiCzYK55Ljn+lVrT5V1aC196MVBA7ra9NJvepjlF2aEJniL JIKBIphuMFMhKBkiYXlPxQdZgB6nd2YLPyH0spfzwEC4/o41618G0ajxNlZ8uyQt9Gry0SsOTvFjP /+HC+IAZejMGdy9syWquzYDD42og5UnjjfNRBhImFcPH6nsUYjNfPRNu86bao832pKC4NTuZZ09xI Ch1hBThmYwVHfyf9+N77Ra3QuMdKioFHtqTolqklmP+8C6Hmf2YKqTOFrof63osirOrhi6/QXa6ek AarHnrWg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWgDS-00000006PDs-1ouY; Tue, 01 Jul 2025 19:00:26 +0000 Received: from [203.254.224.34] (helo=mailout4.samsung.com) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWdt1-0000000631X-0uFk for linux-arm-kernel@lists.infradead.org; Tue, 01 Jul 2025 16:31:14 +0000 Received: from epcas5p1.samsung.com (unknown [182.195.41.39]) by mailout4.samsung.com (KnoxPortal) with ESMTP id 20250701163105epoutp0497bf3b96bfe8221111f980eb82d36d40~OLZ7jCeXI2920729207epoutp04E for ; Tue, 1 Jul 2025 16:31:05 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout4.samsung.com 20250701163105epoutp0497bf3b96bfe8221111f980eb82d36d40~OLZ7jCeXI2920729207epoutp04E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1751387465; bh=NV08uxmKxQiLzKQgdc7Udco8g49OXPRD/TVseEZ79Sg=; h=From:To:Cc:In-Reply-To:Subject:Date:References:From; b=oQ6JxyiSsRb8OkrahQBSHBaP4SgZZP429A35wA+5j7trNIQjUSumf681zPY38ExXE DtAXWVAFrep5Qyi6a4WS7QGBjopcen1fvWduJ1uYz2zPM/3nOWhLMWxmRXHtAk8DnY zHHVzSruJiYtEna+RuSmMWF/ut4IyxrOnuM6kDBk= Received: from epsnrtp01.localdomain (unknown [182.195.42.153]) by epcas5p1.samsung.com (KnoxPortal) with ESMTPS id 20250701163104epcas5p1a96bded3717d037acd2cf0fd71346d8f~OLZ6XQIha2226522265epcas5p1u; Tue, 1 Jul 2025 16:31:04 +0000 (GMT) Received: from epcas5p1.samsung.com (unknown [182.195.38.176]) by epsnrtp01.localdomain (Postfix) with ESMTP id 4bWpS62zKGz6B9m6; Tue, 1 Jul 2025 16:31:02 +0000 (GMT) Received: from epsmtip2.samsung.com (unknown [182.195.34.31]) by epcas5p4.samsung.com (KnoxPortal) with ESMTPA id 20250701133519epcas5p487e7452860a95fd78fe65dea6781a0f4~OJAdeuWxM0466004660epcas5p4i; Tue, 1 Jul 2025 13:35:19 +0000 (GMT) Received: from FDSFTE462 (unknown [107.122.81.248]) by epsmtip2.samsung.com (KnoxPortal) with ESMTPA id 20250701133516epsmtip2097055360b4361ec0ac62f1bab9d901a~OJAap0Ruh1523815238epsmtip2J; Tue, 1 Jul 2025 13:35:16 +0000 (GMT) From: "Shradha Todi" To: "'Krzysztof Kozlowski'" , "'Rob Herring'" Cc: , , , , , , , , , , , , , , , , , , , , In-Reply-To: Subject: RE: [PATCH v2 07/10] dt-bindings: phy: Add PHY bindings support for FSD SoC Date: Tue, 1 Jul 2025 19:05:15 +0530 Message-ID: <02bf01dbea8c$fc835cb0$f58a1610$@samsung.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQFUClgbainc6hQuKSBO0V8ttZVgkwGg1JXJAfs/ltABn1f9rAD4JQ8bAeFf3fS07FvF0A== Content-Language: en-in X-CMS-MailID: 20250701133519epcas5p487e7452860a95fd78fe65dea6781a0f4 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-Sendblock-Type: REQ_APPROVE CMS-TYPE: 105P cpgsPolicy: CPGSC10-541,Y X-CFilter-Loop: Reflected X-CMS-RootMailID: 20250625165319epcas5p3721c19f6e6b482438c62dd1ef784de03 References: <20250625165229.3458-1-shradha.t@samsung.com> <20250625165229.3458-8-shradha.t@samsung.com> <20250627211721.GA153863-robh@kernel.org> <02af01dbea78$24f01310$6ed03930$@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250701_093111_904178_A184B4CD X-CRM114-Status: GOOD ( 37.65 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org > -----Original Message----- > From: Krzysztof Kozlowski > Sent: 01 July 2025 16:55 > To: Shradha Todi ; 'Rob Herring' > Cc: linux-pci=40vger.kernel.org; devicetree=40vger.kernel.org; linux-arm-= kernel=40lists.infradead.org; linux- > samsung-soc=40vger.kernel.org; linux-kernel=40vger.kernel.org; linux-phy= =40lists.infradead.org; linux- > fsd=40tesla.com; mani=40kernel.org; lpieralisi=40kernel.org; kw=40linux.c= om; bhelgaas=40google.com; > jingoohan1=40gmail.com; krzk+dt=40kernel.org; conor+dt=40kernel.org; alim= .akhtar=40samsung.com; > vkoul=40kernel.org; kishon=40kernel.org; arnd=40arndb.de; m.szyprowski=40= samsung.com; > jh80.chung=40samsung.com; pankaj.dubey=40samsung.com > Subject: Re: =5BPATCH v2 07/10=5D dt-bindings: phy: Add PHY bindings supp= ort for FSD SoC >=20 > On 01/07/2025 13:06, Shradha Todi wrote: > > > > > >> -----Original Message----- > >> From: Rob Herring > >> Sent: 28 June 2025 02:47 > >> To: Shradha Todi > >> Cc: linux-pci=40vger.kernel.org; devicetree=40vger.kernel.org; linux-a= rm-kernel=40lists.infradead.org; > > linux- > >> samsung-soc=40vger.kernel.org; linux-kernel=40vger.kernel.org; linux-p= hy=40lists.infradead.org; linux- > >> fsd=40tesla.com; manivannan.sadhasivam=40linaro.org; lpieralisi=40kern= el.org; kw=40linux.com; > >> bhelgaas=40google.com; jingoohan1=40gmail.com; krzk+dt=40kernel.org; c= onor+dt=40kernel.org; > >> alim.akhtar=40samsung.com; vkoul=40kernel.org; kishon=40kernel.org; ar= nd=40arndb.de; > >> m.szyprowski=40samsung.com; jh80.chung=40samsung.com; pankaj.dubey=40s= amsung.com > >> Subject: Re: =5BPATCH v2 07/10=5D dt-bindings: phy: Add PHY bindings s= upport for FSD SoC > >> > >> On Wed, Jun 25, 2025 at 10:22:26PM +0530, Shradha Todi wrote: > >>> Document PHY device tree bindings for Tesla FSD SoCs. > >>> > >>> Signed-off-by: Shradha Todi > >>> --- > >>> .../bindings/phy/samsung,exynos-pcie-phy.yaml =7C 25 +++++++++++++++= ++-- > >>> 1 file changed, 23 insertions(+), 2 deletions(-) > >>> > >>> diff --git a/Documentation/devicetree/bindings/phy/samsung,exynos-pci= e-phy.yaml > >> b/Documentation/devicetree/bindings/phy/samsung,exynos-pcie-phy.yaml > >>> index 41df8bb08ff7..4dc20156cdde 100644 > >>> --- a/Documentation/devicetree/bindings/phy/samsung,exynos-pcie-phy.y= aml > >>> +++ b/Documentation/devicetree/bindings/phy/samsung,exynos-pcie-phy.y= aml > >>> =40=40 -15,10 +15,13 =40=40 properties: > >>> const: 0 > >>> > >>> compatible: > >>> - const: samsung,exynos5433-pcie-phy > >>> + enum: > >>> + - samsung,exynos5433-pcie-phy > >>> + - tesla,fsd-pcie-phy > >>> > >>> reg: > >>> - maxItems: 1 > >>> + minItems: 1 > >>> + maxItems: 2 > >>> > >>> samsung,pmu-syscon: > >>> =24ref: /schemas/types.yaml=23/definitions/phandle > >>> =40=40 -30,6 +33,24 =40=40 properties: > >>> description: phandle for FSYS sysreg interface, used to control > >>> sysreg registers bits for PCIe PHY > >>> > >>> +allOf: > >>> + - if: > >>> + properties: > >>> + compatible: > >>> + contains: > >>> + enum: > >>> + - tesla,fsd-pcie-phy > >>> + then: > >>> + description: > >>> + The PHY controller nodes are represented in the aliases node > >>> + using the following format 'pciephy=7Bn=7D'. Depending on wh= ether > >>> + n is 0 or 1, the phy init sequence is chosen. > >> > >> What? Don't make up your own aliases. > >> > >> If the PHY instances are different, then maybe you need a different > >> compatible. If this is just selecting the PHY mode, you can do that in > >> PHY cells as the mode depends on the consumer. > >> > > > > FSD PCIe has 2 instances of PHY. Both are the same HW Samsung > > PHYs (Therefore share the same register offsets). But the PHY used here >=20 > So same? >=20 > > does not support auto adaptation so we need to tune the PHYs > > according to the use case (considering channel loss, etc). This is why = we >=20 > So not same? Decide. Either it is same or not, cannot be both. >=20 > If you mean that some wiring is different on the board, then how does it > differ in soc thus how it is per-soc property? If these are use-cases, > then how is even suitable for DT? >=20 > I use your Tesla FSD differently and then I exchange DTSI and compatibles= ? >=20 > You are no describing real problem and both binding and your > explanations are vague and imprecise. Binding tells nothing about it, so > it is example of skipping important decisions. >=20 > > have 2 different SW PHY initialization sequence depending on the instan= ce > > number. Do you think having different compatible (something like > > tesla,fsd-pcie-phy0 and tesla,fsd-pcie-phy1) and having phy ID as platf= orm data > > is okay in this case? I actually took reference from files like: >=20 > And in different use case on same soc you are going to reverse > compatibles or instance IDs? > Even though both the PHYs are exactly identical in terms of hardware, they need to be programmed/initialized/configured differently. Sorry for my misuse of the word =22use-case=22. To clarify, these configura= tions will always remain the same for FSD SoC even if you use it differently. I will use different compatibles for them as I understand that it is the be= st option. =20 > > drivers/usb/phy/phy-am335x-control.c >=20 > So you took 15 years old hardware, code and binding as an example. >=20 > No, don't do that ever. >=20 > Anyway, poor choices even in newer code should not drive your design. > Design it properly, describe the hardware. >=20 > > drivers/phy/freescale/phy-fsl-imx8-mipi-dphy.c > > who use alias to differentiate between register offsets for instances. >=20 >=20 >=20 > Best regards, > Krzysztof