Re: [PATCH v6 5/8] drivers: mtd: nand: use FIELD_PREP and GENMASK

From: Miquel Raynal
Date: Tue May 21 2024 - 08:53:44 EST


Hi,

quic_mdalam@xxxxxxxxxxx wrote on Tue, 21 May 2024 16:25:29 +0530:

> Use the bitfield macro FIELD_PREP, and GENMASK to
> do the shift and mask in one go. This makes the code
> more readable.
>
> Signed-off-by: Md Sadre Alam <quic_mdalam@xxxxxxxxxxx>
> ---
> Change in [v6]
>
> * Added FIELD_PREP() and GENMASK() macro

Maybe you want to include linux/bitfield.h now.

Otherwise that's a great improvement.

Thanks,
Miquèl