On Tue, Jan 31, 2017 at 02:09:53PM -0800, Tony Lindgren wrote:
> * Mark Brown <broo...@kernel.org> [170131 13:14]:
> > On Mon, Jan 30, 2017 at 04:12:41PM -0800, Tony Lindgren wrote:
> > > + error = regmap_update_bits(rdev->regmap, rdev->desc->enable_reg,
> > > + rdev->desc->enable_mask,
> > > + rdev->desc->enable_val);
> > > + if (error)
> > > + return error;
> > > +
> > > + if (rdev->desc->enable_val & CPCAP_REG_OFF_MODE_SEC) {
> > > + error = regmap_update_bits(rdev->regmap, regulator->assign_reg,
> > > + regulator->assign_mask,
> > > + regulator->assign_mask);
> > > + if (error)
> > > + return error;
> > > + }> > What does this second register bit do? > My guess is that it triggers regulator idle modes in the PMIC based > on the SoC off mode. There's no documentation for this thing.. > I've just kept what the Motorola Linux kernel tree does here and added > some comments to CPCAP_REG_OFF_MODE_SEC. > I'll address the rest of the comments and post v2, thanks for looking. OK, thanks - comments should cover it here.
signature.asc
Description: PGP signature
