From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Date: Fri, 18 May 2018 12:20:44 +0000 Subject: Re: [PATCH V1 2/5] backlight: qcom-wled: Add support for WLED4 peripheral Message-Id: List-Id: References: <1525341432-15818-1-git-send-email-kgunda@codeaurora.org> <1525341432-15818-3-git-send-email-kgunda@codeaurora.org> <20180507162004.GB2259@tuxbook-pro> <82fa847760309cb3382bf0a8da50162c@codeaurora.org> <7143faaec2d0b7b0fd7e0f3b512839d8@codeaurora.org> In-Reply-To: <7143faaec2d0b7b0fd7e0f3b512839d8@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Kiran Gunda Cc: Mark Rutland , devicetree@vger.kernel.org, Daniel Thompson , Bartlomiej Zolnierkiewicz , Jingoo Han , "linux-kernel@vger.kernel.org" , dri-devel , Bjorn Andersson , linux-fbdev@vger.kernel.org, Jacek Anaszewski , Pavel Machek , linux-arm-msm , Lee Jones , linux-arm-msm-owner@vger.kernel.org, Linux LED Subsystem On Thu, May 17, 2018 at 10:10 AM, wrote: > On 2018-05-17 18:01, Rob Herring wrote: >> >> On Thu, May 17, 2018 at 4:47 AM, wrote: >>> >>> On 2018-05-08 15:55, kgunda@codeaurora.org wrote: >>>> >>>> >>>> On 2018-05-07 21:50, Bjorn Andersson wrote: >>>>> >>>>> >>>>> On Thu 03 May 02:57 PDT 2018, Kiran Gunda wrote: >>>>> >>>>>> WLED4 peripheral is present on some PMICs like pmi8998 >>>>>> and pm660l. It has a different register map and also >>>>>> configurations are different. Add support for it. [...] >>>>>> + value: >>>>>> + For pm8941: from 0 to 25000 with 5000 ua step >>>>>> + Default 20000 uA >>>>>> + For pmi8998: from 0 to 30000 with 5000 ua step >>>>>> + Default 25000 uA. >>>>> >>>>> >>>>> >>>>> These values could be described just as well in mA, so keep the >>>>> original >>>>> unit - in particular since the boot-limit is in mA... >>>>> >>>> Ok. Will keep the original as is in the next series. >>> >>> >>> Here, I may have to go with the approach as in "qcom,ovp". Because for >>> pm8941 >>> the current step is 1 mA (I have wrongly mentioned as 5000uA here) and >>> for >>> PMI8998 >>> the current step is 2.5 mA. Hence, I will add another variable >>> "qcom,current-limit-ua" >>> just like "qcom,ovp-mv". >> >> >> Use unit suffixes defined in bindings/property-units.txt. > > Thanks for pointing it ! hope I can use "qcom,current-limit-microamp" and > "qcom,ovp-millivolt". I am asking this because i found only "-microvolt". > "-millivolt" is not present in the bindings you pointed. That's by design so everyone doesn't just pick whatever random units they like. Does microvolts not give you enough range? Rob