Re: [patch v2 21/35] Hexagon: Add SMP support

Richard Kuo
2011-08-30T20:50:24Z

On Tue, Aug 30, 2011 at 12:30:36PM -0700, Joe Perches wrote:
> > + unsigned long msg = msg = ffz(~*ops);
>
> Really? That's unlikely to be the right code.
> Maybe just one msg?

Whoah, you're right. Although the code does actually work. Just need
one msg though.

> > + void (*func) (void *info) =
> > + data->func;
>
> No need to wrap either of these.

OK, fixed.

Thanks,
Richard Kuo

--

Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.