Re: [Intel-gfx] [PATCH] drivers: i915: Default max backlight brightness value

2011-10-31 Thread Matthew Garrett
On Fri, Oct 14, 2011 at 04:46:57PM -0700, Simon Que wrote: Sorry, I thought I'd replied to this already. Sorry about that. > +static u32 intel_panel_get_default_backlight_period(struct drm_device *dev) > +{ > + /* The default number of clock cycles in one backlight PWM period. */ > + retu

Re: [Intel-gfx] [PATCH] drivers: i915: Default max backlight brightness value

2011-10-31 Thread Simon Que
On Fri, Oct 14, 2011 at 4:46 PM, Simon Que wrote: > In the native backlight driver, use 4096 (0x1000) as the default backlight > period, and use the period as the default max brightness. > > The default brightness is defined in a separate function that can be > expanded to allow for different defa

Re: [Intel-gfx] [PATCH] drivers: i915: Default max backlight brightness value

2011-10-25 Thread Simon Que
Hi, could I get some feedback on this patch, please? Simon On Fri, Oct 14, 2011 at 4:46 PM, Simon Que wrote: > In the native backlight driver, use 4096 (0x1000) as the default backlight > period, and use the period as the default max brightness. > > The default brightness is defined in a separa

[Intel-gfx] [PATCH] drivers: i915: Default max backlight brightness value

2011-10-14 Thread Simon Que
In the native backlight driver, use 4096 (0x1000) as the default backlight period, and use the period as the default max brightness. The default brightness is defined in a separate function that can be expanded to allow for different defaults on different systems in the future. Change-Id: Ie783b5