* [PATCH] ASoC: Fix warning in WM1250-EV1 driver
@ 2011-08-09 15:25 Mark Brown
2011-08-09 15:29 ` Liam Girdwood
0 siblings, 1 reply; 2+ messages in thread
From: Mark Brown @ 2011-08-09 15:25 UTC (permalink / raw)
To: Liam Girdwood; +Cc: alsa-devel, patches, Mark Brown
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
sound/soc/codecs/wm1250-ev1.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/soc/codecs/wm1250-ev1.c b/sound/soc/codecs/wm1250-ev1.c
index bbcf9ec..a98a3ff1 100644
--- a/sound/soc/codecs/wm1250-ev1.c
+++ b/sound/soc/codecs/wm1250-ev1.c
@@ -58,12 +58,12 @@ static struct snd_soc_codec_driver soc_codec_dev_wm1250_ev1 = {
static int __devinit wm1250_ev1_probe(struct i2c_client *i2c,
const struct i2c_device_id *i2c_id)
{
- int ret, id, board, rev;
+ int id, board, rev;
board = i2c_smbus_read_byte_data(i2c, 0);
if (board < 0) {
- dev_err(&i2c->dev, "Failed to read ID: %d\n", ret);
- return ret;
+ dev_err(&i2c->dev, "Failed to read ID: %d\n", board);
+ return board;
}
id = (board & 0xfe) >> 2;
--
1.7.5.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ASoC: Fix warning in WM1250-EV1 driver
2011-08-09 15:25 [PATCH] ASoC: Fix warning in WM1250-EV1 driver Mark Brown
@ 2011-08-09 15:29 ` Liam Girdwood
0 siblings, 0 replies; 2+ messages in thread
From: Liam Girdwood @ 2011-08-09 15:29 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel@Alsa-project.org, patches@opensource.wolfsonmicro.com
On 09/08/11 16:25, Mark Brown wrote:
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> ---
> sound/soc/codecs/wm1250-ev1.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
Acked-by: Liam Girdwood <lrg@ti.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-08-09 15:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-09 15:25 [PATCH] ASoC: Fix warning in WM1250-EV1 driver Mark Brown
2011-08-09 15:29 ` Liam Girdwood
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox