On Mon, Nov 16, 2020 at 12:30 PM Lee Jones <[email protected]> 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 <[email protected]> > Cc: "Christian König" <[email protected]> > Cc: David Airlie <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Lee Jones <[email protected]>
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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel

