From patchwork Mon Mar 17 02:22:19 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Craig McQueen X-Patchwork-Id: 874843 Received: from sender4-op-o12.zoho.com (sender4-op-o12.zoho.com [136.143.188.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3255F41C71; Mon, 17 Mar 2025 02:22:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.12 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742178167; cv=pass; b=GwNmpFiFhaLYkx8IG7wQqb83MRnkeOsjyqys6oaYXxMde4U2pydNAJXA0A1oI8h/qRt9x6X9WNTaat+Ozy7Ww7XesgeCvp4PsV9rvlrI3ePFvLquewXy9kgN7zATLPtS3EsxXW0dIgiillj82M61M2gWyxlemNaRrgKaB66UFlk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742178167; c=relaxed/simple; bh=aBfLkQWIwSuIWYfhmzIDTIbcoGkKK6yDe0udX6G2ZA0=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=FFmcx+bOBYHxcEYwO+5dfdGkXULAFkpWQoSg/THg+f+8nyTCAfVtRd7vaK9kiodKsWd2ItXm0W9csSuZsSczkfhOb/LlQC+CyuAUg1U193Rnc0uRegcra2p2k5i1OwKFQOC/Rlqgc+DZfoGYD1C43IXgwdXlzsV/yl8/LHM58dE= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=mcqueen.au; spf=pass smtp.mailfrom=mcqueen.au; dkim=pass (1024-bit key) header.d=mcqueen.au header.i=craig@mcqueen.au header.b=gfqcsxAh; arc=pass smtp.client-ip=136.143.188.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=mcqueen.au Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=mcqueen.au Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=mcqueen.au header.i=craig@mcqueen.au header.b="gfqcsxAh" ARC-Seal: i=1; a=rsa-sha256; t=1742178161; cv=none; d=zohomail.com; s=zohoarc; b=NQNxU8yhK/DmkLDGMy3Uk5p4xKyHTejcpIFvrJLexcCSKmcrhwT+DWQFfxFO3NkxeNohy21dlY8mcZgFDeVUhHwREbuVm+uo5Mc35KoFyKjQNEzZ/PA30mpFtEXytAaI9cET2XCLBFedgWeS/DJ2NXvEykBqtZthRSmZ7H+ClCw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1742178161; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:MIME-Version:Message-ID:Subject:Subject:To:To:Message-Id:Reply-To; bh=7vKlPXhw27JYjmqjWNXuz7ItPZuCC9iS4GLpaLAzwCs=; b=GinTY8JOXZPf77g/ubYpWfokawzU8gBOfzxiPurFMckq52TIvHCS+uKCHJgDS81eOYtxlenJOn9tvAc1wvm5ZFGe62YN7Vs0pRgVXgRd/eRf8DWIRDa/THi2LNkp0deikIPb5GMGsS6CsuaFqtj+7Q4JcZEn2MFzRC3+gAwLeqI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=mcqueen.au; spf=pass smtp.mailfrom=craig@mcqueen.au; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1742178161; s=zmail; d=mcqueen.au; i=craig@mcqueen.au; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:MIME-Version:Content-Transfer-Encoding:Message-Id:Reply-To; bh=7vKlPXhw27JYjmqjWNXuz7ItPZuCC9iS4GLpaLAzwCs=; b=gfqcsxAh1Ign1RsOz59oPfr0QO42cqo1sS9/l/vtplh+vnWXX+k8ZTFcNR9kVFda ZWQNq2JuNeRc0av+PSEJJkOxcmIPly1AJ3k9BLAEEt49camwpFvRfyxpuSlEqqVTmCg n8W+M0l/UCTHbeo6gLysVQ1Rk3wz7TT2dYXoo5rA= Received: by mx.zohomail.com with SMTPS id 1742178158336158.59315349470774; Sun, 16 Mar 2025 19:22:38 -0700 (PDT) From: Craig McQueen To: linux-leds@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Craig McQueen Subject: [PATCH v3 1/2] leds: uleds documentation fixes Date: Mon, 17 Mar 2025 13:22:19 +1100 Message-ID: <20250317022220.423966-1-craig@mcqueen.au> X-Mailer: git-send-email 2.48.1 Precedence: bulk X-Mailing-List: linux-leds@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ZohoMailClient: External * Add max_brightness field to struct for setup. * Reads are a signed int value, not a single unsigned byte value. Signed-off-by: Craig McQueen --- Documentation/leds/uleds.rst | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/leds/uleds.rst b/Documentation/leds/uleds.rst index 83221098009c..4077745dae00 100644 --- a/Documentation/leds/uleds.rst +++ b/Documentation/leds/uleds.rst @@ -17,16 +17,16 @@ structure to it (found in kernel public header file linux/uleds.h):: struct uleds_user_dev { char name[LED_MAX_NAME_SIZE]; + int max_brightness; }; A new LED class device will be created with the name given. The name can be any valid sysfs device node name, but consider using the LED class naming convention of "devicename:color:function". -The current brightness is found by reading a single byte from the character -device. Values are unsigned: 0 to 255. Reading will block until the brightness -changes. The device node can also be polled to notify when the brightness value -changes. +The current brightness is found by reading an int value from the character +device. Reading will block until the brightness changes. The device node can +also be polled to notify when the brightness value changes. The LED class device will be removed when the open file handle to /dev/uleds is closed. From patchwork Mon Mar 17 02:22:20 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Craig McQueen X-Patchwork-Id: 874232 Received: from sender4-op-o12.zoho.com (sender4-op-o12.zoho.com [136.143.188.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2344F13BAE3; Mon, 17 Mar 2025 02:22:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.12 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742178172; cv=pass; b=Ew9eMES7yhWQxXMESF0lmqXzuCnBCmMRNLT8fTsgsbGGPO5WqPN9wf2o+sJuhe0Jmnhqeh0j58Q8MYs+jJMqEU4v3/JmC3fXCYR0PiPnUIdA8+3rftNYq11II8VVTaB9git9OVrX6v5SlEumE67w/5N6g8I5ym9xX6plqZwOPhI= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742178172; c=relaxed/simple; bh=fe/N6scJUnkyC0tv9cka7xpBgCET89kSUJvPplhqvuk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ZdeVIjlZxmOt7/a/2rig33hUsQj3mS/9bbVdJv/7Fp0mzcoKerLLR/TeS4sg3UiyHkumqhgSBGYAcfY5wLLZ/xtePA7cKXx2IiBiqFXOzw9AQ08x78t/m08OgNGpHmGIJ4KHvcNAM8GnkFHRuJzTztvCpgTCddGXAN7i04vxRmk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=mcqueen.au; spf=pass smtp.mailfrom=mcqueen.au; dkim=pass (1024-bit key) header.d=mcqueen.au header.i=craig@mcqueen.au header.b=CLrfCwIt; arc=pass smtp.client-ip=136.143.188.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=mcqueen.au Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=mcqueen.au Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=mcqueen.au header.i=craig@mcqueen.au header.b="CLrfCwIt" ARC-Seal: i=1; a=rsa-sha256; t=1742178165; cv=none; d=zohomail.com; s=zohoarc; b=hVDlv0ZP8i/tZBjf8qmjb/U0QHvtbha/HsgLh6T4ePr+V2nzjc2uTiWf/tDnfN95ApXCZ2Oh8UhgVc7FcIbVWkoRQepyIzG7loQgkoWc0y3bNOUnH4bSLWhwEtfOX2zatQSOyeqG0YYFpta8S3KFyp4fy2uFzQW2bbtMjDClnsw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1742178165; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=SvoGmevVsPWvAPDqjhbxgnG97H5n7WSHg7XC7RIzW00=; b=DPmXFnQUOQMP+xCVxmtLYRC9d69GuJZ6X+KJ/8smn36ZvdhpRBjNmOQ0+zk4RjcZ6G++YxGU3Ax947E0uO9qVJbT/etklINZDrLHV05GSVbsQC7Bj01TISdY3DHCS4nA8CAwXV4kheL2mvxZN3wFoNO4pNvFjZdc4oJSFSLm9uY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=mcqueen.au; spf=pass smtp.mailfrom=craig@mcqueen.au; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1742178165; s=zmail; d=mcqueen.au; i=craig@mcqueen.au; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Transfer-Encoding:Message-Id:Reply-To; bh=SvoGmevVsPWvAPDqjhbxgnG97H5n7WSHg7XC7RIzW00=; b=CLrfCwItpXwDIVBh7s8onYQiyPf4Ge/xyF3oZU6C/WIu/fVSn07FBPTRhs/R5B+q rh+Cq7CMHyTHv17ipNge0SpDdxKcAU0pexihDUNDdQ7/yGlnI7WcJmWeKD7TP1zyRiZ VF5vQXyEB0RuVBH2M/NYPjN7eexeVkPQLT06ax+g= Received: by mx.zohomail.com with SMTPS id 1742178162057185.40239168533162; Sun, 16 Mar 2025 19:22:42 -0700 (PDT) From: Craig McQueen To: linux-leds@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Craig McQueen Subject: [PATCH v3 2/2] leds: Add some setup ioctls to uleds driver Date: Mon, 17 Mar 2025 13:22:20 +1100 Message-ID: <20250317022220.423966-2-craig@mcqueen.au> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250317022220.423966-1-craig@mcqueen.au> References: <20250317022220.423966-1-craig@mcqueen.au> Precedence: bulk X-Mailing-List: linux-leds@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ZohoMailClient: External * Add an ioctl for setup as an alternative to doing a write. This is similar to the ioctl that was added to uinput for setup. * Add an ioctl to set a default trigger for the LED. Signed-off-by: Craig McQueen --- Documentation/leds/uleds.rst | 6 ++ drivers/leds/uleds.c | 125 ++++++++++++++++++++++++++++------- include/uapi/linux/uleds.h | 30 ++++++++- 3 files changed, 137 insertions(+), 24 deletions(-) diff --git a/Documentation/leds/uleds.rst b/Documentation/leds/uleds.rst index 4077745dae00..0512e577d63a 100644 --- a/Documentation/leds/uleds.rst +++ b/Documentation/leds/uleds.rst @@ -24,6 +24,12 @@ A new LED class device will be created with the name given. The name can be any valid sysfs device node name, but consider using the LED class naming convention of "devicename:color:function". +Alternatively, setup can be done with an ioctl call, passing a pointer to +the structure. + +There is also an ioctl call to configure a default trigger for the LED, by +passing a pointer to a structure containing the name of a trigger. + The current brightness is found by reading an int value from the character device. Reading will block until the brightness changes. The device node can also be polled to notify when the brightness value changes. diff --git a/drivers/leds/uleds.c b/drivers/leds/uleds.c index 374a841f18c3..598e376a4b1b 100644 --- a/drivers/leds/uleds.c +++ b/drivers/leds/uleds.c @@ -6,6 +6,7 @@ * * Based on uinput.c: Aristeu Sergio Rozanski Filho */ +#include #include #include #include @@ -25,13 +26,14 @@ enum uleds_state { }; struct uleds_device { - struct uleds_user_dev user_dev; - struct led_classdev led_cdev; - struct mutex mutex; - enum uleds_state state; - wait_queue_head_t waitq; - int brightness; - bool new_data; + struct uleds_user_dev user_dev; + struct uleds_user_trigger default_trigger; + struct led_classdev led_cdev; + struct mutex mutex; + enum uleds_state state; + wait_queue_head_t waitq; + int brightness; + bool new_data; }; static struct miscdevice uleds_misc; @@ -70,15 +72,17 @@ static int uleds_open(struct inode *inode, struct file *file) return 0; } -static ssize_t uleds_write(struct file *file, const char __user *buffer, - size_t count, loff_t *ppos) +static bool is_led_name_valid(const char *name) { - struct uleds_device *udev = file->private_data; - const char *name; - int ret; + return ((name[0] != '\0') && + (strcmp(name, ".") != 0) && + (strcmp(name, "..") != 0) && + (strchr(name, '/') == NULL)); +} - if (count == 0) - return 0; +static int dev_setup(struct uleds_device *udev, const char __user *buffer) +{ + int ret; ret = mutex_lock_interruptible(&udev->mutex); if (ret) @@ -89,20 +93,13 @@ static ssize_t uleds_write(struct file *file, const char __user *buffer, goto out; } - if (count != sizeof(struct uleds_user_dev)) { - ret = -EINVAL; - goto out; - } - if (copy_from_user(&udev->user_dev, buffer, sizeof(struct uleds_user_dev))) { ret = -EFAULT; goto out; } - name = udev->user_dev.name; - if (!name[0] || !strcmp(name, ".") || !strcmp(name, "..") || - strchr(name, '/')) { + if (!is_led_name_valid(udev->user_dev.name)) { ret = -EINVAL; goto out; } @@ -120,7 +117,6 @@ static ssize_t uleds_write(struct file *file, const char __user *buffer, udev->new_data = true; udev->state = ULEDS_STATE_REGISTERED; - ret = count; out: mutex_unlock(&udev->mutex); @@ -128,6 +124,23 @@ static ssize_t uleds_write(struct file *file, const char __user *buffer, return ret; } +static ssize_t uleds_write(struct file *file, const char __user *buffer, + size_t count, loff_t *ppos) +{ + struct uleds_device *udev = file->private_data; + int ret; + + if (count == 0) + return 0; + if (count != sizeof(struct uleds_user_dev)) + return -EINVAL; + + ret = dev_setup(udev, buffer); + if (ret < 0) + return ret; + return count; +} + static ssize_t uleds_read(struct file *file, char __user *buffer, size_t count, loff_t *ppos) { @@ -179,6 +192,71 @@ static __poll_t uleds_poll(struct file *file, poll_table *wait) return 0; } +/* + * Trigger name validation: Allow only alphanumeric, hyphen or underscore. + */ +static bool is_trigger_name_valid(const char *name) +{ + size_t i; + + if (name[0] == '\0') + return false; + + for (i = 0; i < TRIG_NAME_MAX; i++) { + if (name[i] == '\0') + break; + if (!isalnum(name[i]) && name[i] != '-' && name[i] != '_') + return false; + } + /* Length check and avoid any special names. */ + return ((i < TRIG_NAME_MAX) && + (strcmp(name, "default") != 0)); +} + +static long uleds_ioctl(struct file *file, unsigned int cmd, unsigned long arg) +{ + struct uleds_device *udev = file->private_data; + struct uleds_user_trigger default_trigger; + int retval = 0; + + switch (cmd) { + case ULEDS_IOC_DEV_SETUP: + retval = dev_setup(udev, (const char __user *)arg); + break; + + case ULEDS_IOC_SET_DEFAULT_TRIGGER: + retval = copy_from_user(&default_trigger, + (struct uleds_user_trigger __user *)arg, + sizeof(default_trigger)); + if (retval) + return retval; + retval = mutex_lock_interruptible(&udev->mutex); + if (retval) + return retval; + if (default_trigger.name[0] == '\0') { + udev->led_cdev.default_trigger = NULL; + } else { + if (!is_trigger_name_valid(default_trigger.name)) { + mutex_unlock(&udev->mutex); + return -EINVAL; + } + memcpy(&udev->default_trigger, &default_trigger, + sizeof(udev->default_trigger)); + udev->led_cdev.default_trigger = udev->default_trigger.name; + } + if (udev->state == ULEDS_STATE_REGISTERED) + led_trigger_set_default(&udev->led_cdev); + mutex_unlock(&udev->mutex); + break; + + default: + retval = -ENOIOCTLCMD; + break; + } + + return retval; +} + static int uleds_release(struct inode *inode, struct file *file) { struct uleds_device *udev = file->private_data; @@ -200,6 +278,7 @@ static const struct file_operations uleds_fops = { .read = uleds_read, .write = uleds_write, .poll = uleds_poll, + .unlocked_ioctl = uleds_ioctl, }; static struct miscdevice uleds_misc = { diff --git a/include/uapi/linux/uleds.h b/include/uapi/linux/uleds.h index 4d32a39965f8..0e9861a8c31f 100644 --- a/include/uapi/linux/uleds.h +++ b/include/uapi/linux/uleds.h @@ -15,11 +15,39 @@ #ifndef _UAPI__ULEDS_H_ #define _UAPI__ULEDS_H_ -#define LED_MAX_NAME_SIZE 64 +#define LED_MAX_NAME_SIZE 64 +#define ULEDS_TRIGGER_MAX_NAME_SIZE 64 struct uleds_user_dev { char name[LED_MAX_NAME_SIZE]; int max_brightness; }; +struct uleds_user_trigger { + char name[ULEDS_TRIGGER_MAX_NAME_SIZE]; +}; + + +/* ioctl commands */ + +#define ULEDS_IOC_MAGIC 'l' + +/* + * Initial setup. + * E.g.: + * int retval; + * struct uleds_user_dev dev_setup = { "mainboard:green:battery", 255 }; + * retval = ioctl(fd, ULEDS_IOC_DEV_SETUP, &dev_setup); + */ +#define ULEDS_IOC_DEV_SETUP _IOW(ULEDS_IOC_MAGIC, 0x01, struct uleds_user_dev) + +/* + * Set the default trigger. + * E.g.: + * int retval; + * struct uleds_user_trigger default_trigger = { "battery" }; + * retval = ioctl(fd, ULEDS_IOC_SET_DEFAULT_TRIGGER, &default_trigger); + */ +#define ULEDS_IOC_SET_DEFAULT_TRIGGER _IOW(ULEDS_IOC_MAGIC, 0x02, struct uleds_user_trigger) + #endif /* _UAPI__ULEDS_H_ */