Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Jingoo Han
On Thursday, January 23, 2014 6:28 PM, Liu Ying wrote: > On 01/23/2014 01:44 PM, Jingoo Han wrote: > > On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: > >> On Mon, 20 Jan 2014, Liu Ying wrote: > >>> We don't have to turn backlight on/off everytime a blanking > >>> or unblanking event

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Liu Ying
On 01/23/2014 05:27 PM, Liu Ying wrote: > On 01/23/2014 01:44 PM, Jingoo Han wrote: >> On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: >>> On Mon, 20 Jan 2014, Liu Ying wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Liu Ying
On 01/23/2014 01:44 PM, Jingoo Han wrote: > On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: >> On Mon, 20 Jan 2014, Liu Ying wrote: >>> We don't have to turn backlight on/off everytime a blanking >>> or unblanking event comes because the backlight status may >>> have already been what

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Liu Ying
On 01/23/2014 01:44 PM, Jingoo Han wrote: On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Liu Ying
On 01/23/2014 05:27 PM, Liu Ying wrote: On 01/23/2014 01:44 PM, Jingoo Han wrote: On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-23 Thread Jingoo Han
On Thursday, January 23, 2014 6:28 PM, Liu Ying wrote: On 01/23/2014 01:44 PM, Jingoo Han wrote: On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Jingoo Han
On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: > On Mon, 20 Jan 2014, Liu Ying wrote: > > We don't have to turn backlight on/off everytime a blanking > > or unblanking event comes because the backlight status may > > have already been what we want. Another thought is that one > >

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Liu Ying
On 01/22/2014 05:35 PM, Jani Nikula wrote: > On Mon, 20 Jan 2014, Liu Ying wrote: >> We don't have to turn backlight on/off everytime a blanking >> or unblanking event comes because the backlight status may >> have already been what we want. Another thought is that one >> backlight device may be

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Jani Nikula
On Mon, 20 Jan 2014, Liu Ying wrote: > We don't have to turn backlight on/off everytime a blanking > or unblanking event comes because the backlight status may > have already been what we want. Another thought is that one > backlight device may be shared by multiple framebuffers. We > don't hope

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Jani Nikula
On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is that one backlight device may be shared by multiple

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Liu Ying
On 01/22/2014 05:35 PM, Jani Nikula wrote: On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is that one backlight

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-22 Thread Jingoo Han
On Wednesday, January 22, 2014 6:36 PM, Jani Nikula wrote: On Mon, 20 Jan 2014, Liu Ying ying@freescale.com wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-21 Thread Jingoo Han
On Wednesday, January 22, 2014 2:04 PM, Liu Ying wrote: > > Ping... > > Regards, > Liu Ying Please, don't send the ping within 2 days. It is not a good practice. You sent the v1 patch 6 months ago. However, why I should review the patch within 2 days? Please wait. Best regards, Jingoo Han >

Re:[PATCH v2] backlight: turn backlight on/off when necessary

2014-01-21 Thread Liu Ying
Ping... Regards, Liu Ying On 01/20/2014 12:52 PM, Liu Ying wrote: > We don't have to turn backlight on/off everytime a blanking > or unblanking event comes because the backlight status may > have already been what we want. Another thought is that one > backlight device may be shared by multiple

Re:[PATCH v2] backlight: turn backlight on/off when necessary

2014-01-21 Thread Liu Ying
Ping... Regards, Liu Ying On 01/20/2014 12:52 PM, Liu Ying wrote: We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is that one backlight device may be shared by multiple

Re: [PATCH v2] backlight: turn backlight on/off when necessary

2014-01-21 Thread Jingoo Han
On Wednesday, January 22, 2014 2:04 PM, Liu Ying wrote: Ping... Regards, Liu Ying Please, don't send the ping within 2 days. It is not a good practice. You sent the v1 patch 6 months ago. However, why I should review the patch within 2 days? Please wait. Best regards, Jingoo Han On

[PATCH v2] backlight: turn backlight on/off when necessary

2014-01-19 Thread Liu Ying
We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is that one backlight device may be shared by multiple framebuffers. We don't hope blanking one of the framebuffers may turn the

[PATCH v2] backlight: turn backlight on/off when necessary

2014-01-19 Thread Liu Ying
We don't have to turn backlight on/off everytime a blanking or unblanking event comes because the backlight status may have already been what we want. Another thought is that one backlight device may be shared by multiple framebuffers. We don't hope blanking one of the framebuffers may turn the