Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751324AbbD3HyD (ORCPT ); Thu, 30 Apr 2015 03:54:03 -0400 Received: from mail-vn0-f44.google.com ([209.85.216.44]:34948 "EHLO mail-vn0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750957AbbD3Hx7 convert rfc822-to-8bit (ORCPT ); Thu, 30 Apr 2015 03:53:59 -0400 MIME-Version: 1.0 In-Reply-To: <1430338406-24012-1-git-send-email-anda-maria.nicolae@intel.com> References: <1430338406-24012-1-git-send-email-anda-maria.nicolae@intel.com> Date: Thu, 30 Apr 2015 16:53:56 +0900 X-Google-Sender-Auth: NRhjBM5TdlKCPRiIpaf2P-fTImI Message-ID: Subject: Re: [PATCHv2] power_supply: Add support for Richtek rt9455 battery charger From: Krzysztof Kozlowski To: Anda-Maria Nicolae Cc: sre@kernel.org, dbaryshkov@gmail.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, dwmw2@infradead.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2129 Lines: 41 2015-04-30 5:13 GMT+09:00 Anda-Maria Nicolae : > Based on the datasheet found here: > http://www.richtek.com/download_ds.jsp?p=RT9455 > > Updates from the previous version: > - replaced license GPLv2 with GPL > - replaced vendor prefix rt with richtek > - replaced uppercase properties names from devicetree with lowercase separated by dash properties names > - replaced I/O read/write API with regmap_read/write and regmap_field_read/write > - used kernel multiline comment > - used DELAYED_WORK and scheduled the works in system_power_efficient_wq. It is high probability that the device is in suspend state while charging (i.e. the user leaves the device to charge during night) and it is needed that the scheduled works to be executed as planned. I think when device is suspended (e.g. to RAM) no work will ever be executed. The timers (for delayed work) will not wake up the device... RTC could wake but this is different story. My proposal of deferred timers is affected by idle CPU time. Are you sure that you want to wake up from the system suspend? > - replaced struct power_supply_desc rt9455_charger_desc with static const struct power_supply_desc rt9455_charger_desc > - replaced if (IS_ERR_OR_NULL(info->charger)) with if (IS_ERR(info->charger)) > - replaced {"RTK9455", 0} with { "RTK9455", 0 } > - removed .owner = THIS_MODULE > > Signed-off-by: Anda-Maria Nicolae You missed some of my comments. I mentioned wrong error paths around put_usb_phy in probe. They do not seem to be fixed... Just one hint - mostly new bindings are posted in separate patches at the beginning of the patchset. I actually don't mind. but separating them will probably give you higher chance of review from DT people. This is also mentioned in: Documentation/devicetree/bindings/submitting-patches.txt Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/