[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aLbBMumD9mJK2z5U@smile.fi.intel.com>
Date: Tue, 2 Sep 2025 13:04:34 +0300
From: Andy Shevchenko <andriy.shevchenko@...el.com>
To: Jean-François Lessard <jefflessard3@...il.com>
Cc: Andy Shevchenko <andy@...nel.org>,
Geert Uytterhoeven <geert@...ux-m68k.org>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/5] auxdisplay: linedisp: add num_chars sysfs attribute
On Sun, Aug 31, 2025 at 10:00:27PM -0400, Jean-François Lessard wrote:
> Add a read-only 'num_chars' sysfs attribute to report display digit count.
>
> The num_chars attribute provides essential capability information to
> userspace applications that need to know display dimensions before writing
> messages, complementing the existing message and scroll controls.
...
> &dev_attr_message.attr,
> &dev_attr_scroll_step_ms.attr,
> + &dev_attr_num_chars.attr,
> &dev_attr_map_seg7.attr,
> &dev_attr_map_seg14.attr,
It looks like we have two groups of the attributes here, can we keep the upper
one sorted? (Put the new attribute one line above?)
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists