From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4EF53ECE58C for ; Mon, 7 Oct 2019 16:07:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1D41320700 for ; Mon, 7 Oct 2019 16:07:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727791AbfJGQHV (ORCPT ); Mon, 7 Oct 2019 12:07:21 -0400 Received: from mail-ed1-f65.google.com ([209.85.208.65]:38983 "EHLO mail-ed1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727711AbfJGQHV (ORCPT ); Mon, 7 Oct 2019 12:07:21 -0400 Received: by mail-ed1-f65.google.com with SMTP id a15so12978210edt.6; Mon, 07 Oct 2019 09:07:19 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=NkOSCy7RY8YMsa1CzUdb6qn+7D24sPQ2vGSBGYLAcmM=; b=M15OPNPR5bfk/4i4QtM0eZsiRxLLD+PgaFOJhibsb3HMman5Wa+Bs4Ft5LKBUNnZcH PkkKwNntFiS53d3RLiyuSc6xkz39Eep3yneI43xmEv+/bMPM4Sx6slZEcAWvb4yfxvjg KFPuReaYulhLv17/StR62VgAuioJsbZGBHEDPeGuiHGrsX4rXPyMj/6Iwrtm5whvwWFD JT/mA+AeG1wDQKN+XZASthsyYOWDYYHphyTgTJKvg7V/MhOWa1mKjA/qNi6e1cAiRI8F 9RqUZLCQQ9qASF3SzNvbURAGNhW4JJCfJMTXIVOexP1RiWkhqfdjXV4Eo9/ToJPcxWh4 Nr2g== X-Gm-Message-State: APjAAAXX8LZ6YTiYevAUz749883YX0dU10PGf6zhwjkY/kZkchVirU/C dbRjrIX5jetcQaSe87kjjyjlx78cuHY= X-Google-Smtp-Source: APXvYqyz++kR3+7DzCmTgDLj8RgfpVMZQZUzoljOtxG963IPIwTlnydwl9sRiBm33Zkq/5SqMr5sAA== X-Received: by 2002:a17:906:828c:: with SMTP id h12mr24066464ejx.155.1570464438292; Mon, 07 Oct 2019 09:07:18 -0700 (PDT) Received: from mail-wr1-f53.google.com (mail-wr1-f53.google.com. [209.85.221.53]) by smtp.gmail.com with ESMTPSA id e5sm3466828edl.8.2019.10.07.09.07.17 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 07 Oct 2019 09:07:18 -0700 (PDT) Received: by mail-wr1-f53.google.com with SMTP id j18so15198643wrq.10; Mon, 07 Oct 2019 09:07:17 -0700 (PDT) X-Received: by 2002:adf:e50e:: with SMTP id j14mr23607757wrm.178.1570464437624; Mon, 07 Oct 2019 09:07:17 -0700 (PDT) MIME-Version: 1.0 References: <20191002112545.58481-1-icenowy@aosc.io> <20191002112545.58481-3-icenowy@aosc.io> In-Reply-To: <20191002112545.58481-3-icenowy@aosc.io> From: Chen-Yu Tsai Date: Tue, 8 Oct 2019 00:07:05 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [linux-sunxi] [PATCH 2/2] power: supply: axp20x_usb_power: add applied max Vbus support for AXP813 To: Icenowy Zheng , Sebastian Reichel Cc: "open list:THERMAL" , linux-kernel , linux-sunxi Content-Type: text/plain; charset="UTF-8" Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org Hi, On Wed, Oct 2, 2019 at 7:27 PM Icenowy Zheng wrote: > > AXP813 PMIC has two Vbus maximum value settings -- one is the default > value, which is currently the only supported one; the other is the > really applied value, which is set according to the default value if the > BC detection module detected a charging port, or 500mA if no charging > port is detected. > > Add support for reading and writing of the really applied Vbus maxmium > value. Interestingly it has a larger range than the default value. > > Signed-off-by: Icenowy Zheng > --- > drivers/power/supply/axp20x_usb_power.c | 132 +++++++++++++++++++++++- > 1 file changed, 129 insertions(+), 3 deletions(-) > > diff --git a/drivers/power/supply/axp20x_usb_power.c b/drivers/power/supply/axp20x_usb_power.c > index 5f0a5722b19e..905668a2727f 100644 > --- a/drivers/power/supply/axp20x_usb_power.c > +++ b/drivers/power/supply/axp20x_usb_power.c [...] > @@ -354,6 +451,9 @@ static int axp20x_usb_power_set_property(struct power_supply *psy, > val->intval); > return axp20x_usb_power_set_current_max(power, val->intval); > > + case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT: > + return axp20x_usb_power_set_input_current_limit(power, val->intval); > + So I think there are two things that should be adjusted. First, we should be using POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT for all PMICs. As far as the sysfs documents go, CURRENT_MAX is read-only, and should refer to the hard limit the hardware can support, i.e. maximum power ratings. INPUT_CURRENT_LIMIT and INPUT_VOLTAGE_LIMIT are for configurable upper and lower limits respectively. Sebastian, is my understanding of this correct? We already use INPUT_CURRENT_LIMIT for the AXP813 in the axp20x-ac driver, and it would be nice to have both drivers expose the same attributes. Second, since the value set in register 0x35 is the one that actually has an effect, as opposed to just being a default, we should just use that one. Could you restructure the series based on what I described, with a new patch 1 switching from CURRENT_MAX to INPUT_CURRENT_LIMIT, and then this patch as patch 2? And both patches should have Fixes tags and possibly CC stable so they get backported for people that are using stable kernels? And then the original patch 2 as patch 3. ChenYu > default: > return -EINVAL; > } > @@ -365,7 +465,8 @@ static int axp20x_usb_power_prop_writeable(struct power_supply *psy, > enum power_supply_property psp) > { > return psp == POWER_SUPPLY_PROP_VOLTAGE_MIN || > - psp == POWER_SUPPLY_PROP_CURRENT_MAX; > + psp == POWER_SUPPLY_PROP_CURRENT_MAX || > + psp == POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT; > } > > static enum power_supply_property axp20x_usb_power_properties[] = { > @@ -386,6 +487,15 @@ static enum power_supply_property axp22x_usb_power_properties[] = { > POWER_SUPPLY_PROP_CURRENT_MAX, > }; > > +static enum power_supply_property axp813_usb_power_properties[] = { > + POWER_SUPPLY_PROP_HEALTH, > + POWER_SUPPLY_PROP_PRESENT, > + POWER_SUPPLY_PROP_ONLINE, > + POWER_SUPPLY_PROP_VOLTAGE_MIN, > + POWER_SUPPLY_PROP_CURRENT_MAX, > + POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT, > +}; > + > static const struct power_supply_desc axp20x_usb_power_desc = { > .name = "axp20x-usb", > .type = POWER_SUPPLY_TYPE_USB, > @@ -406,6 +516,16 @@ static const struct power_supply_desc axp22x_usb_power_desc = { > .set_property = axp20x_usb_power_set_property, > }; > > +static const struct power_supply_desc axp813_usb_power_desc = { > + .name = "axp20x-usb", > + .type = POWER_SUPPLY_TYPE_USB, > + .properties = axp813_usb_power_properties, > + .num_properties = ARRAY_SIZE(axp813_usb_power_properties), > + .property_is_writeable = axp20x_usb_power_prop_writeable, > + .get_property = axp20x_usb_power_get_property, > + .set_property = axp20x_usb_power_set_property, > +}; > + > static int configure_iio_channels(struct platform_device *pdev, > struct axp20x_usb_power *power) > { > @@ -487,10 +607,16 @@ static int axp20x_usb_power_probe(struct platform_device *pdev) > usb_power_desc = &axp20x_usb_power_desc; > irq_names = axp20x_irq_names; > } else if (power->axp20x_id == AXP221_ID || > - power->axp20x_id == AXP223_ID || > - power->axp20x_id == AXP813_ID) { > + power->axp20x_id == AXP223_ID) { > usb_power_desc = &axp22x_usb_power_desc; > irq_names = axp22x_irq_names; > + } else if (power->axp20x_id == AXP813_ID) { > + usb_power_desc = &axp813_usb_power_desc; > + irq_names = axp22x_irq_names; > + > + /* Enable USB Battery Charging specification detection */ > + regmap_update_bits(axp20x->regmap, AXP288_BC_GLOBAL, > + AXP813_BC_EN, AXP813_BC_EN); This seems like a duplicate of > } else { > dev_err(&pdev->dev, "Unsupported AXP variant: %ld\n", > axp20x->variant); > -- > 2.21.0 > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com. > To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/20191002112545.58481-3-icenowy%40aosc.io.