From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756762Ab3HAPUk (ORCPT ); Thu, 1 Aug 2013 11:20:40 -0400 Received: from hqemgate14.nvidia.com ([216.228.121.143]:3589 "EHLO hqemgate14.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755700Ab3HAPUj (ORCPT ); Thu, 1 Aug 2013 11:20:39 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Thu, 01 Aug 2013 08:20:38 -0700 Message-ID: <51FA7FB9.8000209@nvidia.com> Date: Thu, 1 Aug 2013 21:03:13 +0530 From: Laxman Dewangan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 MIME-Version: 1.0 To: "balbi@ti.com" CC: "akpm@linux-foundation.org" , "grant.likely@linaro.org" , "rob.herring@calxeda.com" , "rob@landley.net" , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "rtc-linux@googlegroups.com" , "gg@slimlogic.co.uk" , "kishon@ti.com" , Stephen Warren , "pawel.moll@arm.com" , "Mark.Rutland@arm.com" , "ian.campbell@citrix.com" , "broonie@kernel.org" Subject: Re: [PATCH V2] drivers/rtc/rtc-palmas.c: support for backup battery charging References: <1375367471-29908-1-git-send-email-ldewangan@nvidia.com> <20130801150941.GA11286@radagast> In-Reply-To: <20130801150941.GA11286@radagast> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday 01 August 2013 08:39 PM, Felipe Balbi wrote: > * PGP Signed by an unknown key > > Hi, > > On Thu, Aug 01, 2013 at 08:01:11PM +0530, Laxman Dewangan wrote: > I wonder if this should be exposed via power_supply class, although it's > not really a power supply. So perhaps not needed :-s Yes, there is another charger module on the device which actually need the driver through power supply. Charger submodule used for charging the main battery through USB VBUS. > > looks good though (can't test as I don't have any boards available right > now): > > Reviewed-by: Felipe Balbi Thanks for review. >