[PATCH] block: fix kernel-doc for disk_force_media_change()

Randy Dunlap posted 1 patch 2 years, 4 months ago
block/disk-events.c |    1 -
1 file changed, 1 deletion(-)
[PATCH] block: fix kernel-doc for disk_force_media_change()
Posted by Randy Dunlap 2 years, 4 months ago
Drop one function parameter's kernel-doc comment since the parameter
was removed. This prevents a kernel-doc warning:

block/disk-events.c:300: warning: Excess function parameter 'events' description in 'disk_force_media_change'

Fixes: ab6860f62bfe ("block: simplify the disk_force_media_change interface")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Reported-by: kernel test robot <lkp@intel.com>
Closes: lore.kernel.org/r/202309060957.vfl0mUur-lkp@intel.com
Cc: Christoph Hellwig <hch@lst.de>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: linux-block@vger.kernel.org
---
 block/disk-events.c |    1 -
 1 file changed, 1 deletion(-)

diff -- a/block/disk-events.c b/block/disk-events.c
--- a/block/disk-events.c
+++ b/block/disk-events.c
@@ -290,7 +290,6 @@ EXPORT_SYMBOL(disk_check_media_change);
 /**
  * disk_force_media_change - force a media change event
  * @disk: the disk which will raise the event
- * @events: the events to raise
  *
  * Should be called when the media changes for @disk.  Generates a uevent
  * and attempts to free all dentries and inodes and invalidates all block
Re: [PATCH] block: fix kernel-doc for disk_force_media_change()
Posted by Jens Axboe 2 years, 4 months ago
On Mon, 25 Sep 2023 17:52:32 -0700, Randy Dunlap wrote:
> Drop one function parameter's kernel-doc comment since the parameter
> was removed. This prevents a kernel-doc warning:
> 
> block/disk-events.c:300: warning: Excess function parameter 'events' description in 'disk_force_media_change'
> 
> 

Applied, thanks!

[1/1] block: fix kernel-doc for disk_force_media_change()
      commit: a578a25339aca38e23bb5af6e3fc6c2c51f0215c

Best regards,
-- 
Jens Axboe
Re: [PATCH] block: fix kernel-doc for disk_force_media_change()
Posted by Christoph Hellwig 2 years, 4 months ago
Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>