Daniel Thompson <[email protected]> writes: > On Fri, Oct 13, 2017 at 09:42:49PM +0200, Robert Jarzmik wrote: >> Add entries to query the panel ASIC to acquire the model number and the >> status field. This comes in handy to check that the SPI connection is >> functional and that the panel's firmware is alive. >> >> Signed-off-by: Robert Jarzmik <[email protected]> > > This feels to me somewhat like debug information being pushed into sysfs. > There are very few backlight drivers that add additional sysfs attributes > meaning the extra info here is not commonly pushed into the ABI for a > backlight and knowing the info in these properties does not really > help userspace to *use* the hardware. > > Does this kind of thing really belong in sysfs? Ok, as you wish. Let's drop this patch from the serie for now.
Cheers. -- Robert
