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 3CAF2C021B3 for ; Mon, 24 Feb 2025 09:19:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=xfJDzNQjykUEtmxffo47EWBg112fF6ajKPMDhb6k0Xo=; b=H4Yf3cbctPOQq9 BsYi6oCe9A97Lmjv42zBo3aSM1pU/RpQUbUj/P5y3I9jN46vUa9ramwRUohiRATLGN11OhYcjhFB1 fKOjG+lCzgpYRTJxhBfiMHOABccpdEB/UEfJmEQ5eDfkQei8fdYKdH6xZNSoA7exbtuMByWqZdZcE sDipHR2ko3EboyfuVOiy8LXNrs8iCl0ukPazvw195zWyIhWFHsC1FW50ODR5bvV0f4cslbCqpqcrL vQYQzMVvVyaxd12jki3e0y03gbcYp+tb6d5yfdFVLpWIEWSvSJUTlG9JeOSjO8aWAX6HYW7dLgpfI JYHrO/MWe1XRg1XuMGjA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tmUcm-0000000CsXG-3uZv; Mon, 24 Feb 2025 09:19:40 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tmUG0-0000000CmhV-3qya; Mon, 24 Feb 2025 08:56:12 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 5CCF05C4CD7; Mon, 24 Feb 2025 08:55:28 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 11564C4CEE6; Mon, 24 Feb 2025 08:56:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740387367; bh=xynIidDgpFBJhwX+rR7LTSuhkkTdXt9jQlEjIf656u4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=D0KVRt9H3ECd8N6iv+XzErxMGS+7KbCRuMJUu+bQMYbTnnPHwLcuWiUTIUxllT1QE RNtw+Szf2Z0laWEeTI3RBd+YjTiMgOJpEkNatKEWUAsX8gPTlbVs+PJO+i2LHoKqgW xRVb97fzyaAnQ3W7spRTcxyEEzpiTHGpCvQHcBkkJxhu4J7Z3GDJ5MWimoLpdibh+L JeZd+LI4viRzL2Zc7a4wG9YVhyXHPpm/XJcfL14/Y117GDC+ExglNclxm12adh3E0x VWtE/qavf6qxZzv+Afn4YFlss1vzH7k0IpmU5kX0X7J/QqALHccZ/eohy9ECzjfXAw FQJrOswTPLMJQ== Date: Mon, 24 Feb 2025 09:56:04 +0100 From: Krzysztof Kozlowski To: Ivaylo Ivanov Cc: Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Alim Akhtar , Philipp Zabel , Abel Vesa , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 3/8] dt-bindings: phy: add samsung,exynos2200-usbcon-phy schema file Message-ID: <20250224-curly-cyber-spaniel-efdc39@krzk-bin> References: <20250223122227.725233-1-ivo.ivanov.ivanov1@gmail.com> <20250223122227.725233-4-ivo.ivanov.ivanov1@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20250223122227.725233-4-ivo.ivanov.ivanov1@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250224_005609_033577_9D1B18B8 X-CRM114-Status: GOOD ( 16.06 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On Sun, Feb 23, 2025 at 02:22:22PM +0200, Ivaylo Ivanov wrote: > The Exynos2200 SoC has a USB controller PHY, which acts as an > intermediary between a USB controller (typically DWC3) and other PHYs > (UTMI, PIPE3). Add a dt-binding schema for it. > > Signed-off-by: Ivaylo Ivanov > --- > .../phy/samsung,exynos2200-usbcon-phy.yaml | 76 +++++++++++++++++++ > 1 file changed, 76 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/samsung,exynos2200-usbcon-phy.yaml You have undocumented dependencies which prevent merging this file. First, dependencies have to be clearly expressed. Second, you should rather decouple the code from header dependencies, otherwise this cannot be merged for current release (just use clocks with long names, without IDs). > > diff --git a/Documentation/devicetree/bindings/phy/samsung,exynos2200-usbcon-phy.yaml b/Documentation/devicetree/bindings/phy/samsung,exynos2200-usbcon-phy.yaml > new file mode 100644 > index 000000000..7d879ec8b > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/samsung,exynos2200-usbcon-phy.yaml > @@ -0,0 +1,76 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/phy/samsung,exynos2200-usbcon-phy.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Exynos2200 USB controller PHY > + > +maintainers: > + - Ivaylo Ivanov > + > +description: > + Exynos2200 USB controller PHY is an intermediary between a USB controller > + (typically DWC3) and other PHYs (UTMI, PIPE3). Isn't this the same as usbdrd phy? see: samsung,usb3-drd-phy.yaml I think there is no PHY between DWC3 and UTMI/PIPE. There is a PHY controller (so the samsung,usb3-drd-phy.yaml) which we call here the phy. Best regards, Krzysztof -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy