[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230714152701.GA1300718@dev-arch.thelio-3990X>
Date: Fri, 14 Jul 2023 08:27:01 -0700
From: Nathan Chancellor <nathan@...nel.org>
To: Miquel Raynal <miquel.raynal@...tlin.com>
Cc: Yangtao Li <frank.li@...o.com>,
Kyungmin Park <kyungmin.park@...sung.com>,
Richard Weinberger <richard@....at>,
Vignesh Raghavendra <vigneshr@...com>,
linux-mtd@...ts.infradead.org, linux-kernel@...r.kernel.org,
llvm@...ts.linux.dev
Subject: Re: [PATCH 12/18] mtd: nand: omap: Use
devm_platform_get_and_ioremap_resource()
On Thu, Jul 13, 2023 at 09:57:51AM +0200, Miquel Raynal wrote:
> On Fri, 2023-07-07 at 04:06:16 UTC, Yangtao Li wrote:
> > Convert platform_get_resource(), devm_ioremap_resource() to a single
> > call to devm_platform_get_and_ioremap_resource(), as this is exactly
> > what this function does.
> >
> > Signed-off-by: Yangtao Li <frank.li@...o.com>
>
> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.
Please apply the updated version at [1], the current version is
obviously not correct, as clang points out in next-20230714:
drivers/mtd/nand/onenand/onenand_omap2.c:470:2: warning: variable 'c' is uninitialized when used here [-Wuninitialized]
470 | c->onenand.base = devm_platform_get_and_ioremap_resource(pdev, 0, &res);
| ^
drivers/mtd/nand/onenand/onenand_omap2.c:465:25: note: initialize the variable 'c' to silence this warning
465 | struct omap2_onenand *c;
| ^
| = NULL
1 warning generated.
KernelCI also reported this at [2]. I apologize if this has already been
reported or fixed locally, I did not see your branch updated.
[1]: https://lore.kernel.org/20230713104422.29222-1-frank.li@vivo.com/
[2]: https://lore.kernel.org/64b137dc.170a0220.b8a13.feee@mx.google.com/
Cheers,
Nathan
Powered by blists - more mailing lists