Re: [PATCH] mtd: Use DIV_ROUND_UP()

From: Boris Brezillon
Date: Thu Mar 29 2018 - 16:25:26 EST


On Sun, 25 Mar 2018 21:07:43 +0530
Arushi Singhal <arushisinghal19971997@xxxxxxxxx> wrote:

> The kernel.h macro DIV_ROUND_UP performs the computation
> (((n) + (d) - 1) / (d)) but is perhaps more readable.
>
> Signed-off-by: Arushi Singhal <arushisinghal19971997@xxxxxxxxx>

Applied.

Thanks,

Boris

> ---
> drivers/mtd/ftl.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/ftl.c b/drivers/mtd/ftl.c
> index a048429..61d5cf8 100644
> --- a/drivers/mtd/ftl.c
> +++ b/drivers/mtd/ftl.c
> @@ -425,8 +425,8 @@ static int prepare_xfer(partition_t *part, int i)
> }
>
> /* Write the BAM stub */
> - nbam = (part->BlocksPerUnit * sizeof(uint32_t) +
> - le32_to_cpu(part->header.BAMOffset) + SECTOR_SIZE - 1) / SECTOR_SIZE;
> + nbam = DIV_ROUND_UP(part->BlocksPerUnit * sizeof(uint32_t) +
> + le32_to_cpu(part->header.BAMOffset), SECTOR_SIZE);
>
> offset = xfer->Offset + le32_to_cpu(part->header.BAMOffset);
> ctl = cpu_to_le32(BLOCK_CONTROL);



--
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com