Re: [PATCH RESEND] i2c: Remove setting for 1 second timeout from adapter drivers

2015-12-04 Thread Jarkko Nikula
On 12/03/2015 10:52 PM, Wolfram Sang wrote: On Wed, Dec 02, 2015 at 01:02:34PM +0200, Jarkko Nikula wrote: I2C adapter drivers that are using 1 second timeout can leave the timeout unset and let the i2c-core.c: i2c_register_adapter() to set it instead. Signed-off-by: Jarkko Nikula

Re: [PATCH 2/9] i2c: add generic routine to parse DT for timing information

2015-12-04 Thread Wolfram Sang
> > +void of_i2c_parse_timings(struct device_node *node, struct i2c_timings *t, > > bool use_defaults) > > +{ > > + memset(t, 0, sizeof(*t)); > > + > > + if (of_property_read_u32(node, "clock-frequency", >bus_freq_hz) && > > use_defaults) > > + t->bus_freq_hz = 10; > > Why not

Re: [PATCH 2/9] i2c: add generic routine to parse DT for timing information

2015-12-04 Thread Mika Westerberg
On Thu, Dec 03, 2015 at 04:51:32PM +0100, Wolfram Sang wrote: > From: Wolfram Sang > > Inspired from the i2c-rk3x driver (thanks guys!) but refactored and > extended. See built-in docs for further information. > > Signed-off-by: Wolfram Sang

[Patch V1] i2c: imx: init bus recovery info before adding i2c adapter

2015-12-04 Thread Gao Pan
During driver probe, i2c_imx_init_recovery_info() must come before i2c_add_numbered_adapter(), because the get/set_scl() functions are assigned in i2c_register_adapter() under the conditon that bus recover_info are initialized. Otherwise, get/set_scl() function pointers never get assigned. In

Re: [PATCH v3 0/2] ACPI: Add irq_type to gpio interrupt

2015-12-04 Thread Christophe Ricard
Hi Mika, On 01/12/2015 14:03, Mika Westerberg wrote: [...] Now for the same kind of i2c driver using acpi description, the GpioInt polarity/type is at the moment never kept in the irq property. It is possible to check that following about the same path... i2c_device_probe