Commit 28760c19 authored by Wolfram Sang's avatar Wolfram Sang Committed by Takashi Iwai

ALSA: ppc: keywest: check if DEQ was already instantiated

Due to changes in i2c-powermac, for some Macs the DEQ is instantiated
now in i2c-powermac while some need the instantiation here in the
keywest sound driver. The proper fix is to convert this driver to AOA.
Until then support both ways of instantiation. Before this patch, some
Macs lost sound because the sound driver failed when instantiating the
DEQ for the second time.
Signed-off-by: default avatarWolfram Sang <wsa@the-dreams.de>
Tested-by: default avatarDan DeVoto <dand1972@yahoo.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 225068ab
...@@ -31,10 +31,15 @@ ...@@ -31,10 +31,15 @@
*/ */
static struct pmac_keywest *keywest_ctx; static struct pmac_keywest *keywest_ctx;
static bool keywest_probed;
static int keywest_probe(struct i2c_client *client, static int keywest_probe(struct i2c_client *client,
const struct i2c_device_id *id) const struct i2c_device_id *id)
{ {
keywest_probed = true;
/* If instantiated via i2c-powermac, we still need to set the client */
if (!keywest_ctx->client)
keywest_ctx->client = client;
i2c_set_clientdata(client, keywest_ctx); i2c_set_clientdata(client, keywest_ctx);
return 0; return 0;
} }
...@@ -92,7 +97,8 @@ static int keywest_remove(struct i2c_client *client) ...@@ -92,7 +97,8 @@ static int keywest_remove(struct i2c_client *client)
static const struct i2c_device_id keywest_i2c_id[] = { static const struct i2c_device_id keywest_i2c_id[] = {
{ "keywest", 0 }, { "MAC,tas3004", 0 }, /* instantiated by i2c-powermac */
{ "keywest", 0 }, /* instantiated by us if needed */
{ } { }
}; };
...@@ -149,8 +155,13 @@ int snd_pmac_keywest_init(struct pmac_keywest *i2c) ...@@ -149,8 +155,13 @@ int snd_pmac_keywest_init(struct pmac_keywest *i2c)
return err; return err;
} }
/* There was already a device from i2c-powermac. Great, let's return */
if (keywest_probed)
return 0;
/* We assume Macs have consecutive I2C bus numbers starting at 0 */ /* We assume Macs have consecutive I2C bus numbers starting at 0 */
while (adap) { while (adap) {
/* Scan for devices to be bound to */
err = keywest_attach_adapter(adap); err = keywest_attach_adapter(adap);
if (!err) if (!err)
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment