* sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
@ 2024-01-04 14:22 kernel test robot
0 siblings, 0 replies; 5+ messages in thread
From: kernel test robot @ 2024-01-04 14:22 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "only kconfig file changed"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Geert Uytterhoeven <geert@linux-m68k.org>
CC: Mark Brown <broonie@kernel.org>
Hi Geert,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ac865f00af293d081356bec56eea90815094a60e
commit: ea00d95200d02ece71f5814d41b14f2eb16d598b ASoC: Use imply for SND_SOC_ALL_CODECS
date: 3 years, 11 months ago
:::::: branch date: 16 hours ago
:::::: commit date: 3 years, 11 months ago
config: x86_64-buildonly-randconfig-003-20231102 (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202401042239.2zkHgzki-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:192: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:205: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:205: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:351: warning: Function parameter or member 'codecs_pdata' not described in 'snd_ac97_controller_register'
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for REGMAP_SPI
Depends on [n]: SPI [=n]
Selected by [m]:
- SND_SOC_ADAU1781_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
- SND_SOC_ADAU1977_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
vim +192 sound/ac97/bus.c
74426fbff66eea Robert Jarzmik 2017-09-02 181
74426fbff66eea Robert Jarzmik 2017-09-02 182 /**
74426fbff66eea Robert Jarzmik 2017-09-02 183 * snd_ac97_codec_driver_register - register an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 184 * @dev: AC97 driver codec to register
74426fbff66eea Robert Jarzmik 2017-09-02 185 *
74426fbff66eea Robert Jarzmik 2017-09-02 186 * Register an AC97 codec driver to the ac97 bus driver, aka. the AC97 digital
74426fbff66eea Robert Jarzmik 2017-09-02 187 * controller.
74426fbff66eea Robert Jarzmik 2017-09-02 188 *
74426fbff66eea Robert Jarzmik 2017-09-02 189 * Returns 0 on success or error code
74426fbff66eea Robert Jarzmik 2017-09-02 190 */
74426fbff66eea Robert Jarzmik 2017-09-02 191 int snd_ac97_codec_driver_register(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @192 {
74426fbff66eea Robert Jarzmik 2017-09-02 193 drv->driver.bus = &ac97_bus_type;
74426fbff66eea Robert Jarzmik 2017-09-02 194 return driver_register(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 195 }
74426fbff66eea Robert Jarzmik 2017-09-02 196 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_register);
74426fbff66eea Robert Jarzmik 2017-09-02 197
74426fbff66eea Robert Jarzmik 2017-09-02 198 /**
74426fbff66eea Robert Jarzmik 2017-09-02 199 * snd_ac97_codec_driver_unregister - unregister an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 200 * @dev: AC97 codec driver to unregister
74426fbff66eea Robert Jarzmik 2017-09-02 201 *
74426fbff66eea Robert Jarzmik 2017-09-02 202 * Unregister a previously registered ac97 codec driver.
74426fbff66eea Robert Jarzmik 2017-09-02 203 */
74426fbff66eea Robert Jarzmik 2017-09-02 204 void snd_ac97_codec_driver_unregister(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @205 {
74426fbff66eea Robert Jarzmik 2017-09-02 206 driver_unregister(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 207 }
74426fbff66eea Robert Jarzmik 2017-09-02 208 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_unregister);
74426fbff66eea Robert Jarzmik 2017-09-02 209
74426fbff66eea Robert Jarzmik 2017-09-02 210 /**
74426fbff66eea Robert Jarzmik 2017-09-02 211 * snd_ac97_codec_get_platdata - get platform_data
74426fbff66eea Robert Jarzmik 2017-09-02 212 * @adev: the ac97 codec device
74426fbff66eea Robert Jarzmik 2017-09-02 213 *
74426fbff66eea Robert Jarzmik 2017-09-02 214 * For legacy platforms, in order to have platform_data in codec drivers
74426fbff66eea Robert Jarzmik 2017-09-02 215 * available, while ac97 device are auto-created upon probe, this retrieves the
74426fbff66eea Robert Jarzmik 2017-09-02 216 * platdata which was setup on ac97 controller registration.
74426fbff66eea Robert Jarzmik 2017-09-02 217 *
74426fbff66eea Robert Jarzmik 2017-09-02 218 * Returns the platform data pointer
74426fbff66eea Robert Jarzmik 2017-09-02 219 */
74426fbff66eea Robert Jarzmik 2017-09-02 220 void *snd_ac97_codec_get_platdata(const struct ac97_codec_device *adev)
74426fbff66eea Robert Jarzmik 2017-09-02 221 {
74426fbff66eea Robert Jarzmik 2017-09-02 222 struct ac97_controller *ac97_ctrl = adev->ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 223
74426fbff66eea Robert Jarzmik 2017-09-02 224 return ac97_ctrl->codecs_pdata[adev->num];
74426fbff66eea Robert Jarzmik 2017-09-02 225 }
74426fbff66eea Robert Jarzmik 2017-09-02 226 EXPORT_SYMBOL_GPL(snd_ac97_codec_get_platdata);
74426fbff66eea Robert Jarzmik 2017-09-02 227
74426fbff66eea Robert Jarzmik 2017-09-02 228 static void ac97_ctrl_codecs_unregister(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 229 {
74426fbff66eea Robert Jarzmik 2017-09-02 230 int i;
74426fbff66eea Robert Jarzmik 2017-09-02 231
74426fbff66eea Robert Jarzmik 2017-09-02 232 for (i = 0; i < AC97_BUS_MAX_CODECS; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 233 if (ac97_ctrl->codecs[i]) {
74426fbff66eea Robert Jarzmik 2017-09-02 234 ac97_ctrl->codecs[i]->ac97_ctrl = &ac97_unbound_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 235 device_unregister(&ac97_ctrl->codecs[i]->dev);
74426fbff66eea Robert Jarzmik 2017-09-02 236 }
74426fbff66eea Robert Jarzmik 2017-09-02 237 }
74426fbff66eea Robert Jarzmik 2017-09-02 238
74426fbff66eea Robert Jarzmik 2017-09-02 239 static ssize_t cold_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 240 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 241 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 242 {
74426fbff66eea Robert Jarzmik 2017-09-02 243 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 244
74426fbff66eea Robert Jarzmik 2017-09-02 245 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 246 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 247 ac97_ctrl->ops->reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 248 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 249 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 250 }
74426fbff66eea Robert Jarzmik 2017-09-02 251 static DEVICE_ATTR_WO(cold_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 252
74426fbff66eea Robert Jarzmik 2017-09-02 253 static ssize_t warm_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 254 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 255 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 256 {
74426fbff66eea Robert Jarzmik 2017-09-02 257 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 258
74426fbff66eea Robert Jarzmik 2017-09-02 259 if (!dev)
74426fbff66eea Robert Jarzmik 2017-09-02 260 return -ENODEV;
74426fbff66eea Robert Jarzmik 2017-09-02 261
74426fbff66eea Robert Jarzmik 2017-09-02 262 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 263 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 264 ac97_ctrl->ops->warm_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 265 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 266 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 267 }
74426fbff66eea Robert Jarzmik 2017-09-02 268 static DEVICE_ATTR_WO(warm_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 269
74426fbff66eea Robert Jarzmik 2017-09-02 270 static struct attribute *ac97_controller_device_attrs[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 271 &dev_attr_cold_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 272 &dev_attr_warm_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 273 NULL
74426fbff66eea Robert Jarzmik 2017-09-02 274 };
74426fbff66eea Robert Jarzmik 2017-09-02 275
74426fbff66eea Robert Jarzmik 2017-09-02 276 static struct attribute_group ac97_adapter_attr_group = {
74426fbff66eea Robert Jarzmik 2017-09-02 277 .name = "ac97_operations",
74426fbff66eea Robert Jarzmik 2017-09-02 278 .attrs = ac97_controller_device_attrs,
74426fbff66eea Robert Jarzmik 2017-09-02 279 };
74426fbff66eea Robert Jarzmik 2017-09-02 280
74426fbff66eea Robert Jarzmik 2017-09-02 281 static const struct attribute_group *ac97_adapter_groups[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 282 &ac97_adapter_attr_group,
74426fbff66eea Robert Jarzmik 2017-09-02 283 NULL,
74426fbff66eea Robert Jarzmik 2017-09-02 284 };
74426fbff66eea Robert Jarzmik 2017-09-02 285
74426fbff66eea Robert Jarzmik 2017-09-02 286 static void ac97_del_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 287 {
74426fbff66eea Robert Jarzmik 2017-09-02 288 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 289 ac97_ctrl_codecs_unregister(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 290 list_del(&ac97_ctrl->controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 291 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 292
74426fbff66eea Robert Jarzmik 2017-09-02 293 device_unregister(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 294 }
74426fbff66eea Robert Jarzmik 2017-09-02 295
74426fbff66eea Robert Jarzmik 2017-09-02 296 static void ac97_adapter_release(struct device *dev)
74426fbff66eea Robert Jarzmik 2017-09-02 297 {
74426fbff66eea Robert Jarzmik 2017-09-02 298 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 299
74426fbff66eea Robert Jarzmik 2017-09-02 300 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 301 idr_remove(&ac97_adapter_idr, ac97_ctrl->nr);
74426fbff66eea Robert Jarzmik 2017-09-02 302 dev_dbg(&ac97_ctrl->adap, "adapter unregistered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 303 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 304 }
74426fbff66eea Robert Jarzmik 2017-09-02 305
74426fbff66eea Robert Jarzmik 2017-09-02 306 static const struct device_type ac97_adapter_type = {
74426fbff66eea Robert Jarzmik 2017-09-02 307 .groups = ac97_adapter_groups,
74426fbff66eea Robert Jarzmik 2017-09-02 308 .release = ac97_adapter_release,
74426fbff66eea Robert Jarzmik 2017-09-02 309 };
74426fbff66eea Robert Jarzmik 2017-09-02 310
74426fbff66eea Robert Jarzmik 2017-09-02 311 static int ac97_add_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 312 {
74426fbff66eea Robert Jarzmik 2017-09-02 313 int ret;
74426fbff66eea Robert Jarzmik 2017-09-02 314
74426fbff66eea Robert Jarzmik 2017-09-02 315 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 316 ret = idr_alloc(&ac97_adapter_idr, ac97_ctrl, 0, 0, GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 317 ac97_ctrl->nr = ret;
74426fbff66eea Robert Jarzmik 2017-09-02 318 if (ret >= 0) {
74426fbff66eea Robert Jarzmik 2017-09-02 319 dev_set_name(&ac97_ctrl->adap, "ac97-%d", ret);
74426fbff66eea Robert Jarzmik 2017-09-02 320 ac97_ctrl->adap.type = &ac97_adapter_type;
74426fbff66eea Robert Jarzmik 2017-09-02 321 ac97_ctrl->adap.parent = ac97_ctrl->parent;
74426fbff66eea Robert Jarzmik 2017-09-02 322 ret = device_register(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 323 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 324 put_device(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 325 }
74426fbff66eea Robert Jarzmik 2017-09-02 326 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 327 list_add(&ac97_ctrl->controllers, &ac97_controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 328 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 329
74426fbff66eea Robert Jarzmik 2017-09-02 330 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 331 dev_dbg(&ac97_ctrl->adap, "adapter registered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 332 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 333 return ret;
74426fbff66eea Robert Jarzmik 2017-09-02 334 }
74426fbff66eea Robert Jarzmik 2017-09-02 335
74426fbff66eea Robert Jarzmik 2017-09-02 336 /**
74426fbff66eea Robert Jarzmik 2017-09-02 337 * snd_ac97_controller_register - register an ac97 controller
74426fbff66eea Robert Jarzmik 2017-09-02 338 * @ops: the ac97 bus operations
74426fbff66eea Robert Jarzmik 2017-09-02 339 * @dev: the device providing the ac97 DC function
74426fbff66eea Robert Jarzmik 2017-09-02 340 * @slots_available: mask of the ac97 codecs that can be scanned and probed
74426fbff66eea Robert Jarzmik 2017-09-02 341 * bit0 => codec 0, bit1 => codec 1 ... bit 3 => codec 3
74426fbff66eea Robert Jarzmik 2017-09-02 342 *
74426fbff66eea Robert Jarzmik 2017-09-02 343 * Register a digital controller which can control up to 4 ac97 codecs. This is
74426fbff66eea Robert Jarzmik 2017-09-02 344 * the controller side of the AC97 AC-link, while the slave side are the codecs.
74426fbff66eea Robert Jarzmik 2017-09-02 345 *
74426fbff66eea Robert Jarzmik 2017-09-02 346 * Returns a valid controller upon success, negative pointer value upon error
74426fbff66eea Robert Jarzmik 2017-09-02 347 */
74426fbff66eea Robert Jarzmik 2017-09-02 348 struct ac97_controller *snd_ac97_controller_register(
74426fbff66eea Robert Jarzmik 2017-09-02 349 const struct ac97_controller_ops *ops, struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 350 unsigned short slots_available, void **codecs_pdata)
74426fbff66eea Robert Jarzmik 2017-09-02 @351 {
74426fbff66eea Robert Jarzmik 2017-09-02 352 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 353 int ret, i;
74426fbff66eea Robert Jarzmik 2017-09-02 354
74426fbff66eea Robert Jarzmik 2017-09-02 355 ac97_ctrl = kzalloc(sizeof(*ac97_ctrl), GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 356 if (!ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 357 return ERR_PTR(-ENOMEM);
74426fbff66eea Robert Jarzmik 2017-09-02 358
74426fbff66eea Robert Jarzmik 2017-09-02 359 for (i = 0; i < AC97_BUS_MAX_CODECS && codecs_pdata; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 360 ac97_ctrl->codecs_pdata[i] = codecs_pdata[i];
74426fbff66eea Robert Jarzmik 2017-09-02 361
74426fbff66eea Robert Jarzmik 2017-09-02 362 ac97_ctrl->ops = ops;
74426fbff66eea Robert Jarzmik 2017-09-02 363 ac97_ctrl->slots_available = slots_available;
74426fbff66eea Robert Jarzmik 2017-09-02 364 ac97_ctrl->parent = dev;
74426fbff66eea Robert Jarzmik 2017-09-02 365 ret = ac97_add_adapter(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 366
74426fbff66eea Robert Jarzmik 2017-09-02 367 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 368 goto err;
74426fbff66eea Robert Jarzmik 2017-09-02 369 ac97_bus_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 370 ac97_bus_scan(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 371
74426fbff66eea Robert Jarzmik 2017-09-02 372 return ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 373 err:
74426fbff66eea Robert Jarzmik 2017-09-02 374 kfree(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 375 return ERR_PTR(ret);
74426fbff66eea Robert Jarzmik 2017-09-02 376 }
74426fbff66eea Robert Jarzmik 2017-09-02 377 EXPORT_SYMBOL_GPL(snd_ac97_controller_register);
74426fbff66eea Robert Jarzmik 2017-09-02 378
:::::: The code at line 192 was first introduced by commit
:::::: 74426fbff66eea8e8d1f42c8238c268d1e63a832 ALSA: ac97: add an ac97 bus
:::::: TO: Robert Jarzmik <robert.jarzmik@free.fr>
:::::: CC: Mark Brown <broonie@kernel.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
2024-01-22 8:31 kernel test robot
@ 2024-01-22 8:42 ` Geert Uytterhoeven
0 siblings, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2024-01-22 8:42 UTC (permalink / raw)
To: kernel test robot; +Cc: oe-kbuild-all, linux-kernel, Mark Brown, Robert Jarzmik
Hi Kernel Test Robot,
On Mon, Jan 22, 2024 at 9:31 AM kernel test robot <lkp@intel.com> wrote:
> First bad commit (maybe != root cause):
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: ac865f00af293d081356bec56eea90815094a60e
> commit: ea00d95200d02ece71f5814d41b14f2eb16d598b ASoC: Use imply for SND_SOC_ALL_CODECS
> date: 3 years, 11 months ago
> :::::: branch date: 16 hours ago
> :::::: commit date: 3 years, 11 months ago
> config: x86_64-buildonly-randconfig-003-20231102 (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/r/202401042239.2zkHgzki-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> >> sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
> >> sound/ac97/bus.c:192: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_register'
> >> sound/ac97/bus.c:205: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_unregister'
> >> sound/ac97/bus.c:205: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_unregister'
> >> sound/ac97/bus.c:351: warning: Function parameter or member 'codecs_pdata' not described in 'snd_ac97_controller_register'
Definitely not introduced by my commit.
Origin is commit 74426fbff66eea8e ("ALSA: ac97: add an ac97 bus") in v4.15.
>
> Kconfig warnings: (for reference only)
> WARNING: unmet direct dependencies detected for REGMAP_SPI
> Depends on [n]: SPI [=n]
> Selected by [m]:
> - SND_SOC_ADAU1781_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
> - SND_SOC_ADAU1977_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
>
>
> vim +192 sound/ac97/bus.c
>
> 74426fbff66eea Robert Jarzmik 2017-09-02 181
> 74426fbff66eea Robert Jarzmik 2017-09-02 182 /**
> 74426fbff66eea Robert Jarzmik 2017-09-02 183 * snd_ac97_codec_driver_register - register an AC97 codec driver
> 74426fbff66eea Robert Jarzmik 2017-09-02 184 * @dev: AC97 driver codec to register
> 74426fbff66eea Robert Jarzmik 2017-09-02 185 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 186 * Register an AC97 codec driver to the ac97 bus driver, aka. the AC97 digital
> 74426fbff66eea Robert Jarzmik 2017-09-02 187 * controller.
> 74426fbff66eea Robert Jarzmik 2017-09-02 188 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 189 * Returns 0 on success or error code
> 74426fbff66eea Robert Jarzmik 2017-09-02 190 */
> 74426fbff66eea Robert Jarzmik 2017-09-02 191 int snd_ac97_codec_driver_register(struct ac97_codec_driver *drv)
> 74426fbff66eea Robert Jarzmik 2017-09-02 @192 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 193 drv->driver.bus = &ac97_bus_type;
> 74426fbff66eea Robert Jarzmik 2017-09-02 194 return driver_register(&drv->driver);
> 74426fbff66eea Robert Jarzmik 2017-09-02 195 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 196 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_register);
> 74426fbff66eea Robert Jarzmik 2017-09-02 197
> 74426fbff66eea Robert Jarzmik 2017-09-02 198 /**
> 74426fbff66eea Robert Jarzmik 2017-09-02 199 * snd_ac97_codec_driver_unregister - unregister an AC97 codec driver
> 74426fbff66eea Robert Jarzmik 2017-09-02 200 * @dev: AC97 codec driver to unregister
> 74426fbff66eea Robert Jarzmik 2017-09-02 201 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 202 * Unregister a previously registered ac97 codec driver.
> 74426fbff66eea Robert Jarzmik 2017-09-02 203 */
> 74426fbff66eea Robert Jarzmik 2017-09-02 204 void snd_ac97_codec_driver_unregister(struct ac97_codec_driver *drv)
> 74426fbff66eea Robert Jarzmik 2017-09-02 @205 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 206 driver_unregister(&drv->driver);
> 74426fbff66eea Robert Jarzmik 2017-09-02 207 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 208 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_unregister);
> 74426fbff66eea Robert Jarzmik 2017-09-02 209
> 74426fbff66eea Robert Jarzmik 2017-09-02 210 /**
> 74426fbff66eea Robert Jarzmik 2017-09-02 211 * snd_ac97_codec_get_platdata - get platform_data
> 74426fbff66eea Robert Jarzmik 2017-09-02 212 * @adev: the ac97 codec device
> 74426fbff66eea Robert Jarzmik 2017-09-02 213 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 214 * For legacy platforms, in order to have platform_data in codec drivers
> 74426fbff66eea Robert Jarzmik 2017-09-02 215 * available, while ac97 device are auto-created upon probe, this retrieves the
> 74426fbff66eea Robert Jarzmik 2017-09-02 216 * platdata which was setup on ac97 controller registration.
> 74426fbff66eea Robert Jarzmik 2017-09-02 217 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 218 * Returns the platform data pointer
> 74426fbff66eea Robert Jarzmik 2017-09-02 219 */
> 74426fbff66eea Robert Jarzmik 2017-09-02 220 void *snd_ac97_codec_get_platdata(const struct ac97_codec_device *adev)
> 74426fbff66eea Robert Jarzmik 2017-09-02 221 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 222 struct ac97_controller *ac97_ctrl = adev->ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 223
> 74426fbff66eea Robert Jarzmik 2017-09-02 224 return ac97_ctrl->codecs_pdata[adev->num];
> 74426fbff66eea Robert Jarzmik 2017-09-02 225 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 226 EXPORT_SYMBOL_GPL(snd_ac97_codec_get_platdata);
> 74426fbff66eea Robert Jarzmik 2017-09-02 227
> 74426fbff66eea Robert Jarzmik 2017-09-02 228 static void ac97_ctrl_codecs_unregister(struct ac97_controller *ac97_ctrl)
> 74426fbff66eea Robert Jarzmik 2017-09-02 229 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 230 int i;
> 74426fbff66eea Robert Jarzmik 2017-09-02 231
> 74426fbff66eea Robert Jarzmik 2017-09-02 232 for (i = 0; i < AC97_BUS_MAX_CODECS; i++)
> 74426fbff66eea Robert Jarzmik 2017-09-02 233 if (ac97_ctrl->codecs[i]) {
> 74426fbff66eea Robert Jarzmik 2017-09-02 234 ac97_ctrl->codecs[i]->ac97_ctrl = &ac97_unbound_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 235 device_unregister(&ac97_ctrl->codecs[i]->dev);
> 74426fbff66eea Robert Jarzmik 2017-09-02 236 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 237 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 238
> 74426fbff66eea Robert Jarzmik 2017-09-02 239 static ssize_t cold_reset_store(struct device *dev,
> 74426fbff66eea Robert Jarzmik 2017-09-02 240 struct device_attribute *attr, const char *buf,
> 74426fbff66eea Robert Jarzmik 2017-09-02 241 size_t len)
> 74426fbff66eea Robert Jarzmik 2017-09-02 242 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 243 struct ac97_controller *ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 244
> 74426fbff66eea Robert Jarzmik 2017-09-02 245 mutex_lock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 246 ac97_ctrl = to_ac97_controller(dev);
> 74426fbff66eea Robert Jarzmik 2017-09-02 247 ac97_ctrl->ops->reset(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 248 mutex_unlock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 249 return len;
> 74426fbff66eea Robert Jarzmik 2017-09-02 250 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 251 static DEVICE_ATTR_WO(cold_reset);
> 74426fbff66eea Robert Jarzmik 2017-09-02 252
> 74426fbff66eea Robert Jarzmik 2017-09-02 253 static ssize_t warm_reset_store(struct device *dev,
> 74426fbff66eea Robert Jarzmik 2017-09-02 254 struct device_attribute *attr, const char *buf,
> 74426fbff66eea Robert Jarzmik 2017-09-02 255 size_t len)
> 74426fbff66eea Robert Jarzmik 2017-09-02 256 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 257 struct ac97_controller *ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 258
> 74426fbff66eea Robert Jarzmik 2017-09-02 259 if (!dev)
> 74426fbff66eea Robert Jarzmik 2017-09-02 260 return -ENODEV;
> 74426fbff66eea Robert Jarzmik 2017-09-02 261
> 74426fbff66eea Robert Jarzmik 2017-09-02 262 mutex_lock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 263 ac97_ctrl = to_ac97_controller(dev);
> 74426fbff66eea Robert Jarzmik 2017-09-02 264 ac97_ctrl->ops->warm_reset(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 265 mutex_unlock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 266 return len;
> 74426fbff66eea Robert Jarzmik 2017-09-02 267 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 268 static DEVICE_ATTR_WO(warm_reset);
> 74426fbff66eea Robert Jarzmik 2017-09-02 269
> 74426fbff66eea Robert Jarzmik 2017-09-02 270 static struct attribute *ac97_controller_device_attrs[] = {
> 74426fbff66eea Robert Jarzmik 2017-09-02 271 &dev_attr_cold_reset.attr,
> 74426fbff66eea Robert Jarzmik 2017-09-02 272 &dev_attr_warm_reset.attr,
> 74426fbff66eea Robert Jarzmik 2017-09-02 273 NULL
> 74426fbff66eea Robert Jarzmik 2017-09-02 274 };
> 74426fbff66eea Robert Jarzmik 2017-09-02 275
> 74426fbff66eea Robert Jarzmik 2017-09-02 276 static struct attribute_group ac97_adapter_attr_group = {
> 74426fbff66eea Robert Jarzmik 2017-09-02 277 .name = "ac97_operations",
> 74426fbff66eea Robert Jarzmik 2017-09-02 278 .attrs = ac97_controller_device_attrs,
> 74426fbff66eea Robert Jarzmik 2017-09-02 279 };
> 74426fbff66eea Robert Jarzmik 2017-09-02 280
> 74426fbff66eea Robert Jarzmik 2017-09-02 281 static const struct attribute_group *ac97_adapter_groups[] = {
> 74426fbff66eea Robert Jarzmik 2017-09-02 282 &ac97_adapter_attr_group,
> 74426fbff66eea Robert Jarzmik 2017-09-02 283 NULL,
> 74426fbff66eea Robert Jarzmik 2017-09-02 284 };
> 74426fbff66eea Robert Jarzmik 2017-09-02 285
> 74426fbff66eea Robert Jarzmik 2017-09-02 286 static void ac97_del_adapter(struct ac97_controller *ac97_ctrl)
> 74426fbff66eea Robert Jarzmik 2017-09-02 287 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 288 mutex_lock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 289 ac97_ctrl_codecs_unregister(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 290 list_del(&ac97_ctrl->controllers);
> 74426fbff66eea Robert Jarzmik 2017-09-02 291 mutex_unlock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 292
> 74426fbff66eea Robert Jarzmik 2017-09-02 293 device_unregister(&ac97_ctrl->adap);
> 74426fbff66eea Robert Jarzmik 2017-09-02 294 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 295
> 74426fbff66eea Robert Jarzmik 2017-09-02 296 static void ac97_adapter_release(struct device *dev)
> 74426fbff66eea Robert Jarzmik 2017-09-02 297 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 298 struct ac97_controller *ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 299
> 74426fbff66eea Robert Jarzmik 2017-09-02 300 ac97_ctrl = to_ac97_controller(dev);
> 74426fbff66eea Robert Jarzmik 2017-09-02 301 idr_remove(&ac97_adapter_idr, ac97_ctrl->nr);
> 74426fbff66eea Robert Jarzmik 2017-09-02 302 dev_dbg(&ac97_ctrl->adap, "adapter unregistered by %s\n",
> 74426fbff66eea Robert Jarzmik 2017-09-02 303 dev_name(ac97_ctrl->parent));
> 74426fbff66eea Robert Jarzmik 2017-09-02 304 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 305
> 74426fbff66eea Robert Jarzmik 2017-09-02 306 static const struct device_type ac97_adapter_type = {
> 74426fbff66eea Robert Jarzmik 2017-09-02 307 .groups = ac97_adapter_groups,
> 74426fbff66eea Robert Jarzmik 2017-09-02 308 .release = ac97_adapter_release,
> 74426fbff66eea Robert Jarzmik 2017-09-02 309 };
> 74426fbff66eea Robert Jarzmik 2017-09-02 310
> 74426fbff66eea Robert Jarzmik 2017-09-02 311 static int ac97_add_adapter(struct ac97_controller *ac97_ctrl)
> 74426fbff66eea Robert Jarzmik 2017-09-02 312 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 313 int ret;
> 74426fbff66eea Robert Jarzmik 2017-09-02 314
> 74426fbff66eea Robert Jarzmik 2017-09-02 315 mutex_lock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 316 ret = idr_alloc(&ac97_adapter_idr, ac97_ctrl, 0, 0, GFP_KERNEL);
> 74426fbff66eea Robert Jarzmik 2017-09-02 317 ac97_ctrl->nr = ret;
> 74426fbff66eea Robert Jarzmik 2017-09-02 318 if (ret >= 0) {
> 74426fbff66eea Robert Jarzmik 2017-09-02 319 dev_set_name(&ac97_ctrl->adap, "ac97-%d", ret);
> 74426fbff66eea Robert Jarzmik 2017-09-02 320 ac97_ctrl->adap.type = &ac97_adapter_type;
> 74426fbff66eea Robert Jarzmik 2017-09-02 321 ac97_ctrl->adap.parent = ac97_ctrl->parent;
> 74426fbff66eea Robert Jarzmik 2017-09-02 322 ret = device_register(&ac97_ctrl->adap);
> 74426fbff66eea Robert Jarzmik 2017-09-02 323 if (ret)
> 74426fbff66eea Robert Jarzmik 2017-09-02 324 put_device(&ac97_ctrl->adap);
> 74426fbff66eea Robert Jarzmik 2017-09-02 325 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 326 if (!ret)
> 74426fbff66eea Robert Jarzmik 2017-09-02 327 list_add(&ac97_ctrl->controllers, &ac97_controllers);
> 74426fbff66eea Robert Jarzmik 2017-09-02 328 mutex_unlock(&ac97_controllers_mutex);
> 74426fbff66eea Robert Jarzmik 2017-09-02 329
> 74426fbff66eea Robert Jarzmik 2017-09-02 330 if (!ret)
> 74426fbff66eea Robert Jarzmik 2017-09-02 331 dev_dbg(&ac97_ctrl->adap, "adapter registered by %s\n",
> 74426fbff66eea Robert Jarzmik 2017-09-02 332 dev_name(ac97_ctrl->parent));
> 74426fbff66eea Robert Jarzmik 2017-09-02 333 return ret;
> 74426fbff66eea Robert Jarzmik 2017-09-02 334 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 335
> 74426fbff66eea Robert Jarzmik 2017-09-02 336 /**
> 74426fbff66eea Robert Jarzmik 2017-09-02 337 * snd_ac97_controller_register - register an ac97 controller
> 74426fbff66eea Robert Jarzmik 2017-09-02 338 * @ops: the ac97 bus operations
> 74426fbff66eea Robert Jarzmik 2017-09-02 339 * @dev: the device providing the ac97 DC function
> 74426fbff66eea Robert Jarzmik 2017-09-02 340 * @slots_available: mask of the ac97 codecs that can be scanned and probed
> 74426fbff66eea Robert Jarzmik 2017-09-02 341 * bit0 => codec 0, bit1 => codec 1 ... bit 3 => codec 3
> 74426fbff66eea Robert Jarzmik 2017-09-02 342 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 343 * Register a digital controller which can control up to 4 ac97 codecs. This is
> 74426fbff66eea Robert Jarzmik 2017-09-02 344 * the controller side of the AC97 AC-link, while the slave side are the codecs.
> 74426fbff66eea Robert Jarzmik 2017-09-02 345 *
> 74426fbff66eea Robert Jarzmik 2017-09-02 346 * Returns a valid controller upon success, negative pointer value upon error
> 74426fbff66eea Robert Jarzmik 2017-09-02 347 */
> 74426fbff66eea Robert Jarzmik 2017-09-02 348 struct ac97_controller *snd_ac97_controller_register(
> 74426fbff66eea Robert Jarzmik 2017-09-02 349 const struct ac97_controller_ops *ops, struct device *dev,
> 74426fbff66eea Robert Jarzmik 2017-09-02 350 unsigned short slots_available, void **codecs_pdata)
> 74426fbff66eea Robert Jarzmik 2017-09-02 @351 {
> 74426fbff66eea Robert Jarzmik 2017-09-02 352 struct ac97_controller *ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 353 int ret, i;
> 74426fbff66eea Robert Jarzmik 2017-09-02 354
> 74426fbff66eea Robert Jarzmik 2017-09-02 355 ac97_ctrl = kzalloc(sizeof(*ac97_ctrl), GFP_KERNEL);
> 74426fbff66eea Robert Jarzmik 2017-09-02 356 if (!ac97_ctrl)
> 74426fbff66eea Robert Jarzmik 2017-09-02 357 return ERR_PTR(-ENOMEM);
> 74426fbff66eea Robert Jarzmik 2017-09-02 358
> 74426fbff66eea Robert Jarzmik 2017-09-02 359 for (i = 0; i < AC97_BUS_MAX_CODECS && codecs_pdata; i++)
> 74426fbff66eea Robert Jarzmik 2017-09-02 360 ac97_ctrl->codecs_pdata[i] = codecs_pdata[i];
> 74426fbff66eea Robert Jarzmik 2017-09-02 361
> 74426fbff66eea Robert Jarzmik 2017-09-02 362 ac97_ctrl->ops = ops;
> 74426fbff66eea Robert Jarzmik 2017-09-02 363 ac97_ctrl->slots_available = slots_available;
> 74426fbff66eea Robert Jarzmik 2017-09-02 364 ac97_ctrl->parent = dev;
> 74426fbff66eea Robert Jarzmik 2017-09-02 365 ret = ac97_add_adapter(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 366
> 74426fbff66eea Robert Jarzmik 2017-09-02 367 if (ret)
> 74426fbff66eea Robert Jarzmik 2017-09-02 368 goto err;
> 74426fbff66eea Robert Jarzmik 2017-09-02 369 ac97_bus_reset(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 370 ac97_bus_scan(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 371
> 74426fbff66eea Robert Jarzmik 2017-09-02 372 return ac97_ctrl;
> 74426fbff66eea Robert Jarzmik 2017-09-02 373 err:
> 74426fbff66eea Robert Jarzmik 2017-09-02 374 kfree(ac97_ctrl);
> 74426fbff66eea Robert Jarzmik 2017-09-02 375 return ERR_PTR(ret);
> 74426fbff66eea Robert Jarzmik 2017-09-02 376 }
> 74426fbff66eea Robert Jarzmik 2017-09-02 377 EXPORT_SYMBOL_GPL(snd_ac97_controller_register);
> 74426fbff66eea Robert Jarzmik 2017-09-02 378
>
> :::::: The code at line 192 was first introduced by commit
> :::::: 74426fbff66eea8e8d1f42c8238c268d1e63a832 ALSA: ac97: add an ac97 bus
>
> :::::: TO: Robert Jarzmik <robert.jarzmik@free.fr>
> :::::: CC: Mark Brown <broonie@kernel.org>
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 5+ messages in thread
* sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
@ 2024-01-22 8:31 kernel test robot
2024-01-22 8:42 ` Geert Uytterhoeven
0 siblings, 1 reply; 5+ messages in thread
From: kernel test robot @ 2024-01-22 8:31 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: oe-kbuild-all, linux-kernel, Mark Brown
Hi Geert,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ac865f00af293d081356bec56eea90815094a60e
commit: ea00d95200d02ece71f5814d41b14f2eb16d598b ASoC: Use imply for SND_SOC_ALL_CODECS
date: 3 years, 11 months ago
:::::: branch date: 16 hours ago
:::::: commit date: 3 years, 11 months ago
config: x86_64-buildonly-randconfig-003-20231102 (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240104/202401042239.2zkHgzki-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202401042239.2zkHgzki-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:192: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:205: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:205: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:351: warning: Function parameter or member 'codecs_pdata' not described in 'snd_ac97_controller_register'
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for REGMAP_SPI
Depends on [n]: SPI [=n]
Selected by [m]:
- SND_SOC_ADAU1781_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
- SND_SOC_ADAU1977_SPI [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m]
vim +192 sound/ac97/bus.c
74426fbff66eea Robert Jarzmik 2017-09-02 181
74426fbff66eea Robert Jarzmik 2017-09-02 182 /**
74426fbff66eea Robert Jarzmik 2017-09-02 183 * snd_ac97_codec_driver_register - register an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 184 * @dev: AC97 driver codec to register
74426fbff66eea Robert Jarzmik 2017-09-02 185 *
74426fbff66eea Robert Jarzmik 2017-09-02 186 * Register an AC97 codec driver to the ac97 bus driver, aka. the AC97 digital
74426fbff66eea Robert Jarzmik 2017-09-02 187 * controller.
74426fbff66eea Robert Jarzmik 2017-09-02 188 *
74426fbff66eea Robert Jarzmik 2017-09-02 189 * Returns 0 on success or error code
74426fbff66eea Robert Jarzmik 2017-09-02 190 */
74426fbff66eea Robert Jarzmik 2017-09-02 191 int snd_ac97_codec_driver_register(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @192 {
74426fbff66eea Robert Jarzmik 2017-09-02 193 drv->driver.bus = &ac97_bus_type;
74426fbff66eea Robert Jarzmik 2017-09-02 194 return driver_register(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 195 }
74426fbff66eea Robert Jarzmik 2017-09-02 196 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_register);
74426fbff66eea Robert Jarzmik 2017-09-02 197
74426fbff66eea Robert Jarzmik 2017-09-02 198 /**
74426fbff66eea Robert Jarzmik 2017-09-02 199 * snd_ac97_codec_driver_unregister - unregister an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 200 * @dev: AC97 codec driver to unregister
74426fbff66eea Robert Jarzmik 2017-09-02 201 *
74426fbff66eea Robert Jarzmik 2017-09-02 202 * Unregister a previously registered ac97 codec driver.
74426fbff66eea Robert Jarzmik 2017-09-02 203 */
74426fbff66eea Robert Jarzmik 2017-09-02 204 void snd_ac97_codec_driver_unregister(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @205 {
74426fbff66eea Robert Jarzmik 2017-09-02 206 driver_unregister(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 207 }
74426fbff66eea Robert Jarzmik 2017-09-02 208 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_unregister);
74426fbff66eea Robert Jarzmik 2017-09-02 209
74426fbff66eea Robert Jarzmik 2017-09-02 210 /**
74426fbff66eea Robert Jarzmik 2017-09-02 211 * snd_ac97_codec_get_platdata - get platform_data
74426fbff66eea Robert Jarzmik 2017-09-02 212 * @adev: the ac97 codec device
74426fbff66eea Robert Jarzmik 2017-09-02 213 *
74426fbff66eea Robert Jarzmik 2017-09-02 214 * For legacy platforms, in order to have platform_data in codec drivers
74426fbff66eea Robert Jarzmik 2017-09-02 215 * available, while ac97 device are auto-created upon probe, this retrieves the
74426fbff66eea Robert Jarzmik 2017-09-02 216 * platdata which was setup on ac97 controller registration.
74426fbff66eea Robert Jarzmik 2017-09-02 217 *
74426fbff66eea Robert Jarzmik 2017-09-02 218 * Returns the platform data pointer
74426fbff66eea Robert Jarzmik 2017-09-02 219 */
74426fbff66eea Robert Jarzmik 2017-09-02 220 void *snd_ac97_codec_get_platdata(const struct ac97_codec_device *adev)
74426fbff66eea Robert Jarzmik 2017-09-02 221 {
74426fbff66eea Robert Jarzmik 2017-09-02 222 struct ac97_controller *ac97_ctrl = adev->ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 223
74426fbff66eea Robert Jarzmik 2017-09-02 224 return ac97_ctrl->codecs_pdata[adev->num];
74426fbff66eea Robert Jarzmik 2017-09-02 225 }
74426fbff66eea Robert Jarzmik 2017-09-02 226 EXPORT_SYMBOL_GPL(snd_ac97_codec_get_platdata);
74426fbff66eea Robert Jarzmik 2017-09-02 227
74426fbff66eea Robert Jarzmik 2017-09-02 228 static void ac97_ctrl_codecs_unregister(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 229 {
74426fbff66eea Robert Jarzmik 2017-09-02 230 int i;
74426fbff66eea Robert Jarzmik 2017-09-02 231
74426fbff66eea Robert Jarzmik 2017-09-02 232 for (i = 0; i < AC97_BUS_MAX_CODECS; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 233 if (ac97_ctrl->codecs[i]) {
74426fbff66eea Robert Jarzmik 2017-09-02 234 ac97_ctrl->codecs[i]->ac97_ctrl = &ac97_unbound_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 235 device_unregister(&ac97_ctrl->codecs[i]->dev);
74426fbff66eea Robert Jarzmik 2017-09-02 236 }
74426fbff66eea Robert Jarzmik 2017-09-02 237 }
74426fbff66eea Robert Jarzmik 2017-09-02 238
74426fbff66eea Robert Jarzmik 2017-09-02 239 static ssize_t cold_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 240 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 241 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 242 {
74426fbff66eea Robert Jarzmik 2017-09-02 243 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 244
74426fbff66eea Robert Jarzmik 2017-09-02 245 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 246 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 247 ac97_ctrl->ops->reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 248 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 249 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 250 }
74426fbff66eea Robert Jarzmik 2017-09-02 251 static DEVICE_ATTR_WO(cold_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 252
74426fbff66eea Robert Jarzmik 2017-09-02 253 static ssize_t warm_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 254 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 255 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 256 {
74426fbff66eea Robert Jarzmik 2017-09-02 257 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 258
74426fbff66eea Robert Jarzmik 2017-09-02 259 if (!dev)
74426fbff66eea Robert Jarzmik 2017-09-02 260 return -ENODEV;
74426fbff66eea Robert Jarzmik 2017-09-02 261
74426fbff66eea Robert Jarzmik 2017-09-02 262 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 263 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 264 ac97_ctrl->ops->warm_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 265 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 266 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 267 }
74426fbff66eea Robert Jarzmik 2017-09-02 268 static DEVICE_ATTR_WO(warm_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 269
74426fbff66eea Robert Jarzmik 2017-09-02 270 static struct attribute *ac97_controller_device_attrs[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 271 &dev_attr_cold_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 272 &dev_attr_warm_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 273 NULL
74426fbff66eea Robert Jarzmik 2017-09-02 274 };
74426fbff66eea Robert Jarzmik 2017-09-02 275
74426fbff66eea Robert Jarzmik 2017-09-02 276 static struct attribute_group ac97_adapter_attr_group = {
74426fbff66eea Robert Jarzmik 2017-09-02 277 .name = "ac97_operations",
74426fbff66eea Robert Jarzmik 2017-09-02 278 .attrs = ac97_controller_device_attrs,
74426fbff66eea Robert Jarzmik 2017-09-02 279 };
74426fbff66eea Robert Jarzmik 2017-09-02 280
74426fbff66eea Robert Jarzmik 2017-09-02 281 static const struct attribute_group *ac97_adapter_groups[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 282 &ac97_adapter_attr_group,
74426fbff66eea Robert Jarzmik 2017-09-02 283 NULL,
74426fbff66eea Robert Jarzmik 2017-09-02 284 };
74426fbff66eea Robert Jarzmik 2017-09-02 285
74426fbff66eea Robert Jarzmik 2017-09-02 286 static void ac97_del_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 287 {
74426fbff66eea Robert Jarzmik 2017-09-02 288 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 289 ac97_ctrl_codecs_unregister(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 290 list_del(&ac97_ctrl->controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 291 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 292
74426fbff66eea Robert Jarzmik 2017-09-02 293 device_unregister(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 294 }
74426fbff66eea Robert Jarzmik 2017-09-02 295
74426fbff66eea Robert Jarzmik 2017-09-02 296 static void ac97_adapter_release(struct device *dev)
74426fbff66eea Robert Jarzmik 2017-09-02 297 {
74426fbff66eea Robert Jarzmik 2017-09-02 298 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 299
74426fbff66eea Robert Jarzmik 2017-09-02 300 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 301 idr_remove(&ac97_adapter_idr, ac97_ctrl->nr);
74426fbff66eea Robert Jarzmik 2017-09-02 302 dev_dbg(&ac97_ctrl->adap, "adapter unregistered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 303 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 304 }
74426fbff66eea Robert Jarzmik 2017-09-02 305
74426fbff66eea Robert Jarzmik 2017-09-02 306 static const struct device_type ac97_adapter_type = {
74426fbff66eea Robert Jarzmik 2017-09-02 307 .groups = ac97_adapter_groups,
74426fbff66eea Robert Jarzmik 2017-09-02 308 .release = ac97_adapter_release,
74426fbff66eea Robert Jarzmik 2017-09-02 309 };
74426fbff66eea Robert Jarzmik 2017-09-02 310
74426fbff66eea Robert Jarzmik 2017-09-02 311 static int ac97_add_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 312 {
74426fbff66eea Robert Jarzmik 2017-09-02 313 int ret;
74426fbff66eea Robert Jarzmik 2017-09-02 314
74426fbff66eea Robert Jarzmik 2017-09-02 315 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 316 ret = idr_alloc(&ac97_adapter_idr, ac97_ctrl, 0, 0, GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 317 ac97_ctrl->nr = ret;
74426fbff66eea Robert Jarzmik 2017-09-02 318 if (ret >= 0) {
74426fbff66eea Robert Jarzmik 2017-09-02 319 dev_set_name(&ac97_ctrl->adap, "ac97-%d", ret);
74426fbff66eea Robert Jarzmik 2017-09-02 320 ac97_ctrl->adap.type = &ac97_adapter_type;
74426fbff66eea Robert Jarzmik 2017-09-02 321 ac97_ctrl->adap.parent = ac97_ctrl->parent;
74426fbff66eea Robert Jarzmik 2017-09-02 322 ret = device_register(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 323 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 324 put_device(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 325 }
74426fbff66eea Robert Jarzmik 2017-09-02 326 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 327 list_add(&ac97_ctrl->controllers, &ac97_controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 328 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 329
74426fbff66eea Robert Jarzmik 2017-09-02 330 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 331 dev_dbg(&ac97_ctrl->adap, "adapter registered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 332 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 333 return ret;
74426fbff66eea Robert Jarzmik 2017-09-02 334 }
74426fbff66eea Robert Jarzmik 2017-09-02 335
74426fbff66eea Robert Jarzmik 2017-09-02 336 /**
74426fbff66eea Robert Jarzmik 2017-09-02 337 * snd_ac97_controller_register - register an ac97 controller
74426fbff66eea Robert Jarzmik 2017-09-02 338 * @ops: the ac97 bus operations
74426fbff66eea Robert Jarzmik 2017-09-02 339 * @dev: the device providing the ac97 DC function
74426fbff66eea Robert Jarzmik 2017-09-02 340 * @slots_available: mask of the ac97 codecs that can be scanned and probed
74426fbff66eea Robert Jarzmik 2017-09-02 341 * bit0 => codec 0, bit1 => codec 1 ... bit 3 => codec 3
74426fbff66eea Robert Jarzmik 2017-09-02 342 *
74426fbff66eea Robert Jarzmik 2017-09-02 343 * Register a digital controller which can control up to 4 ac97 codecs. This is
74426fbff66eea Robert Jarzmik 2017-09-02 344 * the controller side of the AC97 AC-link, while the slave side are the codecs.
74426fbff66eea Robert Jarzmik 2017-09-02 345 *
74426fbff66eea Robert Jarzmik 2017-09-02 346 * Returns a valid controller upon success, negative pointer value upon error
74426fbff66eea Robert Jarzmik 2017-09-02 347 */
74426fbff66eea Robert Jarzmik 2017-09-02 348 struct ac97_controller *snd_ac97_controller_register(
74426fbff66eea Robert Jarzmik 2017-09-02 349 const struct ac97_controller_ops *ops, struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 350 unsigned short slots_available, void **codecs_pdata)
74426fbff66eea Robert Jarzmik 2017-09-02 @351 {
74426fbff66eea Robert Jarzmik 2017-09-02 352 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 353 int ret, i;
74426fbff66eea Robert Jarzmik 2017-09-02 354
74426fbff66eea Robert Jarzmik 2017-09-02 355 ac97_ctrl = kzalloc(sizeof(*ac97_ctrl), GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 356 if (!ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 357 return ERR_PTR(-ENOMEM);
74426fbff66eea Robert Jarzmik 2017-09-02 358
74426fbff66eea Robert Jarzmik 2017-09-02 359 for (i = 0; i < AC97_BUS_MAX_CODECS && codecs_pdata; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 360 ac97_ctrl->codecs_pdata[i] = codecs_pdata[i];
74426fbff66eea Robert Jarzmik 2017-09-02 361
74426fbff66eea Robert Jarzmik 2017-09-02 362 ac97_ctrl->ops = ops;
74426fbff66eea Robert Jarzmik 2017-09-02 363 ac97_ctrl->slots_available = slots_available;
74426fbff66eea Robert Jarzmik 2017-09-02 364 ac97_ctrl->parent = dev;
74426fbff66eea Robert Jarzmik 2017-09-02 365 ret = ac97_add_adapter(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 366
74426fbff66eea Robert Jarzmik 2017-09-02 367 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 368 goto err;
74426fbff66eea Robert Jarzmik 2017-09-02 369 ac97_bus_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 370 ac97_bus_scan(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 371
74426fbff66eea Robert Jarzmik 2017-09-02 372 return ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 373 err:
74426fbff66eea Robert Jarzmik 2017-09-02 374 kfree(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 375 return ERR_PTR(ret);
74426fbff66eea Robert Jarzmik 2017-09-02 376 }
74426fbff66eea Robert Jarzmik 2017-09-02 377 EXPORT_SYMBOL_GPL(snd_ac97_controller_register);
74426fbff66eea Robert Jarzmik 2017-09-02 378
:::::: The code at line 192 was first introduced by commit
:::::: 74426fbff66eea8e8d1f42c8238c268d1e63a832 ALSA: ac97: add an ac97 bus
:::::: TO: Robert Jarzmik <robert.jarzmik@free.fr>
:::::: CC: Mark Brown <broonie@kernel.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
@ 2023-11-05 3:16 kernel test robot
0 siblings, 0 replies; 5+ messages in thread
From: kernel test robot @ 2023-11-05 3:16 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: oe-kbuild-all, linux-kernel, Mark Brown
Hi Arnd,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: aea6bf908d730b01bd264a8821159db9463c111c
commit: 5eab9265759e2fb042aa452931c3d06ab7ab8dae ASoC: PXA: make SND_PXA2XX_SOC_AC97 user-selectable
date: 10 months ago
config: x86_64-buildonly-randconfig-003-20231102 (https://download.01.org/0day-ci/archive/20231105/202311051152.W491nQsq-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231105/202311051152.W491nQsq-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311051152.W491nQsq-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:192: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:205: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:205: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:351: warning: Function parameter or member 'codecs_pdata' not described in 'snd_ac97_controller_register'
vim +192 sound/ac97/bus.c
74426fbff66eea Robert Jarzmik 2017-09-02 181
74426fbff66eea Robert Jarzmik 2017-09-02 182 /**
74426fbff66eea Robert Jarzmik 2017-09-02 183 * snd_ac97_codec_driver_register - register an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 184 * @dev: AC97 driver codec to register
74426fbff66eea Robert Jarzmik 2017-09-02 185 *
74426fbff66eea Robert Jarzmik 2017-09-02 186 * Register an AC97 codec driver to the ac97 bus driver, aka. the AC97 digital
74426fbff66eea Robert Jarzmik 2017-09-02 187 * controller.
74426fbff66eea Robert Jarzmik 2017-09-02 188 *
74426fbff66eea Robert Jarzmik 2017-09-02 189 * Returns 0 on success or error code
74426fbff66eea Robert Jarzmik 2017-09-02 190 */
74426fbff66eea Robert Jarzmik 2017-09-02 191 int snd_ac97_codec_driver_register(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @192 {
74426fbff66eea Robert Jarzmik 2017-09-02 193 drv->driver.bus = &ac97_bus_type;
74426fbff66eea Robert Jarzmik 2017-09-02 194 return driver_register(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 195 }
74426fbff66eea Robert Jarzmik 2017-09-02 196 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_register);
74426fbff66eea Robert Jarzmik 2017-09-02 197
74426fbff66eea Robert Jarzmik 2017-09-02 198 /**
74426fbff66eea Robert Jarzmik 2017-09-02 199 * snd_ac97_codec_driver_unregister - unregister an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 200 * @dev: AC97 codec driver to unregister
74426fbff66eea Robert Jarzmik 2017-09-02 201 *
74426fbff66eea Robert Jarzmik 2017-09-02 202 * Unregister a previously registered ac97 codec driver.
74426fbff66eea Robert Jarzmik 2017-09-02 203 */
74426fbff66eea Robert Jarzmik 2017-09-02 204 void snd_ac97_codec_driver_unregister(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @205 {
74426fbff66eea Robert Jarzmik 2017-09-02 206 driver_unregister(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 207 }
74426fbff66eea Robert Jarzmik 2017-09-02 208 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_unregister);
74426fbff66eea Robert Jarzmik 2017-09-02 209
74426fbff66eea Robert Jarzmik 2017-09-02 210 /**
74426fbff66eea Robert Jarzmik 2017-09-02 211 * snd_ac97_codec_get_platdata - get platform_data
74426fbff66eea Robert Jarzmik 2017-09-02 212 * @adev: the ac97 codec device
74426fbff66eea Robert Jarzmik 2017-09-02 213 *
74426fbff66eea Robert Jarzmik 2017-09-02 214 * For legacy platforms, in order to have platform_data in codec drivers
74426fbff66eea Robert Jarzmik 2017-09-02 215 * available, while ac97 device are auto-created upon probe, this retrieves the
74426fbff66eea Robert Jarzmik 2017-09-02 216 * platdata which was setup on ac97 controller registration.
74426fbff66eea Robert Jarzmik 2017-09-02 217 *
74426fbff66eea Robert Jarzmik 2017-09-02 218 * Returns the platform data pointer
74426fbff66eea Robert Jarzmik 2017-09-02 219 */
74426fbff66eea Robert Jarzmik 2017-09-02 220 void *snd_ac97_codec_get_platdata(const struct ac97_codec_device *adev)
74426fbff66eea Robert Jarzmik 2017-09-02 221 {
74426fbff66eea Robert Jarzmik 2017-09-02 222 struct ac97_controller *ac97_ctrl = adev->ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 223
74426fbff66eea Robert Jarzmik 2017-09-02 224 return ac97_ctrl->codecs_pdata[adev->num];
74426fbff66eea Robert Jarzmik 2017-09-02 225 }
74426fbff66eea Robert Jarzmik 2017-09-02 226 EXPORT_SYMBOL_GPL(snd_ac97_codec_get_platdata);
74426fbff66eea Robert Jarzmik 2017-09-02 227
74426fbff66eea Robert Jarzmik 2017-09-02 228 static void ac97_ctrl_codecs_unregister(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 229 {
74426fbff66eea Robert Jarzmik 2017-09-02 230 int i;
74426fbff66eea Robert Jarzmik 2017-09-02 231
74426fbff66eea Robert Jarzmik 2017-09-02 232 for (i = 0; i < AC97_BUS_MAX_CODECS; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 233 if (ac97_ctrl->codecs[i]) {
74426fbff66eea Robert Jarzmik 2017-09-02 234 ac97_ctrl->codecs[i]->ac97_ctrl = &ac97_unbound_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 235 device_unregister(&ac97_ctrl->codecs[i]->dev);
74426fbff66eea Robert Jarzmik 2017-09-02 236 }
74426fbff66eea Robert Jarzmik 2017-09-02 237 }
74426fbff66eea Robert Jarzmik 2017-09-02 238
74426fbff66eea Robert Jarzmik 2017-09-02 239 static ssize_t cold_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 240 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 241 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 242 {
74426fbff66eea Robert Jarzmik 2017-09-02 243 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 244
74426fbff66eea Robert Jarzmik 2017-09-02 245 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 246 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 247 ac97_ctrl->ops->reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 248 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 249 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 250 }
74426fbff66eea Robert Jarzmik 2017-09-02 251 static DEVICE_ATTR_WO(cold_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 252
74426fbff66eea Robert Jarzmik 2017-09-02 253 static ssize_t warm_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 254 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 255 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 256 {
74426fbff66eea Robert Jarzmik 2017-09-02 257 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 258
74426fbff66eea Robert Jarzmik 2017-09-02 259 if (!dev)
74426fbff66eea Robert Jarzmik 2017-09-02 260 return -ENODEV;
74426fbff66eea Robert Jarzmik 2017-09-02 261
74426fbff66eea Robert Jarzmik 2017-09-02 262 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 263 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 264 ac97_ctrl->ops->warm_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 265 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 266 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 267 }
74426fbff66eea Robert Jarzmik 2017-09-02 268 static DEVICE_ATTR_WO(warm_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 269
74426fbff66eea Robert Jarzmik 2017-09-02 270 static struct attribute *ac97_controller_device_attrs[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 271 &dev_attr_cold_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 272 &dev_attr_warm_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 273 NULL
74426fbff66eea Robert Jarzmik 2017-09-02 274 };
74426fbff66eea Robert Jarzmik 2017-09-02 275
fa2e5a647ed2ed Rikard Falkeborn 2021-01-31 276 static const struct attribute_group ac97_adapter_attr_group = {
74426fbff66eea Robert Jarzmik 2017-09-02 277 .name = "ac97_operations",
74426fbff66eea Robert Jarzmik 2017-09-02 278 .attrs = ac97_controller_device_attrs,
74426fbff66eea Robert Jarzmik 2017-09-02 279 };
74426fbff66eea Robert Jarzmik 2017-09-02 280
74426fbff66eea Robert Jarzmik 2017-09-02 281 static const struct attribute_group *ac97_adapter_groups[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 282 &ac97_adapter_attr_group,
74426fbff66eea Robert Jarzmik 2017-09-02 283 NULL,
74426fbff66eea Robert Jarzmik 2017-09-02 284 };
74426fbff66eea Robert Jarzmik 2017-09-02 285
74426fbff66eea Robert Jarzmik 2017-09-02 286 static void ac97_del_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 287 {
74426fbff66eea Robert Jarzmik 2017-09-02 288 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 289 ac97_ctrl_codecs_unregister(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 290 list_del(&ac97_ctrl->controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 291 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 292
74426fbff66eea Robert Jarzmik 2017-09-02 293 device_unregister(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 294 }
74426fbff66eea Robert Jarzmik 2017-09-02 295
74426fbff66eea Robert Jarzmik 2017-09-02 296 static void ac97_adapter_release(struct device *dev)
74426fbff66eea Robert Jarzmik 2017-09-02 297 {
74426fbff66eea Robert Jarzmik 2017-09-02 298 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 299
74426fbff66eea Robert Jarzmik 2017-09-02 300 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 301 idr_remove(&ac97_adapter_idr, ac97_ctrl->nr);
74426fbff66eea Robert Jarzmik 2017-09-02 302 dev_dbg(&ac97_ctrl->adap, "adapter unregistered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 303 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 304 }
74426fbff66eea Robert Jarzmik 2017-09-02 305
74426fbff66eea Robert Jarzmik 2017-09-02 306 static const struct device_type ac97_adapter_type = {
74426fbff66eea Robert Jarzmik 2017-09-02 307 .groups = ac97_adapter_groups,
74426fbff66eea Robert Jarzmik 2017-09-02 308 .release = ac97_adapter_release,
74426fbff66eea Robert Jarzmik 2017-09-02 309 };
74426fbff66eea Robert Jarzmik 2017-09-02 310
74426fbff66eea Robert Jarzmik 2017-09-02 311 static int ac97_add_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 312 {
74426fbff66eea Robert Jarzmik 2017-09-02 313 int ret;
74426fbff66eea Robert Jarzmik 2017-09-02 314
74426fbff66eea Robert Jarzmik 2017-09-02 315 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 316 ret = idr_alloc(&ac97_adapter_idr, ac97_ctrl, 0, 0, GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 317 ac97_ctrl->nr = ret;
74426fbff66eea Robert Jarzmik 2017-09-02 318 if (ret >= 0) {
74426fbff66eea Robert Jarzmik 2017-09-02 319 dev_set_name(&ac97_ctrl->adap, "ac97-%d", ret);
74426fbff66eea Robert Jarzmik 2017-09-02 320 ac97_ctrl->adap.type = &ac97_adapter_type;
74426fbff66eea Robert Jarzmik 2017-09-02 321 ac97_ctrl->adap.parent = ac97_ctrl->parent;
74426fbff66eea Robert Jarzmik 2017-09-02 322 ret = device_register(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 323 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 324 put_device(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 325 }
74426fbff66eea Robert Jarzmik 2017-09-02 326 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 327 list_add(&ac97_ctrl->controllers, &ac97_controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 328 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 329
74426fbff66eea Robert Jarzmik 2017-09-02 330 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 331 dev_dbg(&ac97_ctrl->adap, "adapter registered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 332 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 333 return ret;
74426fbff66eea Robert Jarzmik 2017-09-02 334 }
74426fbff66eea Robert Jarzmik 2017-09-02 335
74426fbff66eea Robert Jarzmik 2017-09-02 336 /**
74426fbff66eea Robert Jarzmik 2017-09-02 337 * snd_ac97_controller_register - register an ac97 controller
74426fbff66eea Robert Jarzmik 2017-09-02 338 * @ops: the ac97 bus operations
74426fbff66eea Robert Jarzmik 2017-09-02 339 * @dev: the device providing the ac97 DC function
74426fbff66eea Robert Jarzmik 2017-09-02 340 * @slots_available: mask of the ac97 codecs that can be scanned and probed
74426fbff66eea Robert Jarzmik 2017-09-02 341 * bit0 => codec 0, bit1 => codec 1 ... bit 3 => codec 3
74426fbff66eea Robert Jarzmik 2017-09-02 342 *
74426fbff66eea Robert Jarzmik 2017-09-02 343 * Register a digital controller which can control up to 4 ac97 codecs. This is
74426fbff66eea Robert Jarzmik 2017-09-02 344 * the controller side of the AC97 AC-link, while the slave side are the codecs.
74426fbff66eea Robert Jarzmik 2017-09-02 345 *
74426fbff66eea Robert Jarzmik 2017-09-02 346 * Returns a valid controller upon success, negative pointer value upon error
74426fbff66eea Robert Jarzmik 2017-09-02 347 */
74426fbff66eea Robert Jarzmik 2017-09-02 348 struct ac97_controller *snd_ac97_controller_register(
74426fbff66eea Robert Jarzmik 2017-09-02 349 const struct ac97_controller_ops *ops, struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 350 unsigned short slots_available, void **codecs_pdata)
74426fbff66eea Robert Jarzmik 2017-09-02 @351 {
74426fbff66eea Robert Jarzmik 2017-09-02 352 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 353 int ret, i;
74426fbff66eea Robert Jarzmik 2017-09-02 354
74426fbff66eea Robert Jarzmik 2017-09-02 355 ac97_ctrl = kzalloc(sizeof(*ac97_ctrl), GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 356 if (!ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 357 return ERR_PTR(-ENOMEM);
74426fbff66eea Robert Jarzmik 2017-09-02 358
74426fbff66eea Robert Jarzmik 2017-09-02 359 for (i = 0; i < AC97_BUS_MAX_CODECS && codecs_pdata; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 360 ac97_ctrl->codecs_pdata[i] = codecs_pdata[i];
74426fbff66eea Robert Jarzmik 2017-09-02 361
74426fbff66eea Robert Jarzmik 2017-09-02 362 ac97_ctrl->ops = ops;
74426fbff66eea Robert Jarzmik 2017-09-02 363 ac97_ctrl->slots_available = slots_available;
74426fbff66eea Robert Jarzmik 2017-09-02 364 ac97_ctrl->parent = dev;
74426fbff66eea Robert Jarzmik 2017-09-02 365 ret = ac97_add_adapter(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 366
74426fbff66eea Robert Jarzmik 2017-09-02 367 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 368 goto err;
74426fbff66eea Robert Jarzmik 2017-09-02 369 ac97_bus_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 370 ac97_bus_scan(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 371
74426fbff66eea Robert Jarzmik 2017-09-02 372 return ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 373 err:
74426fbff66eea Robert Jarzmik 2017-09-02 374 kfree(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 375 return ERR_PTR(ret);
74426fbff66eea Robert Jarzmik 2017-09-02 376 }
74426fbff66eea Robert Jarzmik 2017-09-02 377 EXPORT_SYMBOL_GPL(snd_ac97_controller_register);
74426fbff66eea Robert Jarzmik 2017-09-02 378
:::::: The code at line 192 was first introduced by commit
:::::: 74426fbff66eea8e8d1f42c8238c268d1e63a832 ALSA: ac97: add an ac97 bus
:::::: TO: Robert Jarzmik <robert.jarzmik@free.fr>
:::::: CC: Mark Brown <broonie@kernel.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
* sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
@ 2023-08-17 8:50 kernel test robot
0 siblings, 0 replies; 5+ messages in thread
From: kernel test robot @ 2023-08-17 8:50 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: llvm, oe-kbuild-all, linux-kernel, Mark Brown
Hi Arnd,
FYI, the error/warning was bisected to this commit, please ignore it if it's irrelevant.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 4853c74bd7ab7fdb83f319bd9ace8a08c031e9b6
commit: 5eab9265759e2fb042aa452931c3d06ab7ab8dae ASoC: PXA: make SND_PXA2XX_SOC_AC97 user-selectable
date: 7 months ago
config: arm-randconfig-r012-20230817 (https://download.01.org/0day-ci/archive/20230817/202308171657.J7S4SHI3-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: (https://download.01.org/0day-ci/archive/20230817/202308171657.J7S4SHI3-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202308171657.J7S4SHI3-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:192: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_register'
>> sound/ac97/bus.c:205: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:205: warning: Excess function parameter 'dev' description in 'snd_ac97_codec_driver_unregister'
>> sound/ac97/bus.c:351: warning: Function parameter or member 'codecs_pdata' not described in 'snd_ac97_controller_register'
vim +192 sound/ac97/bus.c
74426fbff66eea Robert Jarzmik 2017-09-02 181
74426fbff66eea Robert Jarzmik 2017-09-02 182 /**
74426fbff66eea Robert Jarzmik 2017-09-02 183 * snd_ac97_codec_driver_register - register an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 184 * @dev: AC97 driver codec to register
74426fbff66eea Robert Jarzmik 2017-09-02 185 *
74426fbff66eea Robert Jarzmik 2017-09-02 186 * Register an AC97 codec driver to the ac97 bus driver, aka. the AC97 digital
74426fbff66eea Robert Jarzmik 2017-09-02 187 * controller.
74426fbff66eea Robert Jarzmik 2017-09-02 188 *
74426fbff66eea Robert Jarzmik 2017-09-02 189 * Returns 0 on success or error code
74426fbff66eea Robert Jarzmik 2017-09-02 190 */
74426fbff66eea Robert Jarzmik 2017-09-02 191 int snd_ac97_codec_driver_register(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @192 {
74426fbff66eea Robert Jarzmik 2017-09-02 193 drv->driver.bus = &ac97_bus_type;
74426fbff66eea Robert Jarzmik 2017-09-02 194 return driver_register(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 195 }
74426fbff66eea Robert Jarzmik 2017-09-02 196 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_register);
74426fbff66eea Robert Jarzmik 2017-09-02 197
74426fbff66eea Robert Jarzmik 2017-09-02 198 /**
74426fbff66eea Robert Jarzmik 2017-09-02 199 * snd_ac97_codec_driver_unregister - unregister an AC97 codec driver
74426fbff66eea Robert Jarzmik 2017-09-02 200 * @dev: AC97 codec driver to unregister
74426fbff66eea Robert Jarzmik 2017-09-02 201 *
74426fbff66eea Robert Jarzmik 2017-09-02 202 * Unregister a previously registered ac97 codec driver.
74426fbff66eea Robert Jarzmik 2017-09-02 203 */
74426fbff66eea Robert Jarzmik 2017-09-02 204 void snd_ac97_codec_driver_unregister(struct ac97_codec_driver *drv)
74426fbff66eea Robert Jarzmik 2017-09-02 @205 {
74426fbff66eea Robert Jarzmik 2017-09-02 206 driver_unregister(&drv->driver);
74426fbff66eea Robert Jarzmik 2017-09-02 207 }
74426fbff66eea Robert Jarzmik 2017-09-02 208 EXPORT_SYMBOL_GPL(snd_ac97_codec_driver_unregister);
74426fbff66eea Robert Jarzmik 2017-09-02 209
74426fbff66eea Robert Jarzmik 2017-09-02 210 /**
74426fbff66eea Robert Jarzmik 2017-09-02 211 * snd_ac97_codec_get_platdata - get platform_data
74426fbff66eea Robert Jarzmik 2017-09-02 212 * @adev: the ac97 codec device
74426fbff66eea Robert Jarzmik 2017-09-02 213 *
74426fbff66eea Robert Jarzmik 2017-09-02 214 * For legacy platforms, in order to have platform_data in codec drivers
74426fbff66eea Robert Jarzmik 2017-09-02 215 * available, while ac97 device are auto-created upon probe, this retrieves the
74426fbff66eea Robert Jarzmik 2017-09-02 216 * platdata which was setup on ac97 controller registration.
74426fbff66eea Robert Jarzmik 2017-09-02 217 *
74426fbff66eea Robert Jarzmik 2017-09-02 218 * Returns the platform data pointer
74426fbff66eea Robert Jarzmik 2017-09-02 219 */
74426fbff66eea Robert Jarzmik 2017-09-02 220 void *snd_ac97_codec_get_platdata(const struct ac97_codec_device *adev)
74426fbff66eea Robert Jarzmik 2017-09-02 221 {
74426fbff66eea Robert Jarzmik 2017-09-02 222 struct ac97_controller *ac97_ctrl = adev->ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 223
74426fbff66eea Robert Jarzmik 2017-09-02 224 return ac97_ctrl->codecs_pdata[adev->num];
74426fbff66eea Robert Jarzmik 2017-09-02 225 }
74426fbff66eea Robert Jarzmik 2017-09-02 226 EXPORT_SYMBOL_GPL(snd_ac97_codec_get_platdata);
74426fbff66eea Robert Jarzmik 2017-09-02 227
74426fbff66eea Robert Jarzmik 2017-09-02 228 static void ac97_ctrl_codecs_unregister(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 229 {
74426fbff66eea Robert Jarzmik 2017-09-02 230 int i;
74426fbff66eea Robert Jarzmik 2017-09-02 231
74426fbff66eea Robert Jarzmik 2017-09-02 232 for (i = 0; i < AC97_BUS_MAX_CODECS; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 233 if (ac97_ctrl->codecs[i]) {
74426fbff66eea Robert Jarzmik 2017-09-02 234 ac97_ctrl->codecs[i]->ac97_ctrl = &ac97_unbound_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 235 device_unregister(&ac97_ctrl->codecs[i]->dev);
74426fbff66eea Robert Jarzmik 2017-09-02 236 }
74426fbff66eea Robert Jarzmik 2017-09-02 237 }
74426fbff66eea Robert Jarzmik 2017-09-02 238
74426fbff66eea Robert Jarzmik 2017-09-02 239 static ssize_t cold_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 240 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 241 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 242 {
74426fbff66eea Robert Jarzmik 2017-09-02 243 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 244
74426fbff66eea Robert Jarzmik 2017-09-02 245 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 246 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 247 ac97_ctrl->ops->reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 248 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 249 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 250 }
74426fbff66eea Robert Jarzmik 2017-09-02 251 static DEVICE_ATTR_WO(cold_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 252
74426fbff66eea Robert Jarzmik 2017-09-02 253 static ssize_t warm_reset_store(struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 254 struct device_attribute *attr, const char *buf,
74426fbff66eea Robert Jarzmik 2017-09-02 255 size_t len)
74426fbff66eea Robert Jarzmik 2017-09-02 256 {
74426fbff66eea Robert Jarzmik 2017-09-02 257 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 258
74426fbff66eea Robert Jarzmik 2017-09-02 259 if (!dev)
74426fbff66eea Robert Jarzmik 2017-09-02 260 return -ENODEV;
74426fbff66eea Robert Jarzmik 2017-09-02 261
74426fbff66eea Robert Jarzmik 2017-09-02 262 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 263 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 264 ac97_ctrl->ops->warm_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 265 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 266 return len;
74426fbff66eea Robert Jarzmik 2017-09-02 267 }
74426fbff66eea Robert Jarzmik 2017-09-02 268 static DEVICE_ATTR_WO(warm_reset);
74426fbff66eea Robert Jarzmik 2017-09-02 269
74426fbff66eea Robert Jarzmik 2017-09-02 270 static struct attribute *ac97_controller_device_attrs[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 271 &dev_attr_cold_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 272 &dev_attr_warm_reset.attr,
74426fbff66eea Robert Jarzmik 2017-09-02 273 NULL
74426fbff66eea Robert Jarzmik 2017-09-02 274 };
74426fbff66eea Robert Jarzmik 2017-09-02 275
fa2e5a647ed2ed Rikard Falkeborn 2021-01-31 276 static const struct attribute_group ac97_adapter_attr_group = {
74426fbff66eea Robert Jarzmik 2017-09-02 277 .name = "ac97_operations",
74426fbff66eea Robert Jarzmik 2017-09-02 278 .attrs = ac97_controller_device_attrs,
74426fbff66eea Robert Jarzmik 2017-09-02 279 };
74426fbff66eea Robert Jarzmik 2017-09-02 280
74426fbff66eea Robert Jarzmik 2017-09-02 281 static const struct attribute_group *ac97_adapter_groups[] = {
74426fbff66eea Robert Jarzmik 2017-09-02 282 &ac97_adapter_attr_group,
74426fbff66eea Robert Jarzmik 2017-09-02 283 NULL,
74426fbff66eea Robert Jarzmik 2017-09-02 284 };
74426fbff66eea Robert Jarzmik 2017-09-02 285
74426fbff66eea Robert Jarzmik 2017-09-02 286 static void ac97_del_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 287 {
74426fbff66eea Robert Jarzmik 2017-09-02 288 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 289 ac97_ctrl_codecs_unregister(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 290 list_del(&ac97_ctrl->controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 291 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 292
74426fbff66eea Robert Jarzmik 2017-09-02 293 device_unregister(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 294 }
74426fbff66eea Robert Jarzmik 2017-09-02 295
74426fbff66eea Robert Jarzmik 2017-09-02 296 static void ac97_adapter_release(struct device *dev)
74426fbff66eea Robert Jarzmik 2017-09-02 297 {
74426fbff66eea Robert Jarzmik 2017-09-02 298 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 299
74426fbff66eea Robert Jarzmik 2017-09-02 300 ac97_ctrl = to_ac97_controller(dev);
74426fbff66eea Robert Jarzmik 2017-09-02 301 idr_remove(&ac97_adapter_idr, ac97_ctrl->nr);
74426fbff66eea Robert Jarzmik 2017-09-02 302 dev_dbg(&ac97_ctrl->adap, "adapter unregistered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 303 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 304 }
74426fbff66eea Robert Jarzmik 2017-09-02 305
74426fbff66eea Robert Jarzmik 2017-09-02 306 static const struct device_type ac97_adapter_type = {
74426fbff66eea Robert Jarzmik 2017-09-02 307 .groups = ac97_adapter_groups,
74426fbff66eea Robert Jarzmik 2017-09-02 308 .release = ac97_adapter_release,
74426fbff66eea Robert Jarzmik 2017-09-02 309 };
74426fbff66eea Robert Jarzmik 2017-09-02 310
74426fbff66eea Robert Jarzmik 2017-09-02 311 static int ac97_add_adapter(struct ac97_controller *ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 312 {
74426fbff66eea Robert Jarzmik 2017-09-02 313 int ret;
74426fbff66eea Robert Jarzmik 2017-09-02 314
74426fbff66eea Robert Jarzmik 2017-09-02 315 mutex_lock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 316 ret = idr_alloc(&ac97_adapter_idr, ac97_ctrl, 0, 0, GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 317 ac97_ctrl->nr = ret;
74426fbff66eea Robert Jarzmik 2017-09-02 318 if (ret >= 0) {
74426fbff66eea Robert Jarzmik 2017-09-02 319 dev_set_name(&ac97_ctrl->adap, "ac97-%d", ret);
74426fbff66eea Robert Jarzmik 2017-09-02 320 ac97_ctrl->adap.type = &ac97_adapter_type;
74426fbff66eea Robert Jarzmik 2017-09-02 321 ac97_ctrl->adap.parent = ac97_ctrl->parent;
74426fbff66eea Robert Jarzmik 2017-09-02 322 ret = device_register(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 323 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 324 put_device(&ac97_ctrl->adap);
74426fbff66eea Robert Jarzmik 2017-09-02 325 }
74426fbff66eea Robert Jarzmik 2017-09-02 326 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 327 list_add(&ac97_ctrl->controllers, &ac97_controllers);
74426fbff66eea Robert Jarzmik 2017-09-02 328 mutex_unlock(&ac97_controllers_mutex);
74426fbff66eea Robert Jarzmik 2017-09-02 329
74426fbff66eea Robert Jarzmik 2017-09-02 330 if (!ret)
74426fbff66eea Robert Jarzmik 2017-09-02 331 dev_dbg(&ac97_ctrl->adap, "adapter registered by %s\n",
74426fbff66eea Robert Jarzmik 2017-09-02 332 dev_name(ac97_ctrl->parent));
74426fbff66eea Robert Jarzmik 2017-09-02 333 return ret;
74426fbff66eea Robert Jarzmik 2017-09-02 334 }
74426fbff66eea Robert Jarzmik 2017-09-02 335
74426fbff66eea Robert Jarzmik 2017-09-02 336 /**
74426fbff66eea Robert Jarzmik 2017-09-02 337 * snd_ac97_controller_register - register an ac97 controller
74426fbff66eea Robert Jarzmik 2017-09-02 338 * @ops: the ac97 bus operations
74426fbff66eea Robert Jarzmik 2017-09-02 339 * @dev: the device providing the ac97 DC function
74426fbff66eea Robert Jarzmik 2017-09-02 340 * @slots_available: mask of the ac97 codecs that can be scanned and probed
74426fbff66eea Robert Jarzmik 2017-09-02 341 * bit0 => codec 0, bit1 => codec 1 ... bit 3 => codec 3
74426fbff66eea Robert Jarzmik 2017-09-02 342 *
74426fbff66eea Robert Jarzmik 2017-09-02 343 * Register a digital controller which can control up to 4 ac97 codecs. This is
74426fbff66eea Robert Jarzmik 2017-09-02 344 * the controller side of the AC97 AC-link, while the slave side are the codecs.
74426fbff66eea Robert Jarzmik 2017-09-02 345 *
74426fbff66eea Robert Jarzmik 2017-09-02 346 * Returns a valid controller upon success, negative pointer value upon error
74426fbff66eea Robert Jarzmik 2017-09-02 347 */
74426fbff66eea Robert Jarzmik 2017-09-02 348 struct ac97_controller *snd_ac97_controller_register(
74426fbff66eea Robert Jarzmik 2017-09-02 349 const struct ac97_controller_ops *ops, struct device *dev,
74426fbff66eea Robert Jarzmik 2017-09-02 350 unsigned short slots_available, void **codecs_pdata)
74426fbff66eea Robert Jarzmik 2017-09-02 @351 {
74426fbff66eea Robert Jarzmik 2017-09-02 352 struct ac97_controller *ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 353 int ret, i;
74426fbff66eea Robert Jarzmik 2017-09-02 354
74426fbff66eea Robert Jarzmik 2017-09-02 355 ac97_ctrl = kzalloc(sizeof(*ac97_ctrl), GFP_KERNEL);
74426fbff66eea Robert Jarzmik 2017-09-02 356 if (!ac97_ctrl)
74426fbff66eea Robert Jarzmik 2017-09-02 357 return ERR_PTR(-ENOMEM);
74426fbff66eea Robert Jarzmik 2017-09-02 358
74426fbff66eea Robert Jarzmik 2017-09-02 359 for (i = 0; i < AC97_BUS_MAX_CODECS && codecs_pdata; i++)
74426fbff66eea Robert Jarzmik 2017-09-02 360 ac97_ctrl->codecs_pdata[i] = codecs_pdata[i];
74426fbff66eea Robert Jarzmik 2017-09-02 361
74426fbff66eea Robert Jarzmik 2017-09-02 362 ac97_ctrl->ops = ops;
74426fbff66eea Robert Jarzmik 2017-09-02 363 ac97_ctrl->slots_available = slots_available;
74426fbff66eea Robert Jarzmik 2017-09-02 364 ac97_ctrl->parent = dev;
74426fbff66eea Robert Jarzmik 2017-09-02 365 ret = ac97_add_adapter(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 366
74426fbff66eea Robert Jarzmik 2017-09-02 367 if (ret)
74426fbff66eea Robert Jarzmik 2017-09-02 368 goto err;
74426fbff66eea Robert Jarzmik 2017-09-02 369 ac97_bus_reset(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 370 ac97_bus_scan(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 371
74426fbff66eea Robert Jarzmik 2017-09-02 372 return ac97_ctrl;
74426fbff66eea Robert Jarzmik 2017-09-02 373 err:
74426fbff66eea Robert Jarzmik 2017-09-02 374 kfree(ac97_ctrl);
74426fbff66eea Robert Jarzmik 2017-09-02 375 return ERR_PTR(ret);
74426fbff66eea Robert Jarzmik 2017-09-02 376 }
74426fbff66eea Robert Jarzmik 2017-09-02 377 EXPORT_SYMBOL_GPL(snd_ac97_controller_register);
74426fbff66eea Robert Jarzmik 2017-09-02 378
:::::: The code at line 192 was first introduced by commit
:::::: 74426fbff66eea8e8d1f42c8238c268d1e63a832 ALSA: ac97: add an ac97 bus
:::::: TO: Robert Jarzmik <robert.jarzmik@free.fr>
:::::: CC: Mark Brown <broonie@kernel.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-01-22 8:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-04 14:22 sound/ac97/bus.c:192: warning: Function parameter or member 'drv' not described in 'snd_ac97_codec_driver_register' kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2024-01-22 8:31 kernel test robot
2024-01-22 8:42 ` Geert Uytterhoeven
2023-11-05 3:16 kernel test robot
2023-08-17 8:50 kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.