Hi!

> Add more error messages making it easier to identify problems.
> 
> Signed-off-by: Sebastian Reichel <[email protected]>
> ---
>  sound/soc/omap/rx51.c | 16 +++++++++++++---
>  1 file changed, 13 insertions(+), 3 deletions(-)

> +     if (err) {
> +             dev_err(card->dev, "Failed to add GPIOs\n");
> +             return err;
> +     }
>  
>       return err;
>  }


As you'll return err, anyway, you don't need to do it inside the if()... OTOH 
it 
does not hurt.

Acked-by: Pavel Machek <[email protected]>

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to