[RESEND PATCH V3 6/8] mmc: tegra: handle mmc_of_parse() errors during probe
Simon Baatz
gmbnomis at gmail.com
Tue Jun 11 08:02:30 EST 2013
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 at gmail.com>
>
> Tested-by: Stephen Warren <swarren at nvidia.com>
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
More information about the devicetree-discuss
mailing list