Re: [PATCH v17 01/10] video: backlight: Add helpers to enable and disable backlight

From: Lee Jones
Date: Fri Jan 19 2018 - 09:26:24 EST


On Fri, 19 Jan 2018, Daniel Thompson wrote:

> On Fri, Jan 19, 2018 at 10:39:16AM +0000, Meghana Madhyastha wrote:
> > Add helper functions backlight_enable and backlight_disable to
> > enable/disable a backlight device. These helper functions can
> > then be used by different drm and tinydrm drivers to avoid
> > repetition of code and also to enforce a uniform and consistent
> > way to enable/disable a backlight device.
> >
> > Signed-off-by: Meghana Madhyastha <meghana.madhyastha@xxxxxxxxx>
>
> Acked-by: Daniel Thompson <daniel.thompson@xxxxxxxxxx>
>
> BTW Lee J. looks PRs and merges for the backlight tree, I'll leave it
> to Lee and whoevers-DRM-tree-this-would-naturally-land-in to decide the
> merge path.

So the choices are; either I take the set and send out a pull-request
which can be subsequently merged into the DRM repo, OR they are
applied into an immutable branch in the DRM tree and I can pull that
in instead.

Don't really mind which.

--
Lee Jones
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog