Messages in this thread | | | Date | Tue, 18 Jul 2017 14:31:36 +0100 | From | Mark Brown <> | Subject | Re: [PATCH v2 3/9] regulator: mt6380: Add support for MT6380 |
| |
On Tue, Jul 18, 2017 at 05:49:23PM +0800, sean.wang@mediatek.com wrote:
> + if (!info->modeset_mask) { > + dev_err(&rdev->dev, "regulator %s doesn't support set_mode\n", > + info->desc.name); > + return -EINVAL; > + }
This is mostly fine but the mode operations all have this code - if the regulator doesn't have modesetting facilities it just shouldn't have the ops so the core can handle things in a standard fashion. It'd mean defining a separate set of operations for those regulators but that's fine. [unhandled content-type:application/pgp-signature] | |