gpiolib: demote the hogging log messages to debug
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 5 Jun 2023 12:52:48 +0000 (14:52 +0200)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Wed, 7 Jun 2023 08:25:15 +0000 (10:25 +0200)
Drivers should be silent when they work correctly. There's no reason to
emit info messages when GPIO lines are hogged. Demote the message to
debug.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Suggested-by: Kent Gibson <warthog618@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/gpio/gpiolib.c
drivers/of/unittest.c

index bb81d94..d5f23c2 100644 (file)
@@ -4236,7 +4236,7 @@ int gpiod_hog(struct gpio_desc *desc, const char *name,
        /* Mark GPIO as hogged so it can be identified and removed later */
        set_bit(FLAG_IS_HOGGED, &desc->flags);
 
-       gpiod_info(desc, "hogged as %s%s\n",
+       gpiod_dbg(desc, "hogged as %s%s\n",
                (dflags & GPIOD_FLAGS_BIT_DIR_OUT) ? "output" : "input",
                (dflags & GPIOD_FLAGS_BIT_DIR_OUT) ?
                  (dflags & GPIOD_FLAGS_BIT_DIR_VAL) ? "/high" : "/low" : "");
index 2191c01..0060334 100644 (file)
@@ -1849,19 +1849,19 @@ static void __init of_unittest_overlay_gpio(void)
         * driver is registered
         */
 
-       EXPECT_BEGIN(KERN_INFO,
+       EXPECT_BEGIN(KERN_DEBUG,
                     "gpio-<<int>> (line-B-input): hogged as input\n");
 
-       EXPECT_BEGIN(KERN_INFO,
+       EXPECT_BEGIN(KERN_DEBUG,
                     "gpio-<<int>> (line-A-input): hogged as input\n");
 
        ret = platform_driver_register(&unittest_gpio_driver);
        if (unittest(ret == 0, "could not register unittest gpio driver\n"))
                return;
 
-       EXPECT_END(KERN_INFO,
+       EXPECT_END(KERN_DEBUG,
                   "gpio-<<int>> (line-A-input): hogged as input\n");
-       EXPECT_END(KERN_INFO,
+       EXPECT_END(KERN_DEBUG,
                   "gpio-<<int>> (line-B-input): hogged as input\n");
 
        unittest(probe_pass_count + 2 == unittest_gpio_probe_pass_count,
@@ -1888,7 +1888,7 @@ static void __init of_unittest_overlay_gpio(void)
        probe_pass_count = unittest_gpio_probe_pass_count;
        chip_request_count = unittest_gpio_chip_request_count;
 
-       EXPECT_BEGIN(KERN_INFO,
+       EXPECT_BEGIN(KERN_DEBUG,
                     "gpio-<<int>> (line-D-input): hogged as input\n");
 
        /* overlay_gpio_03 contains gpio node and child gpio hog node */
@@ -1896,7 +1896,7 @@ static void __init of_unittest_overlay_gpio(void)
        unittest(overlay_data_apply("overlay_gpio_03", NULL),
                 "Adding overlay 'overlay_gpio_03' failed\n");
 
-       EXPECT_END(KERN_INFO,
+       EXPECT_END(KERN_DEBUG,
                   "gpio-<<int>> (line-D-input): hogged as input\n");
 
        unittest(probe_pass_count + 1 == unittest_gpio_probe_pass_count,
@@ -1935,7 +1935,7 @@ static void __init of_unittest_overlay_gpio(void)
         *   - processing gpio for overlay_gpio_04b
         */
 
-       EXPECT_BEGIN(KERN_INFO,
+       EXPECT_BEGIN(KERN_DEBUG,
                     "gpio-<<int>> (line-C-input): hogged as input\n");
 
        /* overlay_gpio_04b contains child gpio hog node */
@@ -1943,7 +1943,7 @@ static void __init of_unittest_overlay_gpio(void)
        unittest(overlay_data_apply("overlay_gpio_04b", NULL),
                 "Adding overlay 'overlay_gpio_04b' failed\n");
 
-       EXPECT_END(KERN_INFO,
+       EXPECT_END(KERN_DEBUG,
                   "gpio-<<int>> (line-C-input): hogged as input\n");
 
        unittest(chip_request_count + 1 == unittest_gpio_chip_request_count,