From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Thompson Date: Thu, 01 Feb 2018 11:06:42 +0000 Subject: Re: [PATCH 2/5] backlight: lp855x: document sysfs attributes Message-Id: <20180201110642.qyjwyrdwglevlk5f@oak.lan> List-Id: References: <09cd01366bf3b5c3030ad25450b46a9a3bc8a747.1516978005.git.aishpant@gmail.com> In-Reply-To: <09cd01366bf3b5c3030ad25450b46a9a3bc8a747.1516978005.git.aishpant@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Aishwarya Pant Cc: Milo Kim , linux-fbdev@vger.kernel.org, Bartlomiej Zolnierkiewicz , Jingoo Han , Jonathan Corbet , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Julia Lawall , Greg KH , Lee Jones On Fri, Jan 26, 2018 at 08:23:00PM +0530, Aishwarya Pant wrote: > Add documentation for sysfs interface of Texas Instruments lp855x > backlight driver by reading code and looking through git commit logs. > > Signed-off-by: Aishwarya Pant > --- > .../ABI/testing/sysfs-class-backlight-lp855x | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > create mode 100644 Documentation/ABI/testing/sysfs-class-backlight-lp855x > > diff --git a/Documentation/ABI/testing/sysfs-class-backlight-lp855x b/Documentation/ABI/testing/sysfs-class-backlight-lp855x > new file mode 100644 > index 000000000000..5b9d38958875 > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-class-backlight-lp855x > @@ -0,0 +1,19 @@ > +sysfs interface for Texas Instruments lp855x backlight driver > +------------------------------------------------------------ > + > +What: /sys/class/backlight//chip_id > +Date: Mar, 2012 > +KernelVersion: v3.10 > +Contact: Milo Kim > +Description: > + (RO) Displays the lp8755x chip id. Possible values are: > + lp8550, lp8551, lp8552, lp8553, lp8555, lp8556 or lp8557. > + > + > +What: /sys/class/backlight//bl_ctl_mode > +Date: Mar, 2012 > +KernelVersion: v3.10 > +Contact: Linux kernel mailing list > +Description: > + (RO) Displays whether the brightness is controlled by the pwm > + input("pwm based) or the i2c register("register based"). Since like the first patch in the series, these look like they are for debugging. That thread is still active but whatever the outcome of that thread please assume it also applies to this patch. Daniel.