Re: [PATCH 02/17] scsi: bfa: Rename 'timer_mod' to 'timer_module'

[Linus Torvalds]

On Mon, 14 Apr 2025 at 03:23, Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
> We'd like to introduce timer_mod() in the kernel, so make
> sure the namespace is clear.

No no, this is horribly wrong.
> -       use_idlect1 = omap_dm_timer_modify_idlect_mask(use_idlect1);
> +       use_idlect1 = omap_dm_timer_moduleify_idlect_mask(use_idlect1);

This was clearly all done with some completely bogus
search-and-replace, and the end result is nonsensical.

That "timer_mod" has *NOTHING* to do with a "module". It's "modify".

> -enum s3c64xx_timer_mode {
> +enum s3c64xx_timer_modulee {

Same thing. It's not "mod" as in "module" it's "mode".

> -                       apic->lapic_timer.timer_mode_mask = 3 << 17;
> +                       apic->lapic_timer.timer_modulee_mask = 3 << 17;

More nonsense.

> -       return apic->lapic_timer.timer_mode == APIC_LVT_TIMER_ONESHOT;
> +       return apic->lapic_timer.timer_modulee == APIC_LVT_TIMER_ONESHOT;

More nonsense.

> -       ti_32k_timer_module_init(np, ti_32k_timer.base);
> +       ti_32k_timer_moduleule_init(np, ti_32k_timer.base);

Even more crazy mindless and completely incorrect changes.

> diff --git a/drivers/scsi/bfa/bfa_ioc.c b/drivers/scsi/bfa/bfa_ioc.c
> index aa68d61a2d0d..4c11e3738f36 100644
> --- a/drivers/scsi/bfa/bfa_ioc.c
> +++ b/drivers/scsi/bfa/bfa_ioc.c
>  #define bfa_ioc_timer_start(__ioc)                                     \
> -       bfa_timer_begin((__ioc)->timer_mod, &(__ioc)->ioc_timer,        \
> +       bfa_timer_begin((__ioc)->timer_module, &(__ioc)->ioc_timer,     \

As far as I can tell, this is the *only* correct conversion, with 95%
of the patch just being complete garbage.

Stop doing search-and-replace.

              Linus