On Thu, Jul 26, 2012 at 10:51:18AM +0800, Bryan Wu wrote:
> OK, thanks, I've merge this patch through my tree and can I add your ack to
> it?
Sure,
Acked-by: Mark Brown
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
On Thu, Jul 26, 2012 at 10:51:18AM +0800, Bryan Wu wrote:
OK, thanks, I've merge this patch through my tree and can I add your ack to
it?
Sure,
Acked-by: Mark Brown broo...@opensource.wolfsonmicro.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a
On Thu, Jul 26, 2012 at 2:43 AM, Mark Brown
wrote:
> On Wed, Jul 25, 2012 at 12:46:56PM +0800, Bryan Wu wrote:
>
>> I'm going to Ack this driver and Mark will you merge this as whole patchset?
>
>> Acked-by: Bryan Wu
>
> It's an MFD so Samuel would normally apply if it were going via the MFD
>
On Wed, Jul 25, 2012 at 12:46:56PM +0800, Bryan Wu wrote:
> I'm going to Ack this driver and Mark will you merge this as whole patchset?
> Acked-by: Bryan Wu
It's an MFD so Samuel would normally apply if it were going via the MFD
tree, though if the dependencies are correct there should be no
On Wed, Jul 25, 2012 at 12:46:56PM +0800, Bryan Wu wrote:
I'm going to Ack this driver and Mark will you merge this as whole patchset?
Acked-by: Bryan Wu bryan...@canonical.com
It's an MFD so Samuel would normally apply if it were going via the MFD
tree, though if the dependencies are correct
On Thu, Jul 26, 2012 at 2:43 AM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
On Wed, Jul 25, 2012 at 12:46:56PM +0800, Bryan Wu wrote:
I'm going to Ack this driver and Mark will you merge this as whole patchset?
Acked-by: Bryan Wu bryan...@canonical.com
It's an MFD so Samuel would
On Tue, Jul 24, 2012 at 8:55 PM, Mark Brown
wrote:
> On Tue, Jul 24, 2012 at 08:23:00AM +0800, Bryan Wu wrote:
>> On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
>
>> > If the work is flushed then the state that userspace thought was set
>> > when the driver is removed will actually be set before the
On Tue, Jul 24, 2012 at 08:23:00AM +0800, Bryan Wu wrote:
> On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
> > If the work is flushed then the state that userspace thought was set
> > when the driver is removed will actually be set before the driver is
> > removed. This is fairly minor but might be
On Tue, Jul 24, 2012 at 08:23:00AM +0800, Bryan Wu wrote:
On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
If the work is flushed then the state that userspace thought was set
when the driver is removed will actually be set before the driver is
removed. This is fairly minor but might be
On Tue, Jul 24, 2012 at 8:55 PM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
On Tue, Jul 24, 2012 at 08:23:00AM +0800, Bryan Wu wrote:
On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
If the work is flushed then the state that userspace thought was set
when the driver is removed will
On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
wrote:
> On Sat, Jul 21, 2012 at 02:48:49AM +0800, Bryan Wu wrote:
>
>> Actually cancel_work_sync() is quite similar to flush_work_sync()
>> here. For the timer thing, in fact it is NULL when cancel_work_sync()
>> call __cancel_work_timer().
>
>> And
On Mon, Jul 23, 2012 at 2:19 AM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
On Sat, Jul 21, 2012 at 02:48:49AM +0800, Bryan Wu wrote:
Actually cancel_work_sync() is quite similar to flush_work_sync()
here. For the timer thing, in fact it is NULL when cancel_work_sync()
call
On Sat, Jul 21, 2012 at 02:48:49AM +0800, Bryan Wu wrote:
> Actually cancel_work_sync() is quite similar to flush_work_sync()
> here. For the timer thing, in fact it is NULL when cancel_work_sync()
> call __cancel_work_timer().
> And Mark, do you have any advice about the flush_work_sync() and
>
On Sat, Jul 21, 2012 at 02:48:49AM +0800, Bryan Wu wrote:
Actually cancel_work_sync() is quite similar to flush_work_sync()
here. For the timer thing, in fact it is NULL when cancel_work_sync()
call __cancel_work_timer().
And Mark, do you have any advice about the flush_work_sync() and
On Fri, Jul 20, 2012 at 11:49 PM, Shuah Khan wrote:
> On Fri, 2012-07-20 at 08:43 +, Kim, Milo wrote:
>> TI LP8788 PMU has the current sink as the keyboard led driver.
>> The brightness is controlled by the i2c commands.
>> Configurable parameters can be defined in the platform side.
>>
>>
Hi,
I think a mutex_unlock is missed out,
On Fri, Jul 20, 2012 at 2:28 PM, Kim, Milo wrote:
> +
> +static void lp8788_led_work(struct work_struct *work)
> +{
> + struct lp8788_led *led = container_of(work, struct lp8788_led, work);
> + enum lp8788_isink_number num = led->isink_num;
On Fri, 2012-07-20 at 08:43 +, Kim, Milo wrote:
> TI LP8788 PMU has the current sink as the keyboard led driver.
> The brightness is controlled by the i2c commands.
> Configurable parameters can be defined in the platform side.
>
> Patch v2.
> (a) use workqueue on changing the brightness
>
>
TI LP8788 PMU has the current sink as the keyboard led driver.
The brightness is controlled by the i2c commands.
Configurable parameters can be defined in the platform side.
Patch v2.
(a) use workqueue on changing the brightness
(b) use mutex_lock/unlock when the brightness is set
and the
TI LP8788 PMU has the current sink as the keyboard led driver.
The brightness is controlled by the i2c commands.
Configurable parameters can be defined in the platform side.
Patch v2.
(a) use workqueue on changing the brightness
(b) use mutex_lock/unlock when the brightness is set
and the
On Fri, 2012-07-20 at 08:43 +, Kim, Milo wrote:
TI LP8788 PMU has the current sink as the keyboard led driver.
The brightness is controlled by the i2c commands.
Configurable parameters can be defined in the platform side.
Patch v2.
(a) use workqueue on changing the brightness
(b) use
Hi,
I think a mutex_unlock is missed out,
On Fri, Jul 20, 2012 at 2:28 PM, Kim, Milo milo@ti.com wrote:
+
+static void lp8788_led_work(struct work_struct *work)
+{
+ struct lp8788_led *led = container_of(work, struct lp8788_led, work);
+ enum lp8788_isink_number num =
On Fri, Jul 20, 2012 at 11:49 PM, Shuah Khan shuahk...@gmail.com wrote:
On Fri, 2012-07-20 at 08:43 +, Kim, Milo wrote:
TI LP8788 PMU has the current sink as the keyboard led driver.
The brightness is controlled by the i2c commands.
Configurable parameters can be defined in the platform
22 matches
Mail list logo