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 D2633EA4FC3 for ; Mon, 23 Feb 2026 14:38:45 +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=IHPCAqz04ZhicfySZ1qXfUQEQNz25vsAAaVYGCCzOfU=; b=o/P+tQcaU2LUCD TNI8xRhroH6A++gnftjzeN6VRGMN2/nqtw3L+wT39k23Ua00kkgiewo68+NfnIvTU8RxhtzQksnQt 6Od5hIWo/SRMZ+vcxOEhtSetAGSO5OYVGd9vwpLiV+2w768xK+s9Cqz9fT5OCqBbCygIa0mT/as5r 7oX8GD+YFqpe6qhzlOfhobZhPPuAROUK+p1+N6yZoUn1w5eYMb0e7iyIC5GSzbVocyd/wVaZjXGv8 x212kJZJ20wl9Kpayg9Ch3p9FFotfxQeoutUVnrlPwj/fAAnTXAyqxf77siHmxGYMKwvoQZGPon8J hmg+E5sMrVqBGgyTua3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vuX5B-00000000U96-1W7F; Mon, 23 Feb 2026 14:38:45 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vuX58-00000000U8h-1nru for linux-phy@lists.infradead.org; Mon, 23 Feb 2026 14:38:43 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 427D843D95; Mon, 23 Feb 2026 14:38:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CE43EC116C6; Mon, 23 Feb 2026 14:38:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1771857521; bh=kdP5EttDcynG8swMrF1Ca5QeEDVK8y3yg/juEgym6hc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UbFxIzkFzN9D02sfhiEIunEhNGdXiavpqUHuKBIlZ+vNrLxE8yqI386DKuaMAvHjS r8A3efG+gXeysmaOZM3aVOJ/I96kgNYt200QKfrWFpk/mpCj2jSEtYw77lNzsTCaiU lijVgwJfZ1LkVkyHJ/afdumn4IARYNL7dlKaLABG3UICbLEsvtC4jj4/NTpiXTU2XN 2XMyxVxFZEQL4NRXdtg/pRrSxKGUUZip0KBwWmFpQmuJxC+etA364ntlD07lQosraO AejuVn+3kmAYy12+wzAhJHkLkFVJHFaMoqj9t8elzcgStZL76iYYL1llTgOVgBVeWW I3kusZ/wicdeQ== Date: Mon, 23 Feb 2026 22:38:38 +0800 From: Yixun Lan To: Vinod Koul Cc: linux-phy@lists.infradead.org, Neil Armstrong , Vladimir Oltean , Alex Elder , Ze Huang , spacemit@lists.linux.dev Subject: Re: [PATCH] phy: move spacemit pcie driver to its subfolder Message-ID: <20260223143838-GKA220920@kernel.org> References: <20260223064240.386617-1-vkoul@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20260223064240.386617-1-vkoul@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260223_063842_566897_A567CA2F X-CRM114-Status: GOOD ( 16.81 ) 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 Hi Vinod, On 12:12 Mon 23 Feb , Vinod Koul wrote: > Commit fe4bc1a08638 ("phy: spacemit: support K1 USB2.0 PHY controller") > created spacemit subfolder with usb driver while commit 57e920b92724 > ("phy: spacemit: Introduce PCIe/combo PHY") added pcie driver in phy > folder. Move latter into spacemit subfolder and rename file to > phy-k1-pcie.c > Make sense, this looks better, thanks Reviewed-by: Yixun Lan > Signed-off-by: Vinod Koul > --- > drivers/phy/Kconfig | 12 ------------ > drivers/phy/Makefile | 1 - > drivers/phy/spacemit/Kconfig | 12 ++++++++++++ > drivers/phy/spacemit/Makefile | 1 + > .../phy-k1-pcie.c} | 0 > 5 files changed, 13 insertions(+), 13 deletions(-) > rename drivers/phy/{phy-spacemit-k1-pcie.c => spacemit/phy-k1-pcie.c} (100%) > > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index 02467dfd4fb0..c0574e44f0a3 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -134,18 +134,6 @@ config PHY_NXP_PTN3222 > schemes. It supports all three USB 2.0 data rates: Low Speed, Full > Speed and High Speed. > > -config PHY_SPACEMIT_K1_PCIE > - tristate "PCIe and combo PHY driver for the SpacemiT K1 SoC" > - depends on ARCH_SPACEMIT || COMPILE_TEST > - depends on COMMON_CLK > - depends on HAS_IOMEM > - depends on OF > - select GENERIC_PHY > - default ARCH_SPACEMIT > - help > - Enable support for the PCIe and USB 3 combo PHY and two > - PCIe-only PHYs used in the SpacemiT K1 SoC. > - > source "drivers/phy/allwinner/Kconfig" > source "drivers/phy/amlogic/Kconfig" > source "drivers/phy/apple/Kconfig" > diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile > index a648c2e02a83..2773d596e543 100644 > --- a/drivers/phy/Makefile > +++ b/drivers/phy/Makefile > @@ -16,7 +16,6 @@ obj-$(CONFIG_PHY_SNPS_EUSB2) += phy-snps-eusb2.o > obj-$(CONFIG_USB_LGM_PHY) += phy-lgm-usb.o > obj-$(CONFIG_PHY_AIROHA_PCIE) += phy-airoha-pcie.o > obj-$(CONFIG_PHY_NXP_PTN3222) += phy-nxp-ptn3222.o > -obj-$(CONFIG_PHY_SPACEMIT_K1_PCIE) += phy-spacemit-k1-pcie.o > obj-$(CONFIG_GENERIC_PHY) += allwinner/ \ > amlogic/ \ > apple/ \ > diff --git a/drivers/phy/spacemit/Kconfig b/drivers/phy/spacemit/Kconfig > index 0136aee2e8a2..50b0005acf66 100644 > --- a/drivers/phy/spacemit/Kconfig > +++ b/drivers/phy/spacemit/Kconfig > @@ -2,6 +2,18 @@ > # > # Phy drivers for SpacemiT platforms > # > +config PHY_SPACEMIT_K1_PCIE > + tristate "PCIe and combo PHY driver for the SpacemiT K1 SoC" > + depends on ARCH_SPACEMIT || COMPILE_TEST > + depends on COMMON_CLK > + depends on HAS_IOMEM > + depends on OF > + select GENERIC_PHY > + default ARCH_SPACEMIT > + help > + Enable support for the PCIe and USB 3 combo PHY and two > + PCIe-only PHYs used in the SpacemiT K1 SoC. > + > config PHY_SPACEMIT_K1_USB2 > tristate "SpacemiT K1 USB 2.0 PHY support" > depends on (ARCH_SPACEMIT || COMPILE_TEST) && OF > diff --git a/drivers/phy/spacemit/Makefile b/drivers/phy/spacemit/Makefile > index fec0b425a948..a821a21d6142 100644 > --- a/drivers/phy/spacemit/Makefile > +++ b/drivers/phy/spacemit/Makefile > @@ -1,2 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > +obj-$(CONFIG_PHY_SPACEMIT_K1_PCIE) += phy-k1-pcie.o > obj-$(CONFIG_PHY_SPACEMIT_K1_USB2) += phy-k1-usb2.o > diff --git a/drivers/phy/phy-spacemit-k1-pcie.c b/drivers/phy/spacemit/phy-k1-pcie.c > similarity index 100% > rename from drivers/phy/phy-spacemit-k1-pcie.c > rename to drivers/phy/spacemit/phy-k1-pcie.c > -- > 2.43.0 > > -- Yixun Lan (dlan) -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy