Re: [PATCH 03/11] mtd: nand: denali: remove bogus comment about interrupt handler setup

From: Marek Vasut
Date: Sat Nov 12 2016 - 16:58:07 EST


On 11/09/2016 05:35 AM, Masahiro Yamada wrote:
> The interrupt handler is setup in denali_init(), not in
> denali_drv_init(). This comment is false.
>
> Such a comment adds no value, so just delete it instead of move.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>

Reviewed-by: Marek Vasut <marek.vasut@xxxxxxxxx>

> ---
>
> drivers/mtd/nand/denali.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c
> index 51ddb84..d6f1b29 100644
> --- a/drivers/mtd/nand/denali.c
> +++ b/drivers/mtd/nand/denali.c
> @@ -1436,7 +1436,6 @@ static int denali_ooblayout_free(struct mtd_info *mtd, int section,
> /* initialize driver data structures */
> static void denali_drv_init(struct denali_nand_info *denali)
> {
> - /* setup interrupt handler */
> /*
> * the completion object will be used to notify
> * the callee that the interrupt is done
>


--
Best regards,
Marek Vasut