[PATCH v1] HID: support multiple and large i2c transfers in hid-cp2112

2015-04-23 Thread Ellen Wang
cp2112_i2c_xfer() only supports a single i2c_msg and only reads up to 61 bytes. More than one message at a time and longers reads just return errors. This is a serious limitation. For example, the at24 eeprom driver generates paired write and read messages (for eeprom address and data). And the

Re: [PATCH v1] HID: bug fixes in hid-cp2112 driver

2015-04-23 Thread Jiri Kosina
On Thu, 23 Apr 2015, Ellen Wang wrote: > I've separated out the cp2112_i2c_xfer() bug fix from the patch. Should I > submit it as v2 or a new patch altogether? Please submit this as a completely new series, obsoleting the previous patch. Thanks, -- Jiri Kosina SUSE Labs -- To unsubscribe fro

Re: [PATCH v1] HID: bug fixes in hid-cp2112 driver

2015-04-23 Thread Ellen Wang
I've separated out the cp2112_i2c_xfer() bug fix from the patch. Should I submit it as v2 or a new patch altogether? Thanks. On 4/22/2015 1:03 AM, Wolfram Sang wrote: Well, at24 detects how many bytes it got and continues from there. That's true, but instead of returning short, the old cp

[PATCH] i2c: scx200_acb: avoid I2C bus overclocking.

2015-04-23 Thread Svenning Sørensen
According to I2C spec, max SCL rate is 100 kHz, but SCx200/CS5536 controller is currently driving it at 214 kHz according to my math. SCL is derived from an input clock of 48 MHz, which must be divided by 480 (240 cycles for each SCL high/low state) to be within spec. Signed-off-by: Svenning Soer

[PATCH V3 4/4] arm64: dts: add proxy I2C device driver on APM X-Gene platform

2015-04-23 Thread Feng Kan
Add proxy I2C device driver on APM X-Gene platform. Signed-off-by: Feng Kan Signed-off-by: Hieu Le --- arch/arm64/boot/dts/apm/apm-storm.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/apm/apm-storm.dtsi b/arch/arm64/boot/dts/apm/apm-storm.dtsi index a2b9657..

[PATCH V3 3/4] Documentation: i2c: Add APM X-Gene platform SLIMpro I2C driver documentation

2015-04-23 Thread Feng Kan
Add APM X-Gene platform SLIMpro I2C driver documentation. Signed-off-by: Feng Kan Signed-off-by: Hieu Le --- .../devicetree/bindings/i2c/i2c-xgene-slimpro.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/i2c/i2c-xgene-slimp

[PATCH V3 1/4] i2c: busses: add SLIMpro I2C device driver on APM X-Gene platform

2015-04-23 Thread Feng Kan
Add SLIMpro I2C device driver on APM X-Gene platform. This I2C device driver use the SLIMpro Mailbox driver to tunnel message to the SLIMpro coprocessor to do the work of accessing I2C components. Signed-off-by: Feng Kan Signed-off-by: Hieu Le --- drivers/i2c/busses/Kconfig | 9 +

[PATCH V3 2/4] i2c: busses: xgene: add acpi support for i2c xgene SLIMpro driver

2015-04-23 Thread Feng Kan
This adds support for ACPI for the APM X-Gene I2C SLIMpro proxy driver. Signed-off-by: Feng Kan --- drivers/i2c/busses/i2c-xgene-slimpro.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-xgene-slimpro.c b/drivers/i2c/busses/i2c-xgene-slimp

[PATCH V3 0/4] i2c: busses: xgene: I2C proxy driver for X-Gene

2015-04-23 Thread Feng Kan
This is a proxy I2C driver for APM X-Gene SoC. It uses the mailbox driver to tunnel i2c transactions via the SLIMpro processor. V3 Change: - Add ACPI support - Fix previous comments. Feng Kan (4): i2c: busses: add SLIMpro I2C device driver on APM X-Gene platform i2c: busses: x

Re: [PATCH] i2c: st: add include for pinctrl

2015-04-23 Thread Wolfram Sang
On Thu, Apr 23, 2015 at 01:42:02PM +0200, Wolfram Sang wrote: > The driver uses pinctrl directly and thus should include the appropriate > header. Sort the headers while we are here to have a better view what is > included and what is not. > > Reported-by: Pascal Huerst > Signed-off-by: Wolfram S

Re: [PATCH] i2c: mux: use proper dev when removing "channel-X" symlinks

2015-04-23 Thread Wolfram Sang
On Thu, Apr 23, 2015 at 10:29:09AM +0200, Wolfram Sang wrote: > From: Wolfram Sang > > Those symlinks are created for the mux_dev, so we need to remove it from > there. Currently, it breaks for muxes where the mux_dev is not the device > of the parent adapter like this: > > [ 78.234644] WARNIN

[PATCH] i2c: st: add include for pinctrl

2015-04-23 Thread Wolfram Sang
The driver uses pinctrl directly and thus should include the appropriate header. Sort the headers while we are here to have a better view what is included and what is not. Reported-by: Pascal Huerst Signed-off-by: Wolfram Sang --- drivers/i2c/busses/i2c-st.c | 13 +++-- 1 file changed,

Re: [PATCH] omap: i2c: Add calls for pinctrl state select

2015-04-23 Thread Wolfram Sang
On Wed, Apr 22, 2015 at 12:49:02PM +0200, Pascal Huerst wrote: > > > On 22.04.2015 12:28, Wolfram Sang wrote: > > I assume you have a different kernel config where something is enabled > > which includes the pinctrl-stuff in another include which is included > > by these drivers. Although, my .co

[PATCH] i2c: mux: use proper dev when removing "channel-X" symlinks

2015-04-23 Thread Wolfram Sang
From: Wolfram Sang Those symlinks are created for the mux_dev, so we need to remove it from there. Currently, it breaks for muxes where the mux_dev is not the device of the parent adapter like this: [ 78.234644] WARNING: CPU: 0 PID: 365 at fs/sysfs/dir.c:31 sysfs_warn_dup+0x5c/0x78() [ 78.2