Re: [PATCH v5] resource: add a simple test for walk_iomem_res_desc()

From: Andy Shevchenko
Date: Thu Jun 06 2024 - 06:06:10 EST


On Wed, Jun 05, 2024 at 05:46:16PM -0700, Chia-I Wu wrote:
> On Wed, Jun 5, 2024 at 5:40 PM Chia-I Wu <olvaffe@xxxxxxxxx> wrote:
> >
> > This mainly tests that find_next_iomem_res() does not miss resources.

...

> > + walk_iomem_res_desc(IORES_DESC_NONE, IORESOURCE_SYSTEM_RAM,
> > + res[2].end + 1, res[3].end, &count,
> This should be from "res[3].end + 1" to "res[2].end". Not sure if I
> should resend or if you can make the fix when applying.

Please, slow down. You sent three (!) versions over a day, this is against
the recommendations. And it seems you want to learn a hard way the clear thing:
Hurrying just increases a chance of a mistake.

--
With Best Regards,
Andy Shevchenko