regulator: dbx500-prcmu: Remove unused function dbx500_regulator_testcase()
authorLee Jones <lee.jones@linaro.org>
Thu, 25 Jun 2020 16:36:08 +0000 (17:36 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 25 Jun 2020 19:11:12 +0000 (20:11 +0100)
There isn't any code present within the current kernel to
override this 'weak' function.  Besides returning '0', which
is never checked anyway, the whole function appears to be
superfluous.

Consequently fixes W=1 warning:

 drivers/regulator/dbx500-prcmu.c:113:27: warning: no previous prototype for ‘dbx500_regulator_testcase’ [-Wmissing-prototypes]
 113 | int __attribute__((weak)) dbx500_regulator_testcase(
 | ^~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Lee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20200625163614.4001403-5-lee.jones@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/dbx500-prcmu.c

index f604c8d..c3ad6aa 100644 (file)
@@ -110,13 +110,6 @@ static int ux500_regulator_status_show(struct seq_file *s, void *p)
 }
 DEFINE_SHOW_ATTRIBUTE(ux500_regulator_status);
 
-int __attribute__((weak)) dbx500_regulator_testcase(
-       struct dbx500_regulator_info *regulator_info,
-       int num_regulators)
-{
-       return 0;
-}
-
 int
 ux500_regulator_debug_init(struct platform_device *pdev,
        struct dbx500_regulator_info *regulator_info,
@@ -152,7 +145,6 @@ ux500_regulator_debug_init(struct platform_device *pdev,
        if (!rdebug.state_after_suspend)
                goto exit_free;
 
-       dbx500_regulator_testcase(regulator_info, num_regulators);
        return 0;
 
 exit_free: