Received: by 2002:a05:7412:a9a2:b0:e2:908c:2ebd with SMTP id o34csp2635618rdh; Mon, 30 Oct 2023 03:19:38 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFnKxxKWVumuJJz/2O/cma3IxiYo89odCSBHNxrRlSaJq0zJh2nQ1tZRi5RQh2O+fnGIb8X X-Received: by 2002:a17:902:c40b:b0:1cc:3a8a:f197 with SMTP id k11-20020a170902c40b00b001cc3a8af197mr6267237plk.20.1698661178161; Mon, 30 Oct 2023 03:19:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1698661178; cv=none; d=google.com; s=arc-20160816; b=zOOmOaA6MXAYBdzSrd9CEGdQcjXZMKvu+D54yQKKnD7yV78pshWEp8FiVibjHC8NIb buku8STQ2Ji+FcJdv1xP57V3B5WLdQNqHEaPh5YL/0pbB5/MXFqdx8JpQQDEqlP2dSqP aNR8ESxM9GKaamaokTX7qAjU9/oMXm6LIo/HGyHN43dFWdTjZppfVvlruAEX11cIgkZx gp6MM9iSjESiwlKW7o9AmvC1g//TCwpGn6iI/uTVLKj1ygYd5IZD7fXubAhlO6ufl0Na dkhJf5pBOfPpXacZS9QZ4nk0wonXzNt9qrzB+0FTXiUTr+2pJVjSPEOgKYYDiFUYLph8 BhPw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=3hokGVjmSGg3N05F00qEWxL5ZubsMbTLcVLQm4S1uBM=; fh=IgHQrLPsdr1hVJosYJHJpHjj5JtHhVjUE8MVksLn+5s=; b=oAqjHy9EHInsdrxcZmApOset3zmDbAKtqImT+maoUrxrU5K8tpKd3JjKNndlStmHTu 5LQacrqX3JjZupYxXFrY2OrZqyOZvUXH7X+Sazy3irsRlPohTv0DkVmgrsLo82dPCKff IqGStowuKMWKijIzexO4FYxUdxN2POV5d5kf0RSHYyOl4aim0qfV0fKnj6+tSp9Ph8D9 SJ0+gN+0+pm3f5fyUvRMHdUWuNbssi0gnGeB7WCPgQxpYHOnxNG4ukIViMAKKtt2ThHx TH1q8XeyLbgo8gcLrRZg3yRadbzEhjwUv0D/aVqITUtsZbF1XmLcOVRAcpH0NXlq4PY8 E6eA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id e17-20020a170902f1d100b001b8922e82e3si4815736plc.297.2023.10.30.03.19.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Oct 2023 03:19:38 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 219AA80AC8FA; Mon, 30 Oct 2023 03:19:35 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231741AbjJ3KTI convert rfc822-to-8bit (ORCPT + 99 others); Mon, 30 Oct 2023 06:19:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58558 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232637AbjJ3KSB (ORCPT ); Mon, 30 Oct 2023 06:18:01 -0400 Received: from mxout70.expurgate.net (mxout70.expurgate.net [91.198.224.70]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D40897D83; Mon, 30 Oct 2023 03:05:01 -0700 (PDT) Received: from [127.0.0.1] (helo=localhost) by relay.expurgate.net with smtp (Exim 4.92) (envelope-from ) id 1qxP8g-00HZAG-DY; Mon, 30 Oct 2023 11:04:54 +0100 Received: from [195.243.126.94] (helo=securemail.tdt.de) by relay.expurgate.net with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qxP8f-00G2Us-Fh; Mon, 30 Oct 2023 11:04:53 +0100 Received: from securemail.tdt.de (localhost [127.0.0.1]) by securemail.tdt.de (Postfix) with ESMTP id 12DEB240049; Mon, 30 Oct 2023 11:04:53 +0100 (CET) Received: from mail.dev.tdt.de (unknown [10.2.4.42]) by securemail.tdt.de (Postfix) with ESMTP id 64F0A240040; Mon, 30 Oct 2023 11:04:52 +0100 (CET) Received: from localhost.localdomain (unknown [10.2.3.40]) by mail.dev.tdt.de (Postfix) with ESMTPSA id D8160215D4; Mon, 30 Oct 2023 11:04:51 +0100 (CET) From: Florian Eckert To: Eckert.Florian@googlemail.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, pavel@ucw.cz, lee@kernel.org, kabel@kernel.org, u.kleine-koenig@pengutronix.de, m.brock@vanmierlo.com Cc: linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, linux-leds@vger.kernel.org Subject: [Patch v6 0/8] ledtrig-tty: add additional tty state evaluation Date: Mon, 30 Oct 2023 11:04:40 +0100 Message-ID: <20231030100447.63477-1-fe@dev.tdt.de> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Content-Transfer-Encoding: 8BIT X-purgate: clean X-purgate-ID: 151534::1698660294-DD48F3D8-594CF955/0/0 X-purgate-type: clean Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 30 Oct 2023 03:19:35 -0700 (PDT) Changes in v6: ============== This is a paritial rewrite of the changes to make the function for setting the tty evaluation configurable. This change was requested and comment by Greg K-H at the last review [1]. The main changes are. - Split the changes into smaller commits to make reviewing easier. - Use a completion to sync the sysfs and the delay scheduler work on shared variables. - Adding the base-commit to this overview, that reviewer know which base commit I am using. Base branch is: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git Patch [1/7]: This patch is already included in the tty branch [2], but it is still included in this patchsset, so that the following patches of the base-commit branch could be applied correctly. Patch [2/7]: Add a new helper function tty_get_tiocm(). This got already a 'Acked-by: Greg Kroah-Hartman' [3] and is not changed. Patch [3/7]: Add missing of freeing an allocated ttyname buffer on trigger deactivation. This is a memory leak fix patch and should also be backported to the 'stable' branches. Patch [4/7]: As requested by greg k-h this is more a 'dev_warn' instead of a 'dev_info'. This could also be backported to the 'stable' branches if needed. Patch [5/7]: Use a completion to sync for sysfs read/write and the delay scheduler work. I hope I am using the completion correctly. I wasn't sure if I should secure the sysfs read and write access at the same time via a mutex. With this change, the work is also not stopped as it was before when no ttyname was set via sysfs. A tty should always be set when this trigger is used. And is therefore not a problem from my point of view. Patch [6/7]: Make rx tx activitate configurable. In the previous implementation, there was still the ttytrigger flag variable. This flag variable was replaced by individual variables in the data struct. Now these variables can be accessed without masking. The commit was rebased and cleaned up to use the completion implementation. Patch [7/7]: Adding additional trigger line state sources. The commit was also rebased and cleaned up to use the completion implementation. Changes in v5: ============== v5: https://lore.kernel.org/linux-leds/20231023094205.2706812-1-fe@dev.tdt.de/ - Update commit message as request by greg k-h, to make the commit message more generic and not focusing on my use case [1]. - Removing PATCH v4 1/3 from previous set. This has been already applied to tty-testing [2] by greg k-h. - As requested by greq k-h. I have also made the following changes to PATCH v4 3/3 [3]. * Add a comment to the enum that this is already used for bit evaluation and sysfs read and write. * Renaming the variable 'unsigned long mode' to 'unsigned long ttytrigger' in the ledtrig_tty_data structure to make it clearer that the selected triggers are stored there. * Using sysfs_emit() function to dump the requestd ttytrigger to userland. * Also using the kstrtobool() function to write the selected ttytrigger via the sysfs. This values are booleans. - I also removed the function ledtrig_tty_evaluate() from my last patchses PATCH v4 3/3 [3]. The new API tty_get_tiocm() function is only called once now and checked for each ttytrigger bit. Previously this function was called for each bit, which is not necessary. Links: [1] https://lore.kernel.org/linux-leds/2023102115-stock-scrambled-f7d5@gregkh/ [2] https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git/commit/?h=tty-testing&id=838eb763c3e939a8de8d4c55a17ddcce737685c1 [3] https://lore.kernel.org/linux-leds/20231019112809.881730-4-fe@dev.tdt.de/ Changes in v4: ============== v4: https://lore.kernel.org/linux-leds/20231019112809.881730-1-fe@dev.tdt.de/ - Merging patch 3/4 into patch number 4/4 from previous series, because it fixes a problem that does not exist upstream. This was a note from the build robot regarding my change that I added with previous series. This change was never upstream and therefore this is not relevant. - Update the commit message of patch 1/3 of this series, that this commit also changes the 'ndashes' to simple dashes. There were no changes, so I add the 'Reviewed-by' that the commit received before. - With this patchset version I have reworked my implementation for the evaluation of the additional line state, so that this changes becomes smaller. As basis I have used the staged commits from Christian Marangi that makes this changes to the netdev trigger. This has already been applied to 'for-leds-next-next' by Lee Jones. I adapted this to the tty trigger. Convert device attr to macro: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git/commit/drivers/leds/trigger?h=for-leds-next-next&id=509412749002f4bac4c29f2012fff90c08d8afca Unify sysfs and state handling: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git/commit/drivers/leds/trigger?h=for-leds-next-next&id=0fd93ac8582627bee9a3c824489f302dff722881 Changes in v3: ============== v3: https://lore.kernel.org/linux-leds/20231016071332.597654-1-fe@dev.tdt.de/ - Add missing 'kernel test robot' information to the commit message. - Additional information added to the commit message Changes in v2: ============== v2: https://lore.kernel.org/linux-leds/20230928132632.200263-1-fe@dev.tdt.de/ - rename new function from tty_get_mget() to tty_get_tiocm() as requested by 'Jiri Slaby'. - As suggested by 'Jiri Slaby', fixed tabs in function documentation throughout the file '/drivers/tty/tty_io.c' in a separate commit. - Move the variable definition to the top in function 'ledtrig_tty_work()'. This was reported by the 'kernel test robot' after my change in v1. - Also set the 'max_brightness' to 'blink_brightness' if no 'blink_brightness' was set. This fixes a problem at startup when the brightness is still set to 0 and only 'line_*' is evaluated. I looked in the netdev trigger and that's exactly how it's done there. Changes in v1: ============== v1: https://lore.kernel.org/linux-leds/20230926093607.59536-1-fe@dev.tdt.de/ This is a follow-up patchset, based on the mailing list discussion from March 2023 based on the old patchset v8 [1]. I have changed, the LED trigger handling via the sysfs interfaces as suggested by Uwe Kleine-König. Links: [1] https://lore.kernel.org/linux-leds/20230306094113.273988-1-fe@dev.tdt.de/ Florian Eckert (7): tty: whitespaces in descriptions corrected by replacing tabs with spaces tty: add new helper function tty_get_tiocm leds: ledtrig-tty: free allocated ttyname buffer on deactivate leds: ledtrig-tty: change logging if get icount failed leds: ledtrig-tty: replace mutex with completion leds: ledtrig-tty: make rx tx activitate configurable leds: ledtrig-tty: add additional line state evaluation .../ABI/testing/sysfs-class-led-trigger-tty | 56 ++++ drivers/leds/trigger/ledtrig-tty.c | 256 +++++++++++++++--- drivers/tty/tty_io.c | 130 +++++---- include/linux/tty.h | 1 + 4 files changed, 349 insertions(+), 94 deletions(-) base-commit: ffc253263a1375a65fa6c9f62a893e9767fbebfa -- 2.30.2