[PATCH 2/2] soundwire: sysfs: add slave status and device number before probe
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Thu Sep 17 14:54:50 CEST 2020
Thanks for the review Greg,
>> +int sdw_slave_status_sysfs_init(struct sdw_slave *slave)
>> +{
>> + return device_add_group(&slave->dev, &sdw_slave_status_attr_group);
>
> DOesn't this race with userspace? Why not make this part of the default
> set of device attributes and have the driver core create them
> automatically?
What did you mean by 'default set of device attributes', would you mind
providing a pointer to an example so that I can look into this?
What we have in this patch is added by the SoundWire core but you're
probably thinking of something else. thanks for enlightening the rest of
us:-)
More information about the Alsa-devel
mailing list