From mboxrd@z Thu Jan 1 00:00:00 1970 From: "J, KEERTHY" Subject: Re: [PATCH 2/2] mfd: tps65218.c: Add input voltage options Date: Mon, 24 Dec 2018 14:31:24 +0530 Message-ID: References: <1545120356-7749-1-git-send-email-Christian.Hohnstaedt@wago.com> <1545120356-7749-3-git-send-email-Christian.Hohnstaedt@wago.com> <20181221110128.GO13248@dell> <20181223164030.GQ6707@atomide.com> <20181224063241.GB4846@dell> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20181224063241.GB4846@dell> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Lee Jones , Tony Lindgren Cc: Christian Hohnstaedt , Liam Girdwood , Mark Brown , Mark Rutland , Rob Herring , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-omap@vger.kernel.org List-Id: devicetree@vger.kernel.org On 12/24/2018 12:02 PM, Lee Jones wrote: > On Sun, 23 Dec 2018, Tony Lindgren wrote: > >> * Lee Jones [181221 11:01]: >>> On Tue, 18 Dec 2018, Christian Hohnstaedt wrote: >>> >>>> These options apply to all regulators in this chip. >>>> >>>> strict-supply-voltage: >>>> Set STRICT flag in CONFIG1 >>>> under-voltage-limit: >>>> Select 2.75, 2.95, 3.25 or 3.35 V UVLO in CONFIG1 >>>> under-voltage-hysteresis: >>>> Select 200mV or 400mV UVLOHYS in CONFIG2 >>>> >>>> Signed-off-by: Christian Hohnstaedt >>>> --- >>>> drivers/mfd/tps65218.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++ >>>> 1 file changed, 46 insertions(+) >>> >>> This needs a close review by Tony and any of the other OMAP guys. >> >> Adding Keerthy to Cc, he's probably the best person to review these >> changes. Thanks Tony. > > But you've dropped the patch, so all he has it a commit log to review? > Lee Jones, I am subscribed to linux-omap so i have the patch. I will review them. Thanks, Keerthy