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 723E1C4332F for ; Tue, 20 Dec 2022 22:15:30 +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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=eMH5nY9YgCNg2p1HOh5anUkXaSLRwddJr+KmtkkGi8I=; b=iI602C4FK3xAnbTn6ctWTbDi7i s6A1fFCfuPBo7iuZmD5W0jeAYHm+JL5N4wqbMt2j4Gz95BH93WvXUe15r99O0gZGRQj8QN6JO6xRX WnuQaEG5B3gy9IB5S8BtXZ5Qjc1UfM9XiNkj7PGtE2ronkx4+/45T2nUXCXGE/Pd5i19ait2MPxaE 93akHby7rWVMlvpZQ+yFWRnXVI6eU2p8J8IaHIzpVHjfxjkT17Vo7O2bGnNsu7zWPkHYs1aAPjRYg Nfax0bWRmPA9Gla84SiUnWZfX0HawEjK8xx4UmCgDv/LQYaXZ0ofhZsa7WDC/8gvbqYwCeyndjXEU V/Z3vfwg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p7ktL-004yPn-1K; Tue, 20 Dec 2022 22:15:19 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p7ktI-004yLf-NG for linux-riscv@lists.infradead.org; Tue, 20 Dec 2022 22:15:18 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id EF6CC615BF; Tue, 20 Dec 2022 22:15:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 01C14C433EF; Tue, 20 Dec 2022 22:15:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1671574515; bh=/hXbsdZlpPJ092brHoQdM/vjvfB74qXktqitZJsVKZ4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=qHSQ22X8muE+CwnZqqGtUeYuyyk0+PRIYvmBqqB8UfP0aE6/Dr8/0Ux5uZVL4t5d2 rsVF/fIglHI3PPCJ+5AjFHCgsiOCBZztKBEKWsLOtGVHxC2vzjy31PTGrvrAYgdxXN QJLWqmxKi91UZnaUSSVQSqA77Y2PgTFn/X9Exu4/9bhDHM6pL1tfrXVF+/+QKuVhxq UbPE9OdVKs+VvBDmfGDMVTRXHDPfz2m5YllUGciZ9ZfLiPh+Fyu/1lvyCSNV0CznO/ dyzxJ5sU1xA9UpIZCohRQ1G4bKgC49gPDg0jsziHV7HrJGD7pKjSqzyRAJxg7Hp/hz +mVICg5lrS7jA== Date: Tue, 20 Dec 2022 22:15:10 +0000 From: Conor Dooley To: Hal Feng Cc: linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, linux-clk@vger.kernel.org, Palmer Dabbelt , Rob Herring , Krzysztof Kozlowski , Stephen Boyd , Michael Turquette , Philipp Zabel , Emil Renner Berthing , linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 03/11] reset: Create subdirectory for StarFive drivers Message-ID: References: <20221220005054.34518-1-hal.feng@starfivetech.com> <20221220005054.34518-4-hal.feng@starfivetech.com> MIME-Version: 1.0 In-Reply-To: <20221220005054.34518-4-hal.feng@starfivetech.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221220_141516_883210_B103AE50 X-CRM114-Status: GOOD ( 24.31 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============3246392486820128979==" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org --===============3246392486820128979== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="zh1KmNPWKa/82vyz" Content-Disposition: inline --zh1KmNPWKa/82vyz Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Dec 20, 2022 at 08:50:46AM +0800, Hal Feng wrote: > From: Emil Renner Berthing >=20 > This moves the StarFive JH7100 reset driver to a new subdirectory in > preparation for adding more StarFive reset drivers. >=20 > Signed-off-by: Emil Renner Berthing > Signed-off-by: Hal Feng > --- > MAINTAINERS | 2 +- > drivers/reset/Kconfig | 8 +------- > drivers/reset/Makefile | 2 +- > drivers/reset/starfive/Kconfig | 8 ++++++++ > drivers/reset/starfive/Makefile | 2 ++ > drivers/reset/{ =3D> starfive}/reset-starfive-jh7100.c | 0 > 6 files changed, 13 insertions(+), 9 deletions(-) > create mode 100644 drivers/reset/starfive/Kconfig > create mode 100644 drivers/reset/starfive/Makefile > rename drivers/reset/{ =3D> starfive}/reset-starfive-jh7100.c (100%) >=20 > diff --git a/MAINTAINERS b/MAINTAINERS > index fd90403c33bd..117024b52d06 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -19651,7 +19651,7 @@ STARFIVE JH7100 RESET CONTROLLER DRIVER > M: Emil Renner Berthing > S: Maintained > F: Documentation/devicetree/bindings/reset/starfive,jh7100-reset.yaml > -F: drivers/reset/reset-starfive-jh7100.c > +F: drivers/reset/starfive/reset-starfive-jh7100.c > F: include/dt-bindings/reset/starfive-jh7100.h > =20 > STATIC BRANCH/CALL > diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig > index de176c2fbad9..1e8e1c4954cd 100644 > --- a/drivers/reset/Kconfig > +++ b/drivers/reset/Kconfig > @@ -232,13 +232,6 @@ config RESET_SOCFPGA > This enables the reset driver for the SoCFPGA ARMv7 platforms. This > driver gets initialized early during platform init calls. > =20 > -config RESET_STARFIVE_JH7100 > - bool "StarFive JH7100 Reset Driver" > - depends on SOC_STARFIVE || COMPILE_TEST > - default SOC_STARFIVE > - help > - This enables the reset controller driver for the StarFive JH7100 SoC. > - > config RESET_SUNPLUS > bool "Sunplus SoCs Reset Driver" if COMPILE_TEST > default ARCH_SUNPLUS > @@ -320,6 +313,7 @@ config RESET_ZYNQ > help > This enables the reset controller driver for Xilinx Zynq SoCs. > =20 > +source "drivers/reset/starfive/Kconfig" > source "drivers/reset/sti/Kconfig" > source "drivers/reset/hisilicon/Kconfig" > source "drivers/reset/tegra/Kconfig" > diff --git a/drivers/reset/Makefile b/drivers/reset/Makefile > index 3e7e5fd633a8..fee17a0e3a16 100644 > --- a/drivers/reset/Makefile > +++ b/drivers/reset/Makefile > @@ -1,6 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0 > obj-y +=3D core.o > obj-y +=3D hisilicon/ > +obj-$(CONFIG_SOC_STARFIVE) +=3D starfive/ > obj-$(CONFIG_ARCH_STI) +=3D sti/ > obj-$(CONFIG_ARCH_TEGRA) +=3D tegra/ > obj-$(CONFIG_RESET_A10SR) +=3D reset-a10sr.o > @@ -30,7 +31,6 @@ obj-$(CONFIG_RESET_RZG2L_USBPHY_CTRL) +=3D reset-rzg2l-= usbphy-ctrl.o > obj-$(CONFIG_RESET_SCMI) +=3D reset-scmi.o > obj-$(CONFIG_RESET_SIMPLE) +=3D reset-simple.o > obj-$(CONFIG_RESET_SOCFPGA) +=3D reset-socfpga.o > -obj-$(CONFIG_RESET_STARFIVE_JH7100) +=3D reset-starfive-jh7100.o > obj-$(CONFIG_RESET_SUNPLUS) +=3D reset-sunplus.o > obj-$(CONFIG_RESET_SUNXI) +=3D reset-sunxi.o > obj-$(CONFIG_RESET_TI_SCI) +=3D reset-ti-sci.o > diff --git a/drivers/reset/starfive/Kconfig b/drivers/reset/starfive/Kcon= fig > new file mode 100644 > index 000000000000..cddebdba7177 > --- /dev/null > +++ b/drivers/reset/starfive/Kconfig > @@ -0,0 +1,8 @@ > +# SPDX-License-Identifier: GPL-2.0-only > + > +config RESET_STARFIVE_JH7100 > + bool "StarFive JH7100 Reset Driver" > + depends on SOC_STARFIVE || COMPILE_TEST > + default SOC_STARFIVE You could in theory drop the default that I added & replace it with a y, since the subdir is gated by the symbol. I don't really care though tbh. The movement seems fine to me.. Reviewed-by: Conor Dooley Thanks, Conor. > + help > + This enables the reset controller driver for the StarFive JH7100 SoC. > diff --git a/drivers/reset/starfive/Makefile b/drivers/reset/starfive/Mak= efile > new file mode 100644 > index 000000000000..670d049423f5 > --- /dev/null > +++ b/drivers/reset/starfive/Makefile > @@ -0,0 +1,2 @@ > +# SPDX-License-Identifier: GPL-2.0 > +obj-$(CONFIG_RESET_STARFIVE_JH7100) +=3D reset-starfive-jh7100.o > diff --git a/drivers/reset/reset-starfive-jh7100.c b/drivers/reset/starfi= ve/reset-starfive-jh7100.c > similarity index 100% > rename from drivers/reset/reset-starfive-jh7100.c > rename to drivers/reset/starfive/reset-starfive-jh7100.c > --=20 > 2.38.1 >=20 >=20 --zh1KmNPWKa/82vyz Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCY6Iz7gAKCRB4tDGHoIJi 0kwrAQD7r8oFTH5a1/NbDQ7ZWs5mjPOXmg4cAFGBu3Vqn0qaLgD+L5pc3W4etT/y 5+k6f4WdYdsVirsXBaQu5wFN5I19LgQ= =9xHl -----END PGP SIGNATURE----- --zh1KmNPWKa/82vyz-- --===============3246392486820128979== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv --===============3246392486820128979==--