Hi, On Tuesday, April 3, 2012, Akinobu Mita <akinobu.m...@gmail.com> wrote: > Use for_each_set_bit_from to iterate over all the set bit in a memory > region. > > Signed-off-by: Akinobu Mita <akinobu.m...@gmail.com> > Cc: Sekhar Nori <nsek...@ti.com> > Cc: Kevin Hilman <khil...@ti.com> > Cc: davinci-linux-open-source@linux.davincidsp.com > Cc: Russell King <li...@arm.linux.org.uk> > Cc: linux-arm-ker...@lists.infradead.org > --- > arch/arm/mach-davinci/dma.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/mach-davinci/dma.c b/arch/arm/mach-davinci/dma.c > index fd33919..2a41339 100644 > --- a/arch/arm/mach-davinci/dma.c > +++ b/arch/arm/mach-davinci/dma.c > @@ -557,9 +557,9 @@ static int reserve_contiguous_slots(int ctlr, unsigned int id, > if (i == edma_cc[ctlr]->num_slots) > stop_slot = i; > > - for (j = start_slot; j < stop_slot; j++) > - if (test_bit(j, tmp_inuse)) > - clear_bit(j, edma_cc[ctlr]->edma_inuse); > + j = start_slot; > + for_each_set_bit_from(j, tmp_inuse, stop_slot);
Is the semicolon at the end of the line correct? Regards, Christian > + clear_bit(j, edma_cc[ctlr]->edma_inuse); > > if (count) > return -EBUSY; > -- > 1.7.4.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ >
_______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
