Commit f9d1ab39 authored by Jonas Bonn's avatar Jonas Bonn Committed by Takashi Iwai

ALSA: ASoC: Drop device registration from GTA01 lm4857 driver

Device registration should be handled at the machine level and not
in the driver code itself.  This patch removes the device registration
from the driver code in preparation for moving it to the machine
definition.

[Squashed down two parts to this patch for bisectability - there's also
a third part adding registration of the device to the out of tree GTA01
machine driver -- broonie]
Signed-off-by: default avatarJonas Bonn <jonas.bonn@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent df20cf92
...@@ -602,6 +602,8 @@ static int lm4857_i2c_probe(struct i2c_client *client, ...@@ -602,6 +602,8 @@ static int lm4857_i2c_probe(struct i2c_client *client,
{ {
DBG("Entered %s\n", __func__); DBG("Entered %s\n", __func__);
i2c = client;
lm4857_write_regs(); lm4857_write_regs();
return 0; return 0;
} }
...@@ -610,6 +612,8 @@ static int lm4857_i2c_remove(struct i2c_client *client) ...@@ -610,6 +612,8 @@ static int lm4857_i2c_remove(struct i2c_client *client)
{ {
DBG("Entered %s\n", __func__); DBG("Entered %s\n", __func__);
i2c = NULL;
return 0; return 0;
} }
...@@ -667,48 +671,6 @@ static struct i2c_driver lm4857_i2c_driver = { ...@@ -667,48 +671,6 @@ static struct i2c_driver lm4857_i2c_driver = {
}; };
static struct platform_device *neo1973_snd_device; static struct platform_device *neo1973_snd_device;
static struct i2c_client *lm4857_client;
static int __init neo1973_add_lm4857_device(struct platform_device *pdev,
int i2c_bus,
unsigned short i2c_address)
{
struct i2c_board_info info;
struct i2c_adapter *adapter;
struct i2c_client *client;
int ret;
ret = i2c_add_driver(&lm4857_i2c_driver);
if (ret != 0) {
dev_err(&pdev->dev, "can't add lm4857 driver\n");
return ret;
}
memset(&info, 0, sizeof(struct i2c_board_info));
info.addr = i2c_address;
strlcpy(info.type, "neo1973_lm4857", I2C_NAME_SIZE);
adapter = i2c_get_adapter(i2c_bus);
if (!adapter) {
dev_err(&pdev->dev, "can't get i2c adapter %d\n", i2c_bus);
goto err_driver;
}
client = i2c_new_device(adapter, &info);
i2c_put_adapter(adapter);
if (!client) {
dev_err(&pdev->dev, "can't add lm4857 device at 0x%x\n",
(unsigned int)info.addr);
goto err_driver;
}
lm4857_client = client;
return 0;
err_driver:
i2c_del_driver(&lm4857_i2c_driver);
return -ENODEV;
}
static int __init neo1973_init(void) static int __init neo1973_init(void)
{ {
...@@ -735,8 +697,8 @@ static int __init neo1973_init(void) ...@@ -735,8 +697,8 @@ static int __init neo1973_init(void)
return ret; return ret;
} }
ret = neo1973_add_lm4857_device(neo1973_snd_device, ret = i2c_add_driver(&lm4857_i2c_driver);
0, 0x7C);
if (ret != 0) if (ret != 0)
platform_device_unregister(neo1973_snd_device); platform_device_unregister(neo1973_snd_device);
...@@ -747,7 +709,6 @@ static void __exit neo1973_exit(void) ...@@ -747,7 +709,6 @@ static void __exit neo1973_exit(void)
{ {
DBG("Entered %s\n", __func__); DBG("Entered %s\n", __func__);
i2c_unregister_device(lm4857_client);
i2c_del_driver(&lm4857_i2c_driver); i2c_del_driver(&lm4857_i2c_driver);
platform_device_unregister(neo1973_snd_device); platform_device_unregister(neo1973_snd_device);
} }
......
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