On Tuesday, July 25, 2017 04:13:41 PM Mika Westerberg wrote: > On Tue, Jul 25, 2017 at 12:37:31PM +0300, Mika Westerberg wrote: > > I wonder why we need to do this? > > > > + if (res.start < rentry->res->start) > > + rentry->res->start = res.start; > > + if (res.end > rentry->res->end) > > + rentry->res->end = res.end; > > > > IIRC we don't use rentry for anything else when we go out of the loop. > > Tuning those limits seems unnecessary. > > OK, Ryan pointed out that the later code then uses the adjusted rentry > so this is indeed needed. > > Acked-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> OK, patch applied, thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
