Andrew, you'll want to drop this one from your series b/c I checked in the simpler version. thanks, Len Brown, Intel Open Source Technology Center On Wed, 4 Mar 2009, akpm@xxxxxxxxxxxxxxxxxxxx wrote: > From: Roel Kluin <roel.kluin@xxxxxxxxx> > > With for(i = 0; i < n; i++) { ... } i reaches n. > > Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx> > Acked-by: <yakui.zhao@xxxxxxxxx> > Cc: Len Brown <lenb@xxxxxxxxxx> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > --- > > drivers/acpi/processor_throttling.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff -puN drivers/acpi/processor_throttling.c~acpi-get_throttling_state-cannot-be-larger-state_count drivers/acpi/processor_throttling.c > --- a/drivers/acpi/processor_throttling.c~acpi-get_throttling_state-cannot-be-larger-state_count > +++ a/drivers/acpi/processor_throttling.c > @@ -785,7 +785,7 @@ static int acpi_get_throttling_state(str > if (tx->control == value) > break; > } > - if (i > pr->throttling.state_count) > + if (i >= pr->throttling.state_count) > i = -1; > return i; > } > _ > -- 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