From: sashiko-bot@kernel.org
To: "Shivank Sharma" <shivanksharma2376543@gmail.com>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH] doc: ABI: sysfs-class-backlight: update outdated HAL userspace reference
Date: Tue, 21 Jul 2026 13:28:16 +0000 [thread overview]
Message-ID: <20260721132817.482F01F00A3A@smtp.kernel.org> (raw)
In-Reply-To: <20260721132253.256422-1-shivanksharma2376543@gmail.com>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The patch incompletely updates the 'Users: HAL' references, missing the occurrence under 'actual_brightness'.
--
--- Patch [1]: [PATCH] doc: ABI: sysfs-class-backlight: update outdated HAL userspace reference ---
commit 394037cb75e2eb0f44c91b48f3a8658b6c642c06
Author: Shivank Sharma <shivanksharma2376543@gmail.com>
doc: ABI: sysfs-class-backlight: update outdated HAL userspace reference
This commit replaces the deprecated HAL userspace references in the
sysfs-class-backlight ABI documentation with modern userspace references.
> diff --git a/Documentation/ABI/stable/sysfs-class-backlight b/Documentation/ABI/stable/sysfs-class-backlight
> index 40b8c46b95b28..2f60dd3f40a5c 100644
> --- a/Documentation/ABI/stable/sysfs-class-backlight
> +++ b/Documentation/ABI/stable/sysfs-class-backlight
[ ... ]
> @@ -19,7 +19,7 @@ Description:
> are between 0 and max_brightness. This file will also
> show the brightness level stored in the driver, which
> may not be the actual brightness (see actual_brightness).
> -Users: HAL
> +Users: systemd-logind, udev, general userspace.
>
> What: /sys/class/backlight/<backlight>/actual_brightness
> Date: March 2006
[Severity: Low]
Did this patch miss updating the Users field for actual_brightness?
In Documentation/ABI/stable/sysfs-class-backlight, actual_brightness also
has HAL listed as the user, but it appears to have been left out of this
update:
What: /sys/class/backlight/<backlight>/actual_brightness
Date: March 2006
KernelVersion: 2.6.17
Contact: Richard Purdie <rpurdie@rpsys.net>
Description:
Show the actual brightness by querying the hardware. Due
...
Users: HAL
> @@ -40,7 +40,7 @@ KernelVersion: 2.6.12
> Contact: Richard Purdie <rpurdie@rpsys.net>
> Description:
> Maximum brightness for <backlight>.
> -Users: HAL
> +Users: systemd-logind, udev, general userspace.
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260721132253.256422-1-shivanksharma2376543@gmail.com?part=1
next prev parent reply other threads:[~2026-07-21 13:28 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-21 13:22 [PATCH] doc: ABI: sysfs-class-backlight: update outdated HAL userspace reference Shivank Sharma
2026-07-21 13:28 ` sashiko-bot [this message]
2026-07-21 15:19 ` [PATCH v2] doc: ABI: sysfs-class-backlight: update outdated HAL userspace references Shivank Sharma
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=20260721132817.482F01F00A3A@smtp.kernel.org \
--to=sashiko-bot@kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=sashiko-reviews@lists.linux.dev \
--cc=shivanksharma2376543@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.