On Mon, Nov 16, 2020 at 12:30 PM Lee Jones <lee.jones@xxxxxxxxxx> wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/radeon_irq_kms.c:56: warning: Function parameter or member 'irq' not described in 'radeon_driver_irq_handler_kms' > drivers/gpu/drm/radeon/radeon_irq_kms.c:56: warning: Function parameter or member 'arg' not described in 'radeon_driver_irq_handler_kms' > drivers/gpu/drm/radeon/radeon_irq_kms.c:571: warning: Function parameter or member 'n' not described in 'radeon_irq_kms_set_irq_n_enabled' > drivers/gpu/drm/radeon/radeon_irq_kms.c:571: warning: Excess function parameter 'num' description in 'radeon_irq_kms_set_irq_n_enabled' > > Cc: Alex Deucher <alexander.deucher@xxxxxxx> > Cc: "Christian König" <christian.koenig@xxxxxxx> > Cc: David Airlie <airlied@xxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx> Applied. Thanks! Alex > --- > drivers/gpu/drm/radeon/radeon_irq_kms.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_irq_kms.c b/drivers/gpu/drm/radeon/radeon_irq_kms.c > index b86bc88ad4308..a242b6053d47e 100644 > --- a/drivers/gpu/drm/radeon/radeon_irq_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_irq_kms.c > @@ -43,11 +43,9 @@ > > #define RADEON_WAIT_IDLE_TIMEOUT 200 > > -/** > +/* > * radeon_driver_irq_handler_kms - irq handler for KMS > * > - * @int irq, void *arg: args > - * > * This is the irq handler for the radeon KMS driver (all asics). > * radeon_irq_process is a macro that points to the per-asic > * irq handler callback. > @@ -549,14 +547,14 @@ void radeon_irq_kms_disable_hpd(struct radeon_device *rdev, unsigned hpd_mask) > } > > /** > - * radeon_irq_kms_update_int_n - helper for updating interrupt enable registers > + * radeon_irq_kms_set_irq_n_enabled - helper for updating interrupt enable registers > * > * @rdev: radeon device pointer > * @reg: the register to write to enable/disable interrupts > * @mask: the mask that enables the interrupts > * @enable: whether to enable or disable the interrupt register > * @name: the name of the interrupt register to print to the kernel log > - * @num: the number of the interrupt register to print to the kernel log > + * @n: the number of the interrupt register to print to the kernel log > * > * Helper for updating the enable state of interrupt registers. Checks whether > * or not the interrupt matches the enable state we want. If it doesn't, then > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
