Re: [PATCH 04/39] mtd: nand: denali: remove more unused struct members

From: Boris Brezillon
Date: Sun Nov 27 2016 - 10:13:18 EST


On Sun, 27 Nov 2016 03:05:50 +0900
Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote:

Please add a description here.

Also, this commit tends to validate my fears: you should have wait for
the full rework/cleanup to be done before submitting the first round of
cleanups. Indeed, commit c4ae0977f57d ("mtd: nand: denali: remove unused
struct member denali_nand_info::idx") was removing one of these unused
fields, leaving 2 of them behind.

While I like when things I clearly separated in different commits, when
you push the logic too far, you end up with big series which are not
necessarily easier to review, and several commits that are achieving
the same goal...

> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> ---
>
> drivers/mtd/nand/denali.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/denali.h b/drivers/mtd/nand/denali.h
> index fd1ae08..4fad43b 100644
> --- a/drivers/mtd/nand/denali.h
> +++ b/drivers/mtd/nand/denali.h
> @@ -407,7 +407,6 @@ struct denali_nand_info {
> struct nand_buf buf;
> struct device *dev;
> int total_used_banks;
> - uint32_t block; /* stored for future use */
> uint16_t page;
> void __iomem *flash_reg; /* Mapped io reg base address */
> void __iomem *flash_mem; /* Mapped io reg base address */
> @@ -416,7 +415,6 @@ struct denali_nand_info {
> struct completion complete;
> spinlock_t irq_lock;
> uint32_t irq_status;
> - int irq_debug_array[32];
> int irq;
>
> uint32_t devnum; /* represent how many nands connected */