[PATCH 06/13] i2c: nomadik: use bitops helpers

Linus Walleij linus.walleij at linaro.org
Mon Feb 19 06:16:41 PST 2024


On Thu, Feb 15, 2024 at 5:52 PM Théo Lebrun <theo.lebrun at bootlin.com> wrote:

> Constant register bit fields are declared using hardcoded hex values;
> replace them by calls to BIT() and GENMASK(). Replace custom GEN_MASK()
> macro by the generic FIELD_PREP(). Replace manual bit manipulations by
> the generic FIELD_GET() macro.
>
> Signed-off-by: Théo Lebrun <theo.lebrun at bootlin.com>

I'm a fan of this style.
Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list