[PATCH] mmc: hsmmc: Fix NULL pointer dereference on unload when booted with device tree

2012-11-09 Thread Tony Lindgren
When booting with device tree, we don't have any pdata, and will get the following when trying to unload omap_hsmmc: Unable to handle kernel NULL pointer dereference at virtual address 0044 pgd = edafc000 *pgd=adbbd831, *pte=, *ppte= SMP ARM Modules linked in: omap_hsmmc(-) oma

Re: [PATCH v2 3/3] mmc: omap_hsmmc: update error code for response_busy cmd

2012-11-09 Thread Felipe Balbi
Hi, On Fri, Nov 09, 2012 at 09:36:16PM +0530, Balaji T K wrote: > update error code to cmd->error for commands with response_busy and no data > > Signed-off-by: Balaji T K Just one comment below, otherwise: Reviewed-by: Felipe Balbi > --- > drivers/mmc/host/omap_hsmmc.c |4 ++-- > 1 fil

Re: [PATCH v2 2/3] mmc: omap_hsmmc: no reset of cmd state machine for DCRC

2012-11-09 Thread Felipe Balbi
On Fri, Nov 09, 2012 at 09:36:15PM +0530, Balaji T K wrote: > Avoid soft reset of command internal state machine on data errors. > > Signed-off-by: Balaji T K Reviewed-by: Felipe Balbi > --- > drivers/mmc/host/omap_hsmmc.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > dif

Re: [PATCH v2 1/3] mmc: omap_hsmmc: Fix Oops in case of data errors

2012-11-09 Thread Felipe Balbi
On Fri, Nov 09, 2012 at 09:36:14PM +0530, Balaji T K wrote: > "commit ae4bf788ee9bf7c2d51b0309117d1fcccbdd50a2 > mmc: omap_hsmmc: consolidate error report handling of HSMMC IRQ" > sets both end_cmd and end_trans to 1. > > Setting end_cmd to 1 for Data Timeout/CRC leads to NULL pointer dereference

[PATCH v2 1/3] mmc: omap_hsmmc: Fix Oops in case of data errors

2012-11-09 Thread Balaji T K
"commit ae4bf788ee9bf7c2d51b0309117d1fcccbdd50a2 mmc: omap_hsmmc: consolidate error report handling of HSMMC IRQ" sets both end_cmd and end_trans to 1. Setting end_cmd to 1 for Data Timeout/CRC leads to NULL pointer dereference of host->cmd as the command complete has previously been handled. Set

[PATCH v2 2/3] mmc: omap_hsmmc: no reset of cmd state machine for DCRC

2012-11-09 Thread Balaji T K
Avoid soft reset of command internal state machine on data errors. Signed-off-by: Balaji T K --- drivers/mmc/host/omap_hsmmc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c index 245d7b5..d931e08 100644 ---

[PATCH v2 3/3] mmc: omap_hsmmc: update error code for response_busy cmd

2012-11-09 Thread Balaji T K
update error code to cmd->error for commands with response_busy and no data Signed-off-by: Balaji T K --- drivers/mmc/host/omap_hsmmc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c index d931e08..1503808

[PATCH v2 0/3] mmc: omap_hsmmc: fixes for irq error handler

2012-11-09 Thread Balaji T K
Balaji T K (3): mmc: omap_hsmmc: Fix Oops in case of data errors mmc: omap_hsmmc: no reset of cmd state machine for DCRC mmc: omap_hsmmc: update error code for response_busy cmd drivers/mmc/host/omap_hsmmc.c | 24 +++- 1 files changed, 15 insertions(+), 9 deletions(-)

Re: [PATCH 1/5] mmc: omap_hsmmc: Avoid host->cmd dereference during data transfer failures

2012-11-09 Thread Balaji T K
On Tuesday 06 November 2012 10:22 PM, Venkatraman S wrote: Sometimes, a error occurs just after the Command has been reported to be successful (CC=1) but before data transfer completes (TC=1). Setting end_cmd=1 here leads to a NULL pointer dereference of host->cmd as the command complete has prev

Re: [PATCH v7 0/3] Add modules to support realtek PCIE card reader

2012-11-09 Thread Chris Ball
Hi, On Fri, Nov 09 2012, wwang wrote: > I have noticed that the file rtsx_pci_sdmmc.c in your mmc-next branch > is still v6 patch. In v7 patch, this MMC part has also been modified > to comply with my MFD portion. So please update to this version. > > And one more question. Samuel have merged al

Re: [PATCH] dma: add new DMA control commands

2012-11-09 Thread Artem Bityutskiy
On Thu, 2012-10-18 at 16:29 +0800, Huang Shijie wrote: > 于 2012年10月18日 16:16, Marek Vasut 写道: > > So we can't stream data from the chip? About time to adjust the MTD > > framework to > > allow that. Maybe implement a command queue? > > > > to Artem & David: > is this possible to stream the da

Re: [PATCH] mmc: omap_hsmmc: Fix Oops in case of data errors

