Messages in this thread | | | Date | Wed, 28 May 2014 11:07:03 +0800 | From | "Zhu, Lejun" <> | Subject | Re: [PATCH v3 1/4] mfd: intel_soc_pmic: Core driver |
| |
On 5/27/2014 11:35 PM, Lee Jones wrote: >> This patch provides the common code for the intel_soc_pmic MFD driver, such as read/write register and set up IRQ. (...) >> +/* >> +* Set and clear multiple bits of a PMIC register >> +*/ >> +int intel_soc_pmic_update(int reg, u8 val, u8 mask) >> +{ >> + int ret; >> + >> + mutex_lock(&pmic_lock); >> + >> + if (!pmic) >> + ret = -EIO; >> + else >> + ret = regmap_update_bits(pmic->regmap, reg, mask, val); >> + >> + mutex_unlock(&pmic_lock); >> + >> + return ret; >> +} >> +EXPORT_SYMBOL_GPL(intel_soc_pmic_update); > > I'm really not a fan of all these pointless agregation call-backs. I > see them as unesersary overhead. Just use the regmap API directly.
OK. I'll remove these wrappers from the MFD driver, seems no one likes them...
I'll fix the patch set as you suggested and resubmit. Thank you for reviewing this.
Best Regards Lejun
| |