Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-10-07 Thread Andy Shevchenko
On Fri, Jul 29, 2016 at 6:49 PM, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > >

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-10-07 Thread Andy Shevchenko
On Fri, Jul 29, 2016 at 6:49 PM, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > > I couldn't manage to get the

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-16 Thread Mika Westerberg
On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > +static const struct i2c_device_id mshw0011_id[] = { > + { "MSHW0011:00", 0 }, You should not use ACPI device names in non-ACPI matching table. I think you can just leave this table empty. > + { } > +}; >

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-16 Thread Mika Westerberg
On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > +static const struct i2c_device_id mshw0011_id[] = { > + { "MSHW0011:00", 0 }, You should not use ACPI device names in non-ACPI matching table. I think you can just leave this table empty. > + { } > +}; >

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-15 Thread Sebastian Reichel
Hi, On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > > I couldn't manage

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-15 Thread Sebastian Reichel
Hi, On Fri, Jul 29, 2016 at 05:49:26PM +0200, Benjamin Tissoires wrote: > MSHW0011 replaces the battery firmware by using ACPI operation regions. > The values have been obtained by reverse engineering, and are subject to > errors. Looks like it works on overall pretty well. > > I couldn't manage

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-01 Thread Benjamin Tissoires
On Jul 30 2016 or thereabouts, kbuild test robot wrote: > Hi, > > [auto build test ERROR on battery/master] > [also build test ERROR on v4.7 next-20160729] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: >

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-08-01 Thread Benjamin Tissoires
On Jul 30 2016 or thereabouts, kbuild test robot wrote: > Hi, > > [auto build test ERROR on battery/master] > [also build test ERROR on v4.7 next-20160729] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: >

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test ERROR on battery/master] [also build test ERROR on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test ERROR on battery/master] [also build test ERROR on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test WARNING on battery/master] [also build test WARNING on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test WARNING on battery/master] [also build test WARNING on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test ERROR on battery/master] [also build test ERROR on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread kbuild test robot
Hi, [auto build test ERROR on battery/master] [also build test ERROR on v4.7 next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread Benjamin Tissoires
MSHW0011 replaces the battery firmware by using ACPI operation regions. The values have been obtained by reverse engineering, and are subject to errors. Looks like it works on overall pretty well. I couldn't manage to get the IRQ correctly triggered, so I am using a good old polling thread to

[PATCH 2/2] power: surface3_power: MSHW0011 rev-eng implementation

2016-07-29 Thread Benjamin Tissoires
MSHW0011 replaces the battery firmware by using ACPI operation regions. The values have been obtained by reverse engineering, and are subject to errors. Looks like it works on overall pretty well. I couldn't manage to get the IRQ correctly triggered, so I am using a good old polling thread to