> >> +     mutex_init(&max665x->iolock);
> >
> > What is this needed for?
> 
> It was done for consistency with the other mfd drivers (maxim), e.g.
> 8997 or 8998 as the closest resemblence in this family. Would you
> prefer me removing this mutex locker?

If you're not using mutexes, why would you need to initialise it?

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
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/

Reply via email to