public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
* SD Patches
@ 2011-04-27  2:50 Philip Rakity
  2011-04-28  2:13 ` Subhash Jadavani
  0 siblings, 1 reply; 3+ messages in thread
From: Philip Rakity @ 2011-04-27  2:50 UTC (permalink / raw)
  To: Chris Ball; +Cc: linux-mmc@vger.kernel.org, Arindam Nath


Chris,

It is important that the patches are put into the linux code for testing.

I have tested and reviewed the code but NOT extensively.

It works for me doing eMMC (DDR)  and SDIO (SDR50).

I certainly have no issue with saying I have reviewed the code and tested it -- but again only in the areas I am looking at.

With this provision feel free to add 

Reviewed and Tested-by to the code so we can move forward with its integration into mmc-next.

Philip


^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: SD Patches
  2011-04-27  2:50 SD Patches Philip Rakity
@ 2011-04-28  2:13 ` Subhash Jadavani
  2011-04-28  8:31   ` Wolfram Sang
  0 siblings, 1 reply; 3+ messages in thread
From: Subhash Jadavani @ 2011-04-28  2:13 UTC (permalink / raw)
  To: 'Philip Rakity', 'Chris Ball'
  Cc: linux-mmc, 'Arindam Nath'

I have also reviewed Arindam's SD3.0 patches and looks good to me. Although
I haven't tested those yet.

Regards,
Subhash

> -----Original Message-----
> From: linux-mmc-owner@vger.kernel.org [mailto:linux-mmc-
> owner@vger.kernel.org] On Behalf Of Philip Rakity
> Sent: Tuesday, April 26, 2011 7:51 PM
> To: Chris Ball
> Cc: linux-mmc@vger.kernel.org; Arindam Nath
> Subject: SD Patches
> 
> 
> Chris,
> 
> It is important that the patches are put into the linux code for
> testing.
> 
> I have tested and reviewed the code but NOT extensively.
> 
> It works for me doing eMMC (DDR)  and SDIO (SDR50).
> 
> I certainly have no issue with saying I have reviewed the code and
> tested it -- but again only in the areas I am looking at.
> 
> With this provision feel free to add
> 
> Reviewed and Tested-by to the code so we can move forward with its
> integration into mmc-next.
> 
> Philip
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: SD Patches
  2011-04-28  2:13 ` Subhash Jadavani
@ 2011-04-28  8:31   ` Wolfram Sang
  0 siblings, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2011-04-28  8:31 UTC (permalink / raw)
  To: Subhash Jadavani
  Cc: 'Philip Rakity', 'Chris Ball', linux-mmc,
	'Arindam Nath'

[-- Attachment #1: Type: text/plain, Size: 682 bytes --]

On Wed, Apr 27, 2011 at 07:13:22PM -0700, Subhash Jadavani wrote:
> I have also reviewed Arindam's SD3.0 patches and looks good to me. Although
> I haven't tested those yet.

Thanks. To make it easier next time: The usual pattern for this is replying
directly to the series to [PATCH 0/x] (if you mean the whole series) or reply
to each [PATCH n/x] with something like

Reviewed-by: Your name <your@address>

Tested-by: Your name <your@address>

(I have keyboard macros for those ;))

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-04-28  8:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-27  2:50 SD Patches Philip Rakity
2011-04-28  2:13 ` Subhash Jadavani
2011-04-28  8:31   ` Wolfram Sang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox