On 26.02.2024 18:39, Oleksii Kurochko wrote:
> Signed-off-by: Oleksii Kurochko <oleksii.kuroc...@gmail.com>

Acked-by: Jan Beulich <jbeul...@suse.com>
with one remark:

> +/*
> + * The following functions are defined in common/irq.c, which will be built 
> in
> + * the next commit, so these changes will be removed there.
> + */
> +
> +void cf_check irq_actor_none(struct irq_desc *desc)
> +{
> +    BUG_ON("unimplemented");
> +}
> +
> +unsigned int cf_check irq_startup_none(struct irq_desc *desc)
> +{
> +    BUG_ON("unimplemented");
> +
> +    return 0;
> +}

Neither patch descriptions nor comments should mention "the next commit" or
anything alike. You want to describe things without making assumptions that
two successively submitted patches are also committed in direct succession.

Jan

Reply via email to