linux-kernel - Re: [PATCH v2] backlight: new backlight driver for LP855x devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 26 Jan 2012 15:46:51 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	"Kim, Milo" <Milo.Kim@...com>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Richard Purdie" <rpurdie@...ys.net>
Subject: Re: [PATCH v2] backlight: new backlight driver for LP855x devices

On Tue, 24 Jan 2012 22:24:02 -0800
"Kim, Milo" <Milo.Kim@...com> wrote:

> This patch supports TI LP8550/LP8551/LP8852/LP8553/LP8556 backlight driver.
> 
> The brightness can be controlled by the I2C or PWM input.
> The lp855x driver provides both modes.
> For the PWM control, pwm-specific functions can be defined in the platform data.
> And some information can be read via the sysfs(lp855x device attributes).
> 
> For the details, please refer to 'Documentation/backlight/lp855x-driver.txt'.
> 

I had a few little problems here.

- For some reason my email client doesn't like MIME quoted-printable
  and cannot correctly extract the plain text.  It appears that
  globally deleting =\n and globally replacing =3D with = fixes things.
  Ho hum.  I really should go see if this has been fixed, or hassle
  the sylpheed developer(s).

- The driver generates a great storm of scripts/checkpatch.pl
  whitespace warnings.  Things like using eight spaces instead of a
  tab.  These really should be fixed, so please take a look at the
  checkpatch output then resend?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/