> > > - do {
> > > - status = readl(host->base + DAVINCI_MMCST0);
> > > - if (status == 0)
> > > - break;
> > > - if (handle_core_command(host, status))
> > > - break;
> > > - } while (1);
> > > + status = readl(host->base + DAVINCI_MMCST0);
> > > + handle_core_command(host, status);
> > > return IRQ_HANDLED;
> >
> > Looks OK, but an even better cleanup would be to not have a
> > needless separate function for this ... :)
>
> I will suggest let this patch go through because of it has lot of other
> cleaning as well and we could
> take up this suggestion as well in the 2nd round of cleaning.
>
David,
Please ack this patch so that Kevin could accept it.
Purushotam
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
