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

[Andrew Morton]

On Tue, 24 Jan 2012 22:24:02 -0800
"Kim, Milo" <Milo.Kim@xxxxxx> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/