Skip to content

Commit

Permalink
ALSA: ASoC: Drop device registration from GTA01 lm4857 driver
Browse files Browse the repository at this point in the history
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: Jonas Bonn <jonas.bonn@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
  • Loading branch information
Jonas Bonn authored and Takashi Iwai committed Oct 13, 2008
1 parent df20cf9 commit f9d1ab3
Showing 1 changed file with 6 additions and 45 deletions.
51 changes: 6 additions & 45 deletions sound/soc/s3c24xx/neo1973_wm8753.c
Original file line number Diff line number Diff line change
Expand Up @@ -602,6 +602,8 @@ static int lm4857_i2c_probe(struct i2c_client *client,
{
DBG("Entered %s\n", __func__);

i2c = client;

lm4857_write_regs();
return 0;
}
Expand All @@ -610,6 +612,8 @@ static int lm4857_i2c_remove(struct i2c_client *client)
{
DBG("Entered %s\n", __func__);

i2c = NULL;

return 0;
}

Expand Down Expand Up @@ -667,48 +671,6 @@ static struct i2c_driver lm4857_i2c_driver = {
};

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)
{
Expand All @@ -735,8 +697,8 @@ static int __init neo1973_init(void)
return ret;
}

ret = neo1973_add_lm4857_device(neo1973_snd_device,
0, 0x7C);
ret = i2c_add_driver(&lm4857_i2c_driver);

if (ret != 0)
platform_device_unregister(neo1973_snd_device);

Expand All @@ -747,7 +709,6 @@ static void __exit neo1973_exit(void)
{
DBG("Entered %s\n", __func__);

i2c_unregister_device(lm4857_client);
i2c_del_driver(&lm4857_i2c_driver);
platform_device_unregister(neo1973_snd_device);
}
Expand Down

0 comments on commit f9d1ab3

Please sign in to comment.