Hello, Miquèl!<div>Sure. I agree.</div><div><br /></div><div><br /></div><div class="86e3a5cbc83439f6js-compose-signature"><div>-- </div><div>С Уважением,<br /> </div><div><img src="http://www.itb.spb.ru/img/podp.png" style="float:left" />Дембский Игорь Игоревич</div><div> </div><div>ООО «Инновационные Технологии в Бизнесе»</div><div>тел. +7 (812) 335-01-45 </div><div>e-mail: <a href="mailto:dii@itb.spb.ru" rel="noopener noreferrer">dii@itb.spb.ru</a></div><div>Мы в соц. сетях:</div><div><a href="https://g.page/itbspb" rel="noopener noreferrer"><img src="https://www.itb.spb.ru/bitrix/templates/itb/img/social/google.png" /></a><a href="https://vk.com/itb_spb" rel="noopener noreferrer"><img src="https://www.itb.spb.ru/bitrix/templates/itb/img/social/vkontakte.png" /></a><a href="https://twitter.com/itb_spb" rel="noopener noreferrer"><img src="https://www.itb.spb.ru/bitrix/templates/itb/img/social/twitter.png" /></a><a href="https://yandex.ru/profile/14089663908?lr=2" rel="noopener noreferrer"><img src="https://www.itb.spb.ru/bitrix/templates/itb/img/social/yandex.png" /></a><a href="https://youtube.com/channel/UCRUs5Ranea-kR0KzBhktGEw" rel="noopener noreferrer"><img src="http://itb.spb.ru/bitrix/templates/itb/img/social/youtube.png" /></a></div></div><div><br /></div><br /><br /><div>01.09.2023, 16:42, "Miquel Raynal" <miquel.raynal@bootlin.com>:</div><blockquote><p>Hi Igor,<br /><br /><a href="mailto:dii@itb.spb.ru">dii@itb.spb.ru</a> wrote on Fri,  1 Sep 2023 16:36:41 +0300:<br /><br /></p><blockquote class="210e7a848e8fcb45wmi-quote"> Return value of mtdtest_scan_for_bad_() is always 0. So it causes senseless<br /> checks in some functions such as mtd_subpagetest_init().<br /> <br /> Fixes: 084db4b020c7 ("mtd: tests: introduce helper functions")<br /> Signed-off-by: Dembskiy Igor <<a href="mailto:dii@itb.spb.ru">dii@itb.spb.ru</a>><br /> ---<br />  drivers/mtd/tests/oobtest.c     | 2 --<br />  drivers/mtd/tests/pagetest.c    | 2 --<br />  drivers/mtd/tests/readtest.c    | 2 --<br />  drivers/mtd/tests/speedtest.c   | 3 +--<br />  drivers/mtd/tests/stresstest.c  | 2 --<br />  drivers/mtd/tests/subpagetest.c | 2 --<br />  drivers/mtd/tests/torturetest.c | 2 --<br />  7 files changed, 1 insertion(+), 14 deletions(-)<br /> <br /> diff --git a/drivers/mtd/tests/oobtest.c b/drivers/mtd/tests/oobtest.c<br /> index 13fed398937e..976f40a5656e 100644<br /> --- a/drivers/mtd/tests/oobtest.c<br /> +++ b/drivers/mtd/tests/oobtest.c<br /> @@ -399,8 +399,6 @@ static int __init mtd_oobtest_init(void)<br />          pr_info("test 1 of 5\n");<br />  <br />          err = mtdtest_erase_good_eraseblocks(mtd, bbt, 0, ebcnt);<br /> -     if (err)<br /> -         goto out;<br /></blockquote><p><br />What's the point of keeping "err =" or even keeping a return value if<br />void would be more appropriate?<br /><br />Thanks,<br />Miquèl<br /></p></blockquote>