Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch
@ 2011-11-19  2:15 Axel Lin
  2011-11-19  2:17 ` [PATCH 2/2] ASoC: cs42l73: Show correct revision id Axel Lin
  2011-11-21 17:06 ` [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Mark Brown
  0 siblings, 2 replies; 4+ messages in thread
From: Axel Lin @ 2011-11-19  2:15 UTC (permalink / raw)
  To: alsa-devel
  Cc: Vinod Koul, Liam Girdwood, Mark Brown, Brian Austin, Georgi Vlaev

Return -ENODEV instead of 0 if device id mismatch.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 sound/soc/codecs/cs42l73.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/sound/soc/codecs/cs42l73.c b/sound/soc/codecs/cs42l73.c
index aadfd6b..3eed21e 100644
--- a/sound/soc/codecs/cs42l73.c
+++ b/sound/soc/codecs/cs42l73.c
@@ -1370,6 +1370,7 @@ static __devinit int cs42l73_i2c_probe(struct i2c_client *i2c_client,
 
 
 	if (devid != CS42L73_DEVID) {
+		ret = -ENODEV;
 		dev_err(&i2c_client->dev,
 			"CS42L73 Device ID (%X). Expected %X\n",
 			devid, CS42L73_DEVID);
-- 
1.7.5.4

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

* [PATCH 2/2] ASoC: cs42l73: Show correct revision id
  2011-11-19  2:15 [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Axel Lin
@ 2011-11-19  2:17 ` Axel Lin
  2011-11-21 17:06   ` Mark Brown
  2011-11-21 17:06 ` [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Mark Brown
  1 sibling, 1 reply; 4+ messages in thread
From: Axel Lin @ 2011-11-19  2:17 UTC (permalink / raw)
  To: alsa-devel
  Cc: Vinod Koul, Liam Girdwood, Mark Brown, Brian Austin, Georgi Vlaev

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 sound/soc/codecs/cs42l73.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/soc/codecs/cs42l73.c b/sound/soc/codecs/cs42l73.c
index 3eed21e..066d869 100644
--- a/sound/soc/codecs/cs42l73.c
+++ b/sound/soc/codecs/cs42l73.c
@@ -1384,7 +1384,7 @@ static __devinit int cs42l73_i2c_probe(struct i2c_client *i2c_client,
 	}
 
 	dev_info(&i2c_client->dev,
-		 "Cirrus Logic CS42L73, Revision: %02X\n", ret & 0xFF);
+		 "Cirrus Logic CS42L73, Revision: %02X\n", reg & 0xFF);
 
 	regcache_cache_only(cs42l73->regmap, true);
 
-- 
1.7.5.4

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

* Re: [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch
  2011-11-19  2:15 [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Axel Lin
  2011-11-19  2:17 ` [PATCH 2/2] ASoC: cs42l73: Show correct revision id Axel Lin
@ 2011-11-21 17:06 ` Mark Brown
  1 sibling, 0 replies; 4+ messages in thread
From: Mark Brown @ 2011-11-21 17:06 UTC (permalink / raw)
  To: Axel Lin
  Cc: Brian Austin, alsa-devel, Liam Girdwood, Vinod Koul, Georgi Vlaev

On Sat, Nov 19, 2011 at 10:15:53AM +0800, Axel Lin wrote:
> Return -ENODEV instead of 0 if device id mismatch.

Applied, thanks.

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

* Re: [PATCH 2/2] ASoC: cs42l73: Show correct revision id
  2011-11-19  2:17 ` [PATCH 2/2] ASoC: cs42l73: Show correct revision id Axel Lin
@ 2011-11-21 17:06   ` Mark Brown
  0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2011-11-21 17:06 UTC (permalink / raw)
  To: Axel Lin
  Cc: Brian Austin, alsa-devel, Liam Girdwood, Vinod Koul, Georgi Vlaev

On Sat, Nov 19, 2011 at 10:17:36AM +0800, Axel Lin wrote:
> Signed-off-by: Axel Lin <axel.lin@gmail.com>

Applied, thanks.

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

end of thread, other threads:[~2011-11-21 17:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-19  2:15 [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Axel Lin
2011-11-19  2:17 ` [PATCH 2/2] ASoC: cs42l73: Show correct revision id Axel Lin
2011-11-21 17:06   ` Mark Brown
2011-11-21 17:06 ` [PATCH 1/2] ASoC: cs42l73: Return proper error code if device id mismatch Mark Brown

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