Re: [cocci] [PATCH v4 2/5] arm64: tlb: Convert the opencoded field modify

From: Markus Elfring
Date: Thu Jun 12 2025 - 16:17:38 EST


I see further refinement possibilities for such a change description.


> Replace below code with the wrapper FIELD_MODIFY(MASK, &reg, val)
> - reg &= ~MASK;
> - reg |= FIELD_PREP(MASK, val);

* How do you think about to omit leading minus characters?

* Subsequent blank line?


> More information about semantic patching is available at
> http://coccinelle.lip6.fr/

I suggest to omit this information here (and in similar patches).

Regards,
Markus