2012-11-09 Thread Balaji T K
On Friday 09 November 2012 08:22 PM, Felipe Balbi wrote: On Fri, Nov 09, 2012 at 08:11:19PM +0530, Balaji T K wrote: Setting end_cmd to 1 for Data Timeout/CRC leads to NULL pointer dereference of host->cmd as the command complete has previously been handled. Set end_cmd only in case of Data Time

Re: [PATCH v7 0/3] Add modules to support realtek PCIE card reader

2012-11-09 Thread wwang
于 11/08/2012 03:41 AM, Chris Ball 写道: Hi, On Mon, Oct 29 2012, wwang wrote: I'm happy with the MMC portion. I think we're mainly waiting for the MFD patch -- Samuel, do you have any remaining review comments that need to be fixed? Thanks, - Chris. Hi Chris: I have noticed that the file rts

Re: [PATCH] mmc: omap_hsmmc: Fix Oops in case of data errors

2012-11-09 Thread Felipe Balbi
On Fri, Nov 09, 2012 at 08:11:19PM +0530, Balaji T K wrote: > Setting end_cmd to 1 for Data Timeout/CRC leads to NULL pointer dereference of > host->cmd as the command complete has previously been handled. > Set end_cmd only in case of Data Timeout/CRC. this comment doesnt match code as code is se

[PATCH] mmc: omap_hsmmc: Fix Oops in case of data errors

2012-11-09 Thread Balaji T K
Setting end_cmd to 1 for Data Timeout/CRC leads to NULL pointer dereference of host->cmd as the command complete has previously been handled. Set end_cmd only in case of Data Timeout/CRC. While at it restore error handling behaviour as was before the "commit ae4bf788ee9bf7c2d51b0309117d1fcccbdd50a

[GIT PULL] MMC fixes for 3.7-rc5

2012-11-09 Thread Chris Ball
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git tags/mmc-fixes-for-3.7-rc5 to receive MMC fixes for 3.7-rc5. There are no merge conflicts, and the patches have been tested in linux-next. Thanks. The following changes since commit ddffeb8c4d0331609ef25

[PATCH 2/8] mmc: sdhci: fix coding style in sdhci_set_transfer_mode

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 2ff2b9e..2d3d76e 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -898,9 +898,9 @@

[PATCH 4/8] mmc: sdhci: fix coding style in sdhci_set_clock

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 73bb41f..28da461 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -1138,9 +1138,9

[PATCH 1/8] mmc: sdhci: fix coding style in sdhci_calc_timeout

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index c7851c0..2ff2b9e 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -667,9 +667,9 @@

[PATCH 3/8] mmc: sdhci: fix coding style in sdhci_finish_data

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 2d3d76e..73bb41f 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -924,9 +924,9

[PATCH 8/8] mmc: sdhci: fix coding style in sdhci_add_host

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c | 20 1 files changed, 12 insertions(+), 8 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 47cac71..5cddb74 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@

[PATCH 7/8] mmc: sdhci: fix coding style in sdhci_data_irq

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 4bed582..47cac71 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -2286,9 +2286,9

[PATCH 6/8] mmc: sdhci: fix coding style in sdhci_execute_tuning

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index b947155..4bed582 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -1792,9 +1792,9

[PATCH 5/8] mmc: sdhci: fix coding style in sdhci_do_set_ios

2012-11-09 Thread Tushar Behera
Signed-off-by: Tushar Behera --- drivers/mmc/host/sdhci.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 28da461..b947155 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -1386,9 +

[PATCH 0/8] Fix coding style in sdhci.c

2012-11-09 Thread Tushar Behera
Documents/CodingStyle Chapter 3 recommends usage of braces for both if and else statements if any of the branches contains multiple statements. Cleaning up drivers/mmc/host/sdhci.c for all these occurrences. Tushar Behera (8): mmc: sdhci: fix coding style in sdhci_calc_timeout mmc: sdhci: fix

[PATCH v3] mmc: sdhci-s3c: ensure non-transaction of bus before clk_disable

2012-11-09 Thread Seungwon Jeon
clock shoud be supplied during bus transaction. This patch defers clk_disabe in runtime_suspend if cmd/data line is used. Signed-off-by: Seungwon Jeon Acked-by: Heiko Stuebner --- V3: Removed the error message. (reported by Fengguang Wu) drivers/mmc/host/sdhci-s3c.c | 13 + drive

RE: [mmc:mmc-next 17/28] sdhci-pltfm.c:(.text+0x80): multiple definition of `sdhci_runtime_pm_get'

2012-11-09 Thread Seungwon Jeon
Fengguang Wu wrote: > On Wed, Nov 07, 2012 at 03:43:05PM -0500, Chris Ball wrote: > > Hi, > > > > On Wed, Nov 07 2012, kbuild test robot wrote: > > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git mmc-next > > > head: a6a567a4c25762e36228d2d0c8c1859085132e15 > > > commit: 90ba

Re: [PATCH 2/2] mmc: only support voltage (vdd) that regulator agree's with

2012-11-09 Thread Tushar Behera
On 06/09/2012 12:56 AM, philipspatc...@gmail.com wrote: > From: Philip Rakity > > If we are using a regulator the SD Host Controller and the > regulator should agree about the voltages supported. Use > the common subset that is supported. > > Signed-off-by: Philip Rakity > --- > drivers/mmc/h