Vaibhav Hiremath writes:
> Robert,
>
> It would be helpful if you can test this patch-series and confirm that
> it now fixes the NULL pointer deference issue.
Tested, it works on pxa27x in master mode, in non-DT mode.
For all non-DT patches, you can add my :
Tested-by: Robert Jarz
Vaibhav Hiremath writes:
> Robert,
>
> It would be helpful if you can test this patch-series and confirm that
> it now fixes the NULL pointer deference issue.
>
> Thanks,
> Vaibhav
Hi Vaibhav,
My next slot is probably this comming Sunday. I'll do the test and report.
Cheers.
--
Robert
--
To u
Vaibhav Hiremath writes:
> On Monday 20 July 2015 12:39 PM, Vaibhav Hiremath wrote:
>>
>>
>> On Monday 20 July 2015 12:36 PM, Vaibhav Hiremath wrote:
>>>
>>>
>>> On Saturday 18 July 2015 01:19 AM, Robert Jarzmik wrote:
>>>> Vaibhav Hi
Vaibhav Hiremath writes:
>> So, I applied patches 1-6 to for-next to make some progress.
>>
>> The others need more thought because of the bindings which shall be
>> discussed replying to the patches in question.
>>
>> Thanks for the updated work with lots of proper references.
>>
>
> OK, Thanks
off-by: Vaibhav Hiremath
Acked-by: Robert Jarzmik
Cheers.
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
hav Hiremath
> Cc: Wolfram Sang
Acked-by: Robert Jarzmik
Cheers.
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Vaibhav Hiremath writes:
> I have taken care of all comments on the 3 patches.
>
> Just in case if you have any comments on other patches in series,
> I will wait for a day before pushing next version.
That would be great, yeah. I'll make another pass tomorrow. If that works for
you, and if you
rr(&dev->dev, "failed to get the clk: %ld\n",
PTR_ERR(i2c->clk));
Once that is done, you can add my :
Acked-by: Robert Jarzmik
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Vaibhav Hiremath writes:
> #define _IBMR(i2c) ((i2c)->reg_ibmr)
> @@ -286,6 +287,22 @@ static void i2c_pxa_scream_blue_murder(struct pxa_i2c
> *i2c, const char *why)
> static void i2c_pxa_master_complete(struct pxa_i2c *i2c, int ret);
> static irqreturn_t i2c_pxa_handler(int this_irq, void
Vaibhav Hiremath writes:
> + i2c->clk = devm_clk_get(&dev->dev, NULL);
> + if (IS_ERR(i2c->clk)) {
> + dev_err(&dev->dev, "failed to get the clk\n");
> + dev_err(&dev->dev, "failed to get the clk: %ld\n",
> PTR_ERR(i2c->clk));
> + return PTR_ERR(i2c->
Vaibhav Hiremath writes:
> In case of timeout during msg xfer assert reset to
> i2c controller for both interrupt and PIO mode of operation.
>
> Signed-off-by: Jett.Zhou
> [vaibhav.hirem...@linaro.org: Split & merge patches into logical changes
> and update the Changelog]
> Signed-off-by: Vaibha
Vaibhav Hiremath writes:
> @@ -1270,10 +1270,17 @@ static int i2c_pxa_probe(struct platform_device *dev)
> struct resource *res = NULL;
> int ret, irq;
>
> - i2c = kzalloc(sizeof(struct pxa_i2c), GFP_KERNEL);
> + i2c = devm_kzalloc(&dev->dev, sizeof(struct pxa_i2c), GFP_KERN
Vaibhav Hiremath writes:
> Update i2c_pxa_scream_blue_murder() fn to print more information
> in case of error.
>
> Signed-off-by: Jett.Zhou
> [vaibhav.hirem...@linaro.org: Split patches into logical changes
> and update the Changelog]
> Signed-off-by: Vaibhav Hiremath
> Cc: Wolfram Sang
>
> S
bhav.hirem...@linaro.org: Updated Changelog]
> Signed-off-by: Vaibhav Hiremath
> Cc: Wolfram Sang
Acked-by: Robert Jarzmik
Cheers.
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo i
Vaibhav Hiremath writes:
> In case of timeout during msg xfer assert reset to
> i2c controller for both interrupt and PIO mode of operation.
>
> Signed-off-by: Jett.Zhou
> [vaibhav.hirem...@linaro.org: Split & merge patches into logical changes
> and update the Changelog]
> Signed-off-by: Vaibha
Vaibhav Hiremath writes:
> From: Shouming Wang
>
> In case of timeout in pio mode of operation return I2C_RETRY.
> This behavior will be same as interrupt mode of operation.
>
> Signed-off-by: Shouming Wang
> [vaibhav.hirem...@linaro.org: Updated changelog]
> Signed-off-by: Vaibhav Hiremath
>
Vaibhav Hiremath writes:
> From: Rob Herring
>
> Since there is some problematic i2c slave devices on some
> platforms such as dkb (sometimes), it will drop down sda
> and make i2c bus hang, at that time, it need to config
> scl/sda into gpio to simulate "stop" sequence to recover
> i2c bus, so
Vaibhav Hiremath writes:
> @@ -167,6 +184,8 @@ struct pxa_i2c {
> #define _ICR(i2c)((i2c)->reg_icr)
> #define _ISR(i2c)((i2c)->reg_isr)
> #define _ISAR(i2c) ((i2c)->reg_isar)
> +#define _ILCR(i2c) ((i2c)->reg_ilcr)
> +#define _IWCR(i2c) ((i2c)->reg_iwcr)
>
> /*
> * I2C Slave
Vaibhav Hiremath writes:
> From: "Jett.Zhou"
>
> According to some test in emei_dkb, we found some i2c slave device
> (eg. camera sensor ov2659 power up) introduce noise on sda, so detect
> i2c controller busy, and assert reset to i2c controller to recover as
> early as possible to avoid more la
Vaibhav Hiremath writes:
> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
> index f4ac8c5..d4c798a 100644
> --- a/drivers/i2c/busses/i2c-pxa.c
> +++ b/drivers/i2c/busses/i2c-pxa.c
> @@ -459,8 +459,10 @@ static void i2c_pxa_reset(struct pxa_i2c *i2c)
> writel(I2C_IS
Dmitry Eremin-Solenikov writes:
> As new locomo driver supports SPARSE_IRQ, don't preallocate NR_IRQS
> space for it on poodle.
>
> Signed-off-by: Dmitry Eremin-Solenikov
Acked-by: Robert Jarzmik
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe lin
Dmitry Eremin-Solenikov writes:
> -#define POODLE_LOCOMO_GPIO_AMP_ON LOCOMO_GPIO(8)
> -#define POODLE_LOCOMO_GPIO_MUTE_L LOCOMO_GPIO(10)
> -#define POODLE_LOCOMO_GPIO_MUTE_R LOCOMO_GPIO(11)
> -#define POODLE_LOCOMO_GPIO_232VCC_ON LOCOMO_GPIO(12)
> -#define POODLE_LOCOMO_GPIO_JK_B
Dmitry Eremin-Solenikov writes:
> Change clk_enable/disable() calls to clk_prepare_enable() and
> clk_disable_unprepare().
>
> Signed-off-by: Dmitry Eremin-Solenikov
> ---
> drivers/usb/gadget/udc/pxa25x_udc.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/driv
Hi,
In linux-next, the commit named "i2c: Add missing KERN_* constants to printks",
id 77130fc992cfdcce8c0a748a2daef16fc16e76bb, breaks compilation for pxa
architectures.
I think that hunk should be amended :
> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
> index bdb1
24 matches
Mail list logo