Re: [PATCH V3] ACPI: Add _DEP(Operation Region Dependencies) support to fix battery issue on the Asus T100TA

2014-11-23 Thread Michael Shigorin
On Mon, Nov 24, 2014 at 10:39:55AM +0800, Li, Aubrey wrote: > Well, Can we explicitly tied this up with ASUS T100TA in the code? It works for me on Aava Inari 8 too. --   WBR, Michael Shigorin / http://altlinux.org   -- http://opennet.ru / http://anna-news.info -- To unsubscribe from thi

Re: [PATCH V3] ACPI: Add _DEP(Operation Region Dependencies) support to fix battery issue on the Asus T100TA

2014-11-23 Thread Adam Williamson
On Mon, 2014-11-24 at 10:39 +0800, Li, Aubrey wrote: > On 2014/11/23 21:22, Lan Tianyu wrote: > > ACPI 5.0 introduces _DEP to designate device objects that OSPM should > > assign a higher priority in start ordering due to future operation region > > accesses. > > > > On Asus T100TA, ACPI battery i

Re: [PATCH V3] ACPI: Add _DEP(Operation Region Dependencies) support to fix battery issue on the Asus T100TA

2014-11-23 Thread Li, Aubrey
On 2014/11/23 21:22, Lan Tianyu wrote: > ACPI 5.0 introduces _DEP to designate device objects that OSPM should > assign a higher priority in start ordering due to future operation region > accesses. > > On Asus T100TA, ACPI battery info are read from a I2C slave device via > I2C operation region.

[PATCH] i2c: rk3x: add Kconfig dependency on COMMON_CLK

2014-11-23 Thread Max Schwarz
Now that we are using the clk notifier framework we get compile errors without COMMON_CLK. But the driver fails to probe without COMMON_CLK anyways, so just add that as a Kconfig dependency. Signed-off-by: Max Schwarz --- As reported by the kbuild test robot at Intel Open Source. drivers/i2c/b

Re: [PATCH v2] Add support for SCCB devices into PXA27x I2C controller

2014-11-23 Thread Petr Cvek
I'm sorry for spacing, I didn't realize thunderbird automatically replaces tabs with spaces. Now it should be OK. I created this patch by fixing both I2C controller and SCCB device, so it seems I incorrectly used I2C_M_NOSTART instead I2C_M_STOP. Problem with I2C_M_STOP is that it causes bus f

Re: [4/5] i2c: davinci: use bus recovery infrastructure

2014-11-23 Thread Uwe Kleine-König
Hello Grygorii, On Fri, Nov 21, 2014 at 09:33:22PM +0200, Grygorii Strashko wrote: > On 11/21/2014 09:07 PM, Uwe Kleine-König wrote: > > On Thu, Nov 20, 2014 at 12:03:07PM +0200, Grygorii Strashko wrote: > > Just another general comment about the driver that doesn't influence the > > correctness o

Re: [2/5] i2c: davinci: query STP always when NACK is received

2014-11-23 Thread Uwe Kleine-König
Hello Grygorii, On Fri, Nov 21, 2014 at 05:33:37PM +0200, Grygorii Strashko wrote: > On 11/21/2014 03:10 PM, Uwe Kleine-König wrote: > > On Fri, Nov 21, 2014 at 02:48:57PM +0200, Grygorii Strashko wrote: > >> On 11/21/2014 12:19 AM, Uwe Kleine-König wrote: > diff --git a/drivers/i2c/busses/i2

Re: [PATCH 2/3] i2c: slave-eeprom: add eeprom simulator driver

2014-11-23 Thread Uwe Kleine-König
Hello Wolfram, On Sat, Nov 22, 2014 at 07:26:30PM +0100, Wolfram Sang wrote: > > > this mail is thematically more a reply to patch 1 and maybe just serves > > my understanding of the slave support. > > Sure. This shows how badly needed the documentation is :) > > ... > > > + break; > >

Re: [PATCH 2/3] i2c: slave-eeprom: add eeprom simulator driver

2014-11-23 Thread Uwe Kleine-König
Hallo Wolfram, On Sat, Nov 22, 2014 at 07:14:06PM +0100, Wolfram Sang wrote: > > > > + case I2C_SLAVE_REQ_READ_END: > > > > + eeprom->buffer_idx++; > > > You don't check here for buffer_idx >= ARRAY_SIZE(buffer)? > > > Ditto in the I2C_SLAVE_REQ_WRITE_END case. > > I just notic

Re: [5/5] i2c: davinci: use ICPFUNC to toggle I2C as gpio for bus recovery

2014-11-23 Thread Uwe Kleine-König
Hello, On Thu, Nov 20, 2014 at 12:03:08PM +0200, Grygorii Strashko wrote: > @@ -664,6 +759,7 @@ static int davinci_i2c_probe(struct platform_device *pdev) > if (!of_property_read_u32(pdev->dev.of_node, "clock-frequency", > &prop)) > dev->pd

Re: [PATCH v8 6/8] OF: i2c: Add OF notifier handler

2014-11-23 Thread Wolfram Sang
> I picked up the patch into my tree and made the above changes because > they're pretty trivial. If Wolfram is okay with it then I can take the > whole series through my tree. Otherwise I'll put the of_reconfig_* empty > stubs into a separate branch that he and broonie can both pull. With your f

Re: [PATCH] Add support for SCCB devices into PXA27x I2C controller

2014-11-23 Thread Sergei Shtylyov
Hello. On 11/23/2014 1:52 AM, Petr Cvek wrote: Add support for SCCB devices into PXA27x I2C controller. Fix generated START but no STOP for message without I2C_M_NOSTART flag. Add support for I2C_M_IGNORE_NAK flag. Signed-off-by: Petr Cvek --- drivers/i2c/busses/i2c-pxa.c | 22

Re: [PATCH v8 5/8] of: i2c: Export single device registration method

2014-11-23 Thread Wolfram Sang
On Fri, Nov 21, 2014 at 01:46:31AM +, Grant Likely wrote: > On Tue, 28 Oct 2014 22:36:02 +0200 > , Pantelis Antoniou > wrote: > > Dynamically inserting i2c client device nodes requires the use > > of a single device registration method. Rework and export it. > > > > Signed-off-by: Pantelis A

[PATCH V3] ACPI: Add _DEP(Operation Region Dependencies) support to fix battery issue on the Asus T100TA

2014-11-23 Thread Lan Tianyu
ACPI 5.0 introduces _DEP to designate device objects that OSPM should assign a higher priority in start ordering due to future operation region accesses. On Asus T100TA, ACPI battery info are read from a I2C slave device via I2C operation region. Before I2C operation region handler is installed, b

Re: [PATCH 2/4] i2c: omap: implement workaround for handling invalid BB-bit values

2014-11-23 Thread Alexander Kochetkov
23 нояб. 2014 г., в 7:43, Felipe Balbi написал(а): > maybe there was a typo? I tested on v3.18-rc3 :-) I do my tests on kernel from angstrom with almost all i2c-omap patches backported from linux/master. Then I rebased them to wrong (old) kernel version and posted to the list. Angstrom kernel

Re: [PATCH] Add support for SCCB devices into PXA27x I2C controller

2014-11-23 Thread Wolfram Sang
On Sat, Nov 22, 2014 at 11:52:07PM +0100, Petr Cvek wrote: > Add support for SCCB devices into PXA27x I2C controller. > > Fix generated START but no STOP for message without I2C_M_NOSTART flag. Add > support for I2C_M_IGNORE_NAK flag. > > Signed-off-by: Petr Cvek I can't apply the patch. What i