Re: [PATCH 0/7] mmc: omap: Fixes, cleanup and add ERASE capability

2013-12-06 Thread Aaro Koskinen
Hi,

On Wed, Dec 04, 2013 at 09:14:07PM +, Jarkko Nikula wrote:
 I've sent these patches earlier to linux-mmc list. This time set is one
 patch smaller since commit 31ee9181eb92
 (mmc: omap: Fix DMA configuration to not rely on device id) does
 practically same what one of my patch did.
 
 Patch 1 is an obvious fix. 2-4 are preparing for 5 which also should fix
 the error introduced by 31ee9181eb92:
 http://www.spinics.net/lists/linux-omap/msg100790.html
 
 Patch 6 continue cleanup and 7 adds the ERASE capability to the driver so
 one could utilize discards with mkfs, mount option or with the fstrim.
 
 Set goes on top of v3.13-rc2-208-g8ecffd791448.

Patch 5 had some fuzz with plain 3.13-rc2. Otherwise these look fine.
I tested these on 770 and N800, so for all patches:

Tested-by: Aaro Koskinen aaro.koski...@iki.fi

A.
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/7] mmc: omap: Fixes, cleanup and add ERASE capability

2013-12-06 Thread Jarkko Nikula
On 12/06/2013 01:41 PM, Aaro Koskinen wrote:
 Set goes on top of v3.13-rc2-208-g8ecffd791448.
 
 Patch 5 had some fuzz with plain 3.13-rc2. Otherwise these look fine.
 I tested these on 770 and N800, so for all patches:
 
 Tested-by: Aaro Koskinen aaro.koski...@iki.fi
 
Yes, reason for your fuzz observation is that 3.13-rc2 don't have those
two patches from Tony but 8ecffd791448 had.

-- 
Jarkko
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/7] mmc: omap: Fixes, cleanup and add ERASE capability

2013-12-05 Thread Tony Lindgren
* Jarkko Nikula jarkko.nik...@bitmer.com [131204 11:15]:
 Hi
 
 I've sent these patches earlier to linux-mmc list. This time set is one
 patch smaller since commit 31ee9181eb92
 (mmc: omap: Fix DMA configuration to not rely on device id) does
 practically same what one of my patch did.
 
 Patch 1 is an obvious fix. 2-4 are preparing for 5 which also should fix
 the error introduced by 31ee9181eb92:
 http://www.spinics.net/lists/linux-omap/msg100790.html
 
 Patch 6 continue cleanup and 7 adds the ERASE capability to the driver so
 one could utilize discards with mkfs, mount option or with the fstrim.
 
 Set goes on top of v3.13-rc2-208-g8ecffd791448.

Great they look good to me:

Acked-by: Tony Lindgren t...@atomide.com
 
 Jarkko Nikula (7):
   mmc: omap: Fix NULL pointer dereference due uninitialized
 cover_tasklet
   mmc: omap: Convert to devm_kzalloc
   mmc: omap: Remove duplicate host-irq assignment
   mmc: omap: Remove mem_res field from struct mmc_omap_host
   mmc: omap: Convert to devm_ioremap_resource
   mmc: omap: Remove always set use_dma flag from struct mmc_omap_host
   mmc: omap: Add erase capability
 
  drivers/mmc/host/omap.c | 93 
 +++--
  1 file changed, 36 insertions(+), 57 deletions(-)
 
 -- 
 1.8.4.3
 
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 0/7] mmc: omap: Fixes, cleanup and add ERASE capability

2013-12-04 Thread Jarkko Nikula
Hi

I've sent these patches earlier to linux-mmc list. This time set is one
patch smaller since commit 31ee9181eb92
(mmc: omap: Fix DMA configuration to not rely on device id) does
practically same what one of my patch did.

Patch 1 is an obvious fix. 2-4 are preparing for 5 which also should fix
the error introduced by 31ee9181eb92:
http://www.spinics.net/lists/linux-omap/msg100790.html

Patch 6 continue cleanup and 7 adds the ERASE capability to the driver so
one could utilize discards with mkfs, mount option or with the fstrim.

Set goes on top of v3.13-rc2-208-g8ecffd791448.

Jarkko Nikula (7):
  mmc: omap: Fix NULL pointer dereference due uninitialized
cover_tasklet
  mmc: omap: Convert to devm_kzalloc
  mmc: omap: Remove duplicate host-irq assignment
  mmc: omap: Remove mem_res field from struct mmc_omap_host
  mmc: omap: Convert to devm_ioremap_resource
  mmc: omap: Remove always set use_dma flag from struct mmc_omap_host
  mmc: omap: Add erase capability

 drivers/mmc/host/omap.c | 93 +++--
 1 file changed, 36 insertions(+), 57 deletions(-)

-- 
1.8.4.3

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html