From: Hans de Goede <hdegoede@redhat.com>
To: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Pavel Machek" <pavel@ucw.cz>, "Lee Jones" <lee@kernel.org>,
"Kate Hsuan" <hpa@redhat.com>,
"Sebastian Reichel" <sre@kernel.org>
Cc: "Hans de Goede" <hdegoede@redhat.com>,
platform-driver-x86@vger.kernel.org,
"André Apitzsch" <git@apitzsch.eu>,
"Heiner Kallweit" <hkallweit1@gmail.com>,
linux-leds@vger.kernel.org, linux-pm@vger.kernel.org
Subject: [PATCH v2 0/3] power: supply: power-supply-leds: Add activate() callback to triggers
Date: Fri, 31 May 2024 15:46:59 +0200 [thread overview]
Message-ID: <20240531134702.166145-1-hdegoede@redhat.com> (raw)
Hi All,
This series adds an activate callback to the power-supply LED triggers to
ensure that power-supply LEDs get the correct initial value when the LED
gets registered after the power_supply has been registered.
Note that in 6.10-rc1 commit 822c91e72eac ("leds: trigger: Store brightness
set by led_trigger_event()") already solves this for the case were
the trigger only changes only the brightness. Since the power-supply
triggers can also set blinking and colors of mc LEDs that is not enough.
Patches 1-2 do some refactoring / prep work and patch 3 adds the actual
activate callback.
This series applies on top of 6.10-rc1 + "[PATCH v10 0/6] KTD2026 indicator
LED for X86 Xiaomi Pad2" that series should show up in
https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git/
soon, since this is also LED related I guess this could be merged
through the LED tree too. Or Lee could provide an IB branch for
Sebastian to merge that branch + this series through the psy tree.
Sebastian, what has your preference ?
Regards,
Hans
Hans de Goede (3):
power: supply: power-supply-leds: Add
power_supply_[un]register_led_trigger()
power: supply: power-supply-leds: Share trig pointer for online and
charging_full
power: supply: power-supply-leds: Add activate() callback to triggers
drivers/power/supply/power_supply_leds.c | 178 +++++++++++++----------
include/linux/power_supply.h | 9 +-
2 files changed, 103 insertions(+), 84 deletions(-)
--
2.45.1
next reply other threads:[~2024-05-31 13:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-31 13:46 Hans de Goede [this message]
2024-05-31 13:47 ` [PATCH v2 1/3] power: supply: power-supply-leds: Add power_supply_[un]register_led_trigger() Hans de Goede
2024-05-31 13:47 ` [PATCH v2 2/3] power: supply: power-supply-leds: Share trig pointer for online and charging_full Hans de Goede
2024-05-31 13:47 ` [PATCH v2 3/3] power: supply: power-supply-leds: Add activate() callback to triggers Hans de Goede
2024-05-31 14:08 ` [PATCH v2 0/3] " Andy Shevchenko
2024-06-05 21:38 ` Sebastian Reichel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240531134702.166145-1-hdegoede@redhat.com \
--to=hdegoede@redhat.com \
--cc=andy@kernel.org \
--cc=git@apitzsch.eu \
--cc=hkallweit1@gmail.com \
--cc=hpa@redhat.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=lee@kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=platform-driver-x86@vger.kernel.org \
--cc=sre@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox