[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CABVgOSkzui0e_1LhrMAeJG1RMFqyq472XNHod0tS452mUg2m+g@mail.gmail.com>
Date: Thu, 13 Jul 2023 12:17:50 +0800
From: David Gow <davidgow@...gle.com>
To: Mark Brown <broonie@...nel.org>
Cc: Brendan Higgins <brendan.higgins@...ux.dev>,
linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org,
kunit-dev@...glegroups.com
Subject: Re: [PATCH] regmap: Provide user selectable option to enable regmap
On Wed, 12 Jul 2023 at 07:22, Mark Brown <broonie@...nel.org> wrote:
>
> Since apparently enabling all the KUnit tests shouldn't enable any new
> subsystems it is hard to enable the regmap KUnit tests in normal KUnit
> testing scenarios that don't enable any drivers. Add a Kconfig option
> to help with this and include it in the KUnit all tests config.
>
> Signed-off-by: Mark Brown <broonie@...nel.org>
> ---
Thanks: I wasn't totally convinced of the extra CONFIG_REGMAP_BUILD
option at first, but having thought about it some more (and seen the
ASoC topology one, which has always annoyed me for being a pain to
enable), I've come around.
Reviewed-by: David Gow <davidgow@...gle.com>
Cheers,
-- David
> drivers/base/regmap/Kconfig | 12 +++++++++++-
> tools/testing/kunit/configs/all_tests.config | 2 ++
> 2 files changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig
> index 0db2021f7477..b1affac70d5d 100644
> --- a/drivers/base/regmap/Kconfig
> +++ b/drivers/base/regmap/Kconfig
> @@ -4,7 +4,7 @@
> # subsystems should select the appropriate symbols.
>
> config REGMAP
> - bool "Register Map support" if KUNIT_ALL_TESTS
> + bool
> default y if (REGMAP_I2C || REGMAP_SPI || REGMAP_SPMI || REGMAP_W1 || REGMAP_AC97 || REGMAP_MMIO || REGMAP_IRQ || REGMAP_SOUNDWIRE || REGMAP_SOUNDWIRE_MBQ || REGMAP_SCCB || REGMAP_I3C || REGMAP_SPI_AVMM || REGMAP_MDIO || REGMAP_FSI)
> select IRQ_DOMAIN if REGMAP_IRQ
> select MDIO_BUS if REGMAP_MDIO
> @@ -23,6 +23,16 @@ config REGMAP_KUNIT
> default KUNIT_ALL_TESTS
> select REGMAP_RAM
>
> +config REGMAP_BUILD
> + bool "Enable regmap build"
> + depends on KUNIT
> + select REGMAP
> + help
> + This option exists purely to allow the regmap KUnit tests to
> + be enabled without having to enable some driver that uses
> + regmap due to unfortunate issues with how KUnit tests are
> + normally enabled.
> +
> config REGMAP_AC97
> tristate
>
> diff --git a/tools/testing/kunit/configs/all_tests.config b/tools/testing/kunit/configs/all_tests.config
> index 0393940c706a..873f3e06ccad 100644
> --- a/tools/testing/kunit/configs/all_tests.config
> +++ b/tools/testing/kunit/configs/all_tests.config
> @@ -33,5 +33,7 @@ CONFIG_DAMON_PADDR=y
> CONFIG_DEBUG_FS=y
> CONFIG_DAMON_DBGFS=y
>
> +CONFIG_REGMAP_BUILD=y
> +
> CONFIG_SECURITY=y
> CONFIG_SECURITY_APPARMOR=y
>
> ---
> base-commit: 06c2afb862f9da8dc5efa4b6076a0e48c3fbaaa5
> change-id: 20230701-regmap-kunit-enable-a08718e77dd4
>
> Best regards,
> --
> Mark Brown <broonie@...nel.org>
>
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4003 bytes)
Powered by blists - more mailing lists