On Thu, Oct 16, 2008 at 06:05:22PM -0200, Werner Almesberger wrote:
> Balaji Rao wrote:
> > How does this look now ?
>> Lean and mean :-) You could make it even leaner by just calling
> adc_async_read from adc_sync_read. And I think you may have a
> double kfree in adc_sync_read/pcf50633_work, don't you ?
To call adc_async_read from adc_sync_read we need to pass the adc_request as
a callback param, which we won't be able to do as it's created inside
async_read. Sure, another layer of indirection can solve this :) But I
don't think it's worth it.
Yes, there's a double free out there! Thanks for noticing it.
- Balaji