Re: [PATCH 4/4] [meida] media-device: dynamically allocate struct media_devnode

kbuild test robot
2016-03-23T20:29:21Z

[-- Attachment #1: Type: text/plain, Size: 4748 bytes --]

Hi Mauro,

[auto build test WARNING on sailus-media/master]
[cannot apply to v4.5 next-20160323]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url: https://github.com/0day-ci/linux/commits/Mauro-Carvalho-Chehab/media-device-Simplify-compat32-logic/20160324-033012
base: git://linuxtv.org/media_tree.git master
reproduce: make htmldocs

All warnings (new ones prefixed by >>):

include/linux/init.h:1: warning: no structured comments found
kernel/sys.c:1: warning: no structured comments found
drivers/dma-buf/seqno-fence.c:1: warning: no structured comments found
drivers/dma-buf/reservation.c:1: warning: no structured comments found
include/linux/reservation.h:1: warning: no structured comments found
include/media/media-device.h:626: warning: No description found for parameter 'mdev'
include/media/media-device.h:626: warning: Excess function parameter 'dev' description in 'media_device_unregister_devres'
include/media/media-device.h:626: warning: No description found for parameter 'mdev'
include/media/media-device.h:626: warning: Excess function parameter 'dev' description in 'media_device_unregister_devres'
>> include/media/media-devnode.h:102: warning: No description found for parameter 'media_dev'
>> include/media/media-devnode.h:126: warning: No description found for parameter 'mdev'
>> include/media/media-devnode.h:126: warning: Excess function parameter 'media_dev' description in 'media_devnode_register'

vim +/media_dev +102 include/media/media-devnode.h

cf4b9211 Laurent Pinchart 2009-12-09 96 /* device info */
cf4b9211 Laurent Pinchart 2009-12-09 97 int minor;
cf4b9211 Laurent Pinchart 2009-12-09 98 unsigned long flags; /* Use bitops to access flags */
cf4b9211 Laurent Pinchart 2009-12-09 99
cf4b9211 Laurent Pinchart 2009-12-09 100 /* callbacks */
7aca681b Mauro Carvalho Chehab 2016-03-23 101 void (*release)(struct media_devnode *devnode);
cf4b9211 Laurent Pinchart 2009-12-09 @102 };
cf4b9211 Laurent Pinchart 2009-12-09 103
cf4b9211 Laurent Pinchart 2009-12-09 104 /* dev to media_devnode */
cf4b9211 Laurent Pinchart 2009-12-09 105 #define to_media_devnode(cd) container_of(cd, struct media_devnode, dev)
cf4b9211 Laurent Pinchart 2009-12-09 106
fe3c565e Mauro Carvalho Chehab 2015-12-13 107 /**
fe3c565e Mauro Carvalho Chehab 2015-12-13 108 * media_devnode_register - register a media device node
fe3c565e Mauro Carvalho Chehab 2015-12-13 109 *
adc4daf1 Mauro Carvalho Chehab 2016-03-23 110 * @media_dev: struct media_device we want to register a device node
adc4daf1 Mauro Carvalho Chehab 2016-03-23 111 * @devnode: the device node to unregister
fe3c565e Mauro Carvalho Chehab 2015-12-13 112 * @owner: should be filled with %THIS_MODULE
fe3c565e Mauro Carvalho Chehab 2015-12-13 113 *
fe3c565e Mauro Carvalho Chehab 2015-12-13 114 * The registration code assigns minor numbers and registers the new device node
fe3c565e Mauro Carvalho Chehab 2015-12-13 115 * with the kernel. An error is returned if no free minor number can be found,
fe3c565e Mauro Carvalho Chehab 2015-12-13 116 * or if the registration of the device node fails.
fe3c565e Mauro Carvalho Chehab 2015-12-13 117 *
fe3c565e Mauro Carvalho Chehab 2015-12-13 118 * Zero is returned on success.
fe3c565e Mauro Carvalho Chehab 2015-12-13 119 *
fe3c565e Mauro Carvalho Chehab 2015-12-13 120 * Note that if the media_devnode_register call fails, the release() callback of
fe3c565e Mauro Carvalho Chehab 2015-12-13 121 * the media_devnode structure is *not* called, so the caller is responsible for
fe3c565e Mauro Carvalho Chehab 2015-12-13 122 * freeing any data.
fe3c565e Mauro Carvalho Chehab 2015-12-13 123 */
adc4daf1 Mauro Carvalho Chehab 2016-03-23 124 int __must_check media_devnode_register(struct media_device *mdev,
adc4daf1 Mauro Carvalho Chehab 2016-03-23 125 struct media_devnode *devnode,
85de721c Sakari Ailus 2013-12-12 @126 struct module *owner);
fe3c565e Mauro Carvalho Chehab 2015-12-13 127
fe3c565e Mauro Carvalho Chehab 2015-12-13 128 /**
fe3c565e Mauro Carvalho Chehab 2015-12-13 129 * media_devnode_unregister - unregister a media device node

:::::: The code at line 102 was first introduced by commit
:::::: cf4b9211b5680cd9ca004232e517fb7ec5bf5316 [media] media: Media device node support

:::::: TO: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
:::::: CC: Mauro Carvalho Chehab <mchehab@redhat.com>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 6251 bytes --]