Re: [PATCH] mmc: sdhci: dove: Fix build breakage

2013-10-23 Thread Jeremie Samuel
Soory, I thought I had corrected this typo. Signed-off-by: Jeremie Samuel -- Jeremie Samuel Parrot S.A. Software Engineer 14, quai de Jemmapes R&D/OS Platform 75010 Paris, France http://www.parrot.com On 23/10/2013 15:10, Thierry Reding wrote: Co

Re: [PATCH v2 0/8] sdhci: Move real work out of an atomic context

2013-10-21 Thread Jeremie Samuel
jul 13: Resend in order to include the patches in kernel 3.12 Version of 24 may 13: First version of my patches (based on Anton Vorontsov's patches http://thread.gmane.org/gmane.linux.kernel.mmc/2579) Sorry for the issues. It's my first submission on a Linux mailing list. Cordially, -

[PATCH v2 1/8] sdhci: Turn timeout timer into delayed work

2013-10-21 Thread Jeremie Samuel
l.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 13 +++-- include/linux/mmc/sdhci.h |3 ++- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 66bce7a..b3

[PATCH v2 4/8] sdhci: Use threaded IRQ handler

2013-10-21 Thread Jeremie Samuel
ntsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 -- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 362a838..0bb892c 100644 --- a/drivers/mmc/host/sdhci.c

[PATCH v2 2/8] sdhci: Turn tuning timeout timer into delayed work

2013-10-21 Thread Jeremie Samuel
Same as previous patch Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 ++--- include/linux/mmc/sdhci.h |2 +- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

[PATCH v2 3/8] sdhci: Use work structs instead of tasklets

2013-10-21 Thread Jeremie Samuel
Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci-bcm-kona.c |2 +- drivers/mmc/host/sdhci-dove.c |6 ++-- drivers/mmc/host/sdhci-s3c.c |2 +- drivers/mmc/host/sdhci-sirf.c |4 +-- drivers/mmc/host/sdhci-spear.c|2 +- drivers/mmc/host/sd

[PATCH v2 8/8] sdhci: Use jiffies instead of a timeout counter

2013-10-21 Thread Jeremie Samuel
Just a cosmetic change, should not affect functionality. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions

[PATCH v2 7/8] sdhci: Get rid of mdelay()s where it is safe and makes sense

2013-10-21 Thread Jeremie Samuel
Sure, we might end up calling msleep() there too, just not via this safe patch. PAtch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |8 1 file changed, 4 insertions(+), 4

[PATCH v2 6/8] sdhci: Turn host->lock into a mutex

2013-10-21 Thread Jeremie Samuel
Finally, we can get rid of the host->lock spinlock, and turn it into a mutex. This patch does just this. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci-esdhc-imx.c |4 +- driv

[PATCH v2 5/8] sdhci: Delay led blinking

2013-10-21 Thread Jeremie Samuel
The function sdhci_led_control is called in an atomic context by the led_trigger driver. So, it is necessary to delay the activation or deactivation of the led. Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 42 ++ include/linux/mmc

[PATCH v2 0/8] sdhci: Move real work out of an atomic context

2013-10-21 Thread Jeremie Samuel
old patch from Anton Vorontsov. I tried to fix all the problems involved by the patches. I tested it for several time now with SD cards and SDIO. So, this patch set reworks sdhci code to avoid atomic context, almost completely. Thanks, Jeremie Samuel Jeremie Samuel (8): sdhci: Turn timeout

Re: [PATCH 3/8] sdhci: Use work structs instead of tasklets

2013-10-21 Thread Jeremie Samuel
Hi Chris, Thanks for the feedback. Do you have a way for me to compile the driver for all the architectures impacted by the patch? So I could send a patch which compile for all these architectures. Cordially, Jeremie -- Jeremie Samuel Parrot S.A. Software Engineer

[PATCH 4/8] sdhci: Use threaded IRQ handler

2013-10-16 Thread Jeremie Samuel
ntsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 -- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 362a838..0bb892c 100644 --- a/drivers/mmc/host/sdhci.c

[PATCH 3/8] sdhci: Use work structs instead of tasklets

2013-10-16 Thread Jeremie Samuel
Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 55 + include/linux/mmc/sdhci.h |4 ++-- 2 files changed, 27 insertions(+), 32 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 03

[PATCH 7/8] sdhci: Get rid of mdelay()s where it is safe and makes sense

2013-10-16 Thread Jeremie Samuel
Sure, we might end up calling msleep() there too, just not via this safe patch. PAtch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |8 1 file changed, 4 insertions(+), 4

[PATCH 1/8] sdhci: Turn timeout timer into delayed work

2013-10-16 Thread Jeremie Samuel
l.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 13 +++-- include/linux/mmc/sdhci.h |3 ++- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 66bce7a..b3

[PATCH 8/8] sdhci: Use jiffies instead of a timeout counter

2013-10-16 Thread Jeremie Samuel
Just a cosmetic change, should not affect functionality. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions

[PATCH 5/8] sdhci: Delay led blinking

2013-10-16 Thread Jeremie Samuel
The function sdhci_led_control is called in an atomic context by the led_trigger driver. So, it is necessary to delay the activation or deactivation of the led. Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 42 ++ include/linux/mmc

[PATCH 2/8] sdhci: Turn tuning timeout timer into delayed work

2013-10-16 Thread Jeremie Samuel
Same as previous patch Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 ++--- include/linux/mmc/sdhci.h |2 +- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

[PATCH 6/8] sdhci: Turn host->lock into a mutex

2013-10-16 Thread Jeremie Samuel
Finally, we can get rid of the host->lock spinlock, and turn it into a mutex. This patch does just this. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci-esdhc-imx.c |4 +- driv

[PATCH 0/8] sdhci: Move real work out of an atomic context

2013-10-16 Thread Jeremie Samuel
old patch from Anton Vorontsov. I tried to fix all the problems involved by the patches. I tested it for several time now with SD cards and SDIO. So, this patch set reworks sdhci code to avoid atomic context, almost completely. Thanks, Jeremie Samuel Jeremie Samuel (8): sdhci: Turn timeout

Re: [PATCH 6/8] sdhci: Turn host->lock into a mutex

2013-08-26 Thread Jeremie Samuel
Hi Chris, Thank you for the feedback. I'll send you the modifications in September. But I will not be able to test the changes. I don't have the hardware. Regards, -- Jeremie Samuel Parrot S.A. Software Engineer 14, quai de Jemmapes R&D/OS Platform

Re: [PATCH 0/8] sdhci: Move real work out of an atomic context

2013-07-11 Thread Jeremie Samuel
On 09/07/2013 17:52, Philip Rakity wrote: On Jul 9, 2013, at 4:44 PM, Jeremie Samuel wrote: Hi all, Currently the sdhci driver does everything in the atomic context. And what is worse, PIO transfers are made from the IRQ handler. Some patches were already submitted to solve this issue

[PATCH 8/8] sdhci: Use jiffies instead of a timeout counter

2013-07-09 Thread Jeremie Samuel
Just a cosmetic change, should not affect functionality. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions

[PATCH 2/8] sdhci: Turn tuning timeout timer into delayed work

2013-07-09 Thread Jeremie Samuel
Same as previous patch Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 ++--- include/linux/mmc/sdhci.h |2 +- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

[PATCH 5/8] sdhci: Delay led blinking

2013-07-09 Thread Jeremie Samuel
The function sdhci_led_control is called in an atomic context by the led_trigger driver. So, it is necessary to delay the activation or deactivation of the led. Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 42 ++ include/linux/mmc

[PATCH 1/8] sdhci: Turn timeout timer into delayed work

2013-07-09 Thread Jeremie Samuel
l.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 13 +++-- include/linux/mmc/sdhci.h |3 ++- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index a78bd4f..3f

[PATCH 7/8] sdhci: Get rid of mdelay()s where it is safe and makes sense

2013-07-09 Thread Jeremie Samuel
Sure, we might end up calling msleep() there too, just not via this safe patch. PAtch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |8 1 file changed, 4 insertions(+), 4

[PATCH 6/8] sdhci: Turn host->lock into a mutex

2013-07-09 Thread Jeremie Samuel
Finally, we can get rid of the host->lock spinlock, and turn it into a mutex. This patch does just this. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |

[PATCH 3/8] sdhci: Use work structs instead of tasklets

2013-07-09 Thread Jeremie Samuel
Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 55 + include/linux/mmc/sdhci.h |4 ++-- 2 files changed, 27 insertions(+), 32 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 06

[PATCH 4/8] sdhci: Use threaded IRQ handler

2013-07-09 Thread Jeremie Samuel
ntsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 -- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index ab635d5..21ef1e0 100644 --- a/drivers/mmc/host/sdhci.c

[PATCH 0/8] sdhci: Move real work out of an atomic context

2013-07-09 Thread Jeremie Samuel
old patch from Anton Vorontsov. I tried to fix all the problems involved by the patches. I tested it for several time now with SD cards and SDIO. So, this patch set reworks sdhci code to avoid atomic context, almost completely. Thanks, Jeremie Samuel Jeremie Samuel (8): sdhci: Turn timeout

Re: [PATCH 0/8] sdhci: Move real work out of an atomic context

2013-06-13 Thread Jeremie Samuel
Hi, I posted these patches a few weeks ago. Is it possible to get a feedback for this submission? Thank you for your help. -- Jeremie Samuel Parrot S.A. Software Engineer 14, quai de Jemmapes R&D/OS Platform 75010 Paris, France http://www.parrot.com O

[PATCH 8/8] sdhci: Use jiffies instead of a timeout counter

2013-05-24 Thread Jeremie Samuel
Just a cosmetic change, should not affect functionality. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions

[PATCH 6/8] sdhci: Turn host->lock into a mutex

2013-05-24 Thread Jeremie Samuel
Finally, we can get rid of the host->lock spinlock, and turn it into a mutex. This patch does just this. Patch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |

[PATCH 7/8] sdhci: Get rid of mdelay()s where it is safe and makes sense

2013-05-24 Thread Jeremie Samuel
Sure, we might end up calling msleep() there too, just not via this safe patch. PAtch based on: http://thread.gmane.org/gmane.linux.kernel.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c |8 1 file changed, 4 insertions(+), 4

[PATCH 2/8] sdhci: Turn tuning timeout timer into delayed work

2013-05-24 Thread Jeremie Samuel
Same as previous patch Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 ++--- include/linux/mmc/sdhci.h |2 +- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c

[PATCH 5/8] sdhci: Delay led blinking

2013-05-24 Thread Jeremie Samuel
The function sdhci_led_control is called in an atomic context by the led_trigger driver. So, it is necessary to delay the activation or deactivation of the led. Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 42 ++ include/linux/mmc

[PATCH 3/8] sdhci: Use work structs instead of tasklets

2013-05-24 Thread Jeremie Samuel
Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 55 + include/linux/mmc/sdhci.h |4 ++-- 2 files changed, 27 insertions(+), 32 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 7d

[PATCH 4/8] sdhci: Use threaded IRQ handler

2013-05-24 Thread Jeremie Samuel
ntsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 46 -- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 063c20a..f361fec 100644 --- a/drivers/mmc/host/sdhci.c

[PATCH 0/8] sdhci: Move real work out of an atomic context

2013-05-24 Thread Jeremie Samuel
old patch from Anton Vorontsov. I tried to fix all the problems involved by the patches. I tested it for several time now with SD cards and SDIO. So, this patch set reworks sdhci code to avoid atomic context, almost completely. Thanks, Jeremie Samuel Jeremie Samuel (8): sdhci: Turn timeout

[PATCH 1/8] sdhci: Turn timeout timer into delayed work

2013-05-24 Thread Jeremie Samuel
l.mmc/2579. Signed-off-by: Anton Vorontsov Signed-off-by: Jeremie Samuel --- drivers/mmc/host/sdhci.c | 13 +++-- include/linux/mmc/sdhci.h |3 ++- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 2ea429c..9d

Control drive strength in software

2013-02-04 Thread Jeremie Samuel
ant to add this a call to this function in mmc_sd_switch_hs or in mmc_sd_init_card? Thanks, Jeremie Samuel -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html