Re: [PATCH V2] genirq/affinity: add helper of irq_affinity_calc_sets

From: Christoph Hellwig
Date: Tue Jul 20 2021 - 03:53:27 EST


> +int irq_affinity_calc_sets(unsigned int affvecs, struct irq_affinity *affd)
> +{
> + /*
> + * Simple invocations do not provide a calc_sets() callback. Call
> + * the generic one.
> + */
> + if (!affd->calc_sets)
> + default_calc_sets(affd, affvecs);
> + else
> + affd->calc_sets(affd, affvecs);

Nit: avoid pointless negations:

if (affd->calc_sets)
affd->calc_sets(affd, affvecs);
else
default_calc_sets(affd, affvecs);

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>