Messages in this thread | | | Date | Mon, 27 Aug 2018 12:01:43 +0200 | From | Pavel Machek <> | Subject | Re: [PATCH V5 3/8] backlight: qcom-wled: Add new properties for PMI8998 |
| |
Hi!
On Fri 2018-08-24 15:57:42, Kiran Gunda wrote: > Update the bindings with the new properties used for > PMI8998.
> Changes from V3: > - Removed the default values.
Why?
> +- qcom,current-limit-microamp > + Usage: optional > + Value type: <u32> > + Definition: uA; per-string current limit; value from 0 to 30000 with > + 2500 uA step. > > - qcom,current-boost-limit > Usage: optional > Value type: <u32> > Definition: mA; boost current limit. > For pm8941: one of: 105, 385, 525, 805, 980, 1260, 1400, > - 1680. Default: 805 mA > + 1680. > For pmi8998: one of: 105, 280, 450, 620, 970, 1150, 1300, > - 1500. Default: 970 mA > + 1500. >
I'd say that optional properties should list default values...?
> - qcom,ovp > Usage: optional > Value type: <u32> > Definition: V; Over-voltage protection limit; one of: > - 27, 29, 32, 35. default: 29V > + 27, 29, 32, 35. > This property is supported only for PM8941. >
Same here.
> +- qcom,ovp-millivolt > + Usage: optional > + Value type: <u32> > + Definition: mV; Over-voltage protection limit; > + For pmi8998: one of 18100, 19600, 29600, 31100 > + If this property is not specified for PM8941, it > + falls back to "qcom,ovp" property. > +
"voltage-limit-millivolt"? "ovp" is not really well known acronym. Pavel
-- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html [unhandled content-type:application/pgp-signature] | |