ALSA: hwdep: Take private_data as drvdata for sysfs

For referring to a different object from sysfs ops, take hwdep
private_data as stored via dev_set_drvdata() at creating the device
object.  In that way, the same sysfs ops can be used by different
device types.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2014-02-25 10:00:58 +01:00
parent caa751bad4
commit f806bdb2f7
2 changed files with 17 additions and 30 deletions

View file

@ -441,6 +441,8 @@ static int snd_hwdep_dev_register(struct snd_device *device)
struct device *d = snd_get_device(SNDRV_DEVICE_TYPE_HWDEP, struct device *d = snd_get_device(SNDRV_DEVICE_TYPE_HWDEP,
hwdep->card, hwdep->device); hwdep->card, hwdep->device);
if (d) { if (d) {
if (hwdep->private_data)
dev_set_drvdata(d, hwdep->private_data);
err = sysfs_create_groups(&d->kobj, hwdep->groups); err = sysfs_create_groups(&d->kobj, hwdep->groups);
if (err < 0) if (err < 0)
dev_warn(card->dev, dev_warn(card->dev,

View file

@ -163,8 +163,7 @@ static ssize_t power_on_acct_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
snd_hda_update_power_acct(codec); snd_hda_update_power_acct(codec);
return sprintf(buf, "%u\n", jiffies_to_msecs(codec->power_on_acct)); return sprintf(buf, "%u\n", jiffies_to_msecs(codec->power_on_acct));
} }
@ -173,8 +172,7 @@ static ssize_t power_off_acct_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
snd_hda_update_power_acct(codec); snd_hda_update_power_acct(codec);
return sprintf(buf, "%u\n", jiffies_to_msecs(codec->power_off_acct)); return sprintf(buf, "%u\n", jiffies_to_msecs(codec->power_off_acct));
} }
@ -251,8 +249,7 @@ static ssize_t type##_show(struct device *dev, \
struct device_attribute *attr, \ struct device_attribute *attr, \
char *buf) \ char *buf) \
{ \ { \
struct snd_hwdep *hwdep = dev_get_drvdata(dev); \ struct hda_codec *codec = dev_get_drvdata(dev); \
struct hda_codec *codec = hwdep->private_data; \
return sprintf(buf, "0x%x\n", codec->type); \ return sprintf(buf, "0x%x\n", codec->type); \
} }
@ -261,8 +258,7 @@ static ssize_t type##_show(struct device *dev, \
struct device_attribute *attr, \ struct device_attribute *attr, \
char *buf) \ char *buf) \
{ \ { \
struct snd_hwdep *hwdep = dev_get_drvdata(dev); \ struct hda_codec *codec = dev_get_drvdata(dev); \
struct hda_codec *codec = hwdep->private_data; \
return sprintf(buf, "%s\n", \ return sprintf(buf, "%s\n", \
codec->type ? codec->type : ""); \ codec->type ? codec->type : ""); \
} }
@ -281,8 +277,7 @@ static ssize_t type##_store(struct device *dev, \
struct device_attribute *attr, \ struct device_attribute *attr, \
const char *buf, size_t count) \ const char *buf, size_t count) \
{ \ { \
struct snd_hwdep *hwdep = dev_get_drvdata(dev); \ struct hda_codec *codec = dev_get_drvdata(dev); \
struct hda_codec *codec = hwdep->private_data; \
unsigned long val; \ unsigned long val; \
int err = kstrtoul(buf, 0, &val); \ int err = kstrtoul(buf, 0, &val); \
if (err < 0) \ if (err < 0) \
@ -296,8 +291,7 @@ static ssize_t type##_store(struct device *dev, \
struct device_attribute *attr, \ struct device_attribute *attr, \
const char *buf, size_t count) \ const char *buf, size_t count) \
{ \ { \
struct snd_hwdep *hwdep = dev_get_drvdata(dev); \ struct hda_codec *codec = dev_get_drvdata(dev); \
struct hda_codec *codec = hwdep->private_data; \
char *s = kstrndup_noeol(buf, 64); \ char *s = kstrndup_noeol(buf, 64); \
if (!s) \ if (!s) \
return -ENOMEM; \ return -ENOMEM; \
@ -318,8 +312,7 @@ static ssize_t type##_store(struct device *dev, \
struct device_attribute *attr, \ struct device_attribute *attr, \
const char *buf, size_t count) \ const char *buf, size_t count) \
{ \ { \
struct snd_hwdep *hwdep = dev_get_drvdata(dev); \ struct hda_codec *codec = dev_get_drvdata(dev); \
struct hda_codec *codec = hwdep->private_data; \
int err = 0; \ int err = 0; \
if (*buf) \ if (*buf) \
err = type##_codec(codec); \ err = type##_codec(codec); \
@ -333,8 +326,7 @@ static ssize_t init_verbs_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
int i, len = 0; int i, len = 0;
mutex_lock(&codec->user_mutex); mutex_lock(&codec->user_mutex);
for (i = 0; i < codec->init_verbs.used; i++) { for (i = 0; i < codec->init_verbs.used; i++) {
@ -373,8 +365,7 @@ static ssize_t init_verbs_store(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
int err = parse_init_verbs(codec, buf); int err = parse_init_verbs(codec, buf);
if (err < 0) if (err < 0)
return err; return err;
@ -385,8 +376,7 @@ static ssize_t hints_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
int i, len = 0; int i, len = 0;
mutex_lock(&codec->user_mutex); mutex_lock(&codec->user_mutex);
for (i = 0; i < codec->hints.used; i++) { for (i = 0; i < codec->hints.used; i++) {
@ -480,8 +470,7 @@ static ssize_t hints_store(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
int err = parse_hints(codec, buf); int err = parse_hints(codec, buf);
if (err < 0) if (err < 0)
return err; return err;
@ -507,8 +496,7 @@ static ssize_t init_pin_configs_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
return pin_configs_show(codec, &codec->init_pins, buf); return pin_configs_show(codec, &codec->init_pins, buf);
} }
@ -516,8 +504,7 @@ static ssize_t user_pin_configs_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
return pin_configs_show(codec, &codec->user_pins, buf); return pin_configs_show(codec, &codec->user_pins, buf);
} }
@ -525,8 +512,7 @@ static ssize_t driver_pin_configs_show(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
char *buf) char *buf)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
return pin_configs_show(codec, &codec->driver_pins, buf); return pin_configs_show(codec, &codec->driver_pins, buf);
} }
@ -550,8 +536,7 @@ static ssize_t user_pin_configs_store(struct device *dev,
struct device_attribute *attr, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
{ {
struct snd_hwdep *hwdep = dev_get_drvdata(dev); struct hda_codec *codec = dev_get_drvdata(dev);
struct hda_codec *codec = hwdep->private_data;
int err = parse_user_pin_configs(codec, buf); int err = parse_user_pin_configs(codec, buf);
if (err < 0) if (err < 0)
return err; return err;