[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250514162807.708010-4-lee@kernel.org>
Date: Wed, 14 May 2025 17:27:55 +0100
From: Lee Jones <lee@...nel.org>
To: lee@...nel.org,
bettyzhou@...gle.com,
ynaffit@...gle.com,
tkjos@...gle.com,
jacek.anaszewski@...il.com,
Pavel Machek <pavel@...nel.org>,
linux-leds@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: [PATCH 4/5] leds: led-test: Provide test for registration with a name that is too long
Insist on legacy (non-DT) registration and provide a default_label and
devicename that when concatenated together results in a device name that
is unacceptably long.
Signed-off-by: Lee Jones <lee@...nel.org>
---
drivers/leds/led-test.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/drivers/leds/led-test.c b/drivers/leds/led-test.c
index 3d71c7a23e21..741dc0269515 100644
--- a/drivers/leds/led-test.c
+++ b/drivers/leds/led-test.c
@@ -123,11 +123,28 @@ static void led_test_class_init_data_missing_devicename(struct kunit *test)
KUNIT_EXPECT_EQ(test, ret, -EINVAL);
}
+static void led_test_class_init_data_name_too_long(struct kunit *test)
+{
+ struct led_test_ddata *ddata = test->priv;
+ struct led_classdev *cdev = &ddata->cdev;
+ struct device *dev = ddata->dev;
+ int ret;
+
+ struct led_init_data init_data = {
+ .devicename = "led-test-devicename-very-long-names-fail",
+ .default_label = "led-test-label-also-very-long-names-fail",
+ };
+
+ ret = led_classdev_register_ext(dev, cdev, &init_data);
+ KUNIT_EXPECT_EQ(test, ret, -E2BIG);
+}
+
static struct kunit_case led_test_cases[] = {
KUNIT_CASE(led_test_class_register),
KUNIT_CASE(led_test_class_add_lookup_and_get),
KUNIT_CASE(led_test_class_init_data_missing_default_label),
KUNIT_CASE(led_test_class_init_data_missing_devicename),
+ KUNIT_CASE(led_test_class_init_data_name_too_long),
{ }
};
--
2.49.0.1045.g170613ef41-goog
Powered by blists - more mailing lists