Hi Stephen, On Mon, Jun 10, 2013 at 03:11:14PM -0600, Stephen Warren wrote: > On 06/09/2013 02:14 PM, Simon Baatz wrote: > > Signed-off-by: Simon Baatz <gmbnomis@xxxxxxxxx> > > Tested-by: Stephen Warren <swarren@xxxxxxxxxx> Thanks! > (On Seaboard/Springbank board, i.e. Tegra20) > > > diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c > > > +err_parse_dt: > > err_power_req: > > err_alloc_tegra_host: > > Nit: It'd be nice if that new label got inserted into the middle of the > list, so err_power_req, err_parse_dt, err_alloc_tegra_host. That way if > we ever needed to add separate code there, they'd be in the right order > already. Still, this isn't worth fixing unless you have to repost for > some other reason I guess. Yes, you are right. I updated the patch locally, so the update will be in if I need to repost (or if the nicer version is explicitly requested). Since this change is only syntactical, I simply assume that your Tested-by is still valid even with the change ;-) - Simon -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html