Messages in this thread | | | Date | Wed, 3 Jul 2013 19:05:11 +0100 | From | Mark Brown <> | Subject | Re: [PATCH 3/4] mfd: s2mps11: Add device tree support |
| |
On Mon, Jun 24, 2013 at 04:50:57PM +0530, Yadwinder Singh Brar wrote:
> + regulator-ramp-delay for BUCKs = [6250/12500/25000(default)/50000] uV/us
> + Optional property for BUCK[2/3/4/6] only: > + regulator-ramp-disable: boolean, disables ramp delay on hardware.
If these were device specific bindings they ought to be namespaced as samsung, or similar. However I think they are generic enough that they could just be added to the standard regulator bindings, ramp rates are common enough and units of time are a simple way to express them. I'd also suggest just using a ramp delay of zero for no ramp, it's simpler and will probably fall naturally out of the bindings.
Please just send a separate patch adding these to the generic regulator binding document instead of this one, we can add generic code/ops for this later. [unhandled content-type:application/pgp-signature] | |