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=-16.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_2 autolearn=unavailable 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 20523C4338F for ; Fri, 23 Jul 2021 15:58:39 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E0CC460725 for ; Fri, 23 Jul 2021 15:58:38 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org E0CC460725 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=Huawei.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tu/lQYvYAQQeGEVf/zwOWm1nhnJ4UnmkxKVGNZZERf0=; b=H85N3dHIgxDxw9 fLtWTePWxbYnkAyvMjAnSVFn6bHwioosAvP9Ev5ug9UutjcvHWMFRGKFN23/3Vk+R5/eev+4SS3hs DAqEFwv1BoK3yMnadKRno3kll0MQSU5yk6zAayX1bX3zLkRHTNmRX11pJOLgY2O1msWMUbTsZLs9u AP7RNb2iM7V3AhclB/dm3wQzjAjZ1RsS8s3rvnhqegJGK1SyFNGJBNDdw5jCWQt/2LU9x32Ftvx6J +vQEI7C5cmkuMvM+ouIBMa72lG6906si+2GioTIRB9wPZjQpPPxJLDBUI4sHmVgnO5gC6ZEuKcU9x liFk19NnAAmY/VMcnowQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6xXw-005H8f-5j; Fri, 23 Jul 2021 15:57:08 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6xXo-005H54-Jd for linux-arm-kernel@lists.infradead.org; Fri, 23 Jul 2021 15:57:02 +0000 Received: from fraeml712-chm.china.huawei.com (unknown [172.18.147.206]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4GWYYW0jbQz6H7lk; Fri, 23 Jul 2021 23:45:19 +0800 (CST) Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by fraeml712-chm.china.huawei.com (10.206.15.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Fri, 23 Jul 2021 17:56:56 +0200 Received: from localhost (10.210.170.238) by lhreml710-chm.china.huawei.com (10.201.108.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2176.2; Fri, 23 Jul 2021 16:56:55 +0100 Date: Fri, 23 Jul 2021 16:56:29 +0100 From: Jonathan Cameron To: Billy Tsai CC: , , , , , , , , , , , , Subject: Re: [v2 8/8] iio: adc: aspeed: Support battery sensing. Message-ID: <20210723165629.00005610@Huawei.com> In-Reply-To: <20210723081621.29477-9-billy_tsai@aspeedtech.com> References: <20210723081621.29477-1-billy_tsai@aspeedtech.com> <20210723081621.29477-9-billy_tsai@aspeedtech.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; i686-w64-mingw32) MIME-Version: 1.0 X-Originating-IP: [10.210.170.238] X-ClientProxiedBy: lhreml704-chm.china.huawei.com (10.201.108.53) To lhreml710-chm.china.huawei.com (10.201.108.61) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210723_085701_018833_7A7884C7 X-CRM114-Status: GOOD ( 25.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 23 Jul 2021 16:16:21 +0800 Billy Tsai wrote: > In ast2600, ADC integrate dividing circuit at last input channel for > battery sensing. This patch use the dts property "battery-sensing" to > enable this feature makes the last channel of each adc can tolerance this feature allows the last channel of each ADC to tolerate a higher voltage than the reference voltage. > higher voltage than reference voltage. > > Signed-off-by: Billy Tsai This looks fine otherwise, but one more general question inline. Thanks, Jonathan > --- > drivers/iio/adc/aspeed_adc.c | 60 +++++++++++++++++++++++++++++++++--- > 1 file changed, 55 insertions(+), 5 deletions(-) > > diff --git a/drivers/iio/adc/aspeed_adc.c b/drivers/iio/adc/aspeed_adc.c > index 7e674b607e36..6c7e2bb7b1ac 100644 > --- a/drivers/iio/adc/aspeed_adc.c > +++ b/drivers/iio/adc/aspeed_adc.c > @@ -45,6 +45,9 @@ > #define ASPEED_ADC_REF_VOLTAGE_1200mV FIELD_PREP(ASPEED_ADC_REF_VOLTAGE, 1) > #define ASPEED_ADC_REF_VOLTAGE_EXT_HIGH FIELD_PREP(ASPEED_ADC_REF_VOLTAGE, 2) > #define ASPEED_ADC_REF_VOLTAGE_EXT_LOW FIELD_PREP(ASPEED_ADC_REF_VOLTAGE, 3) > +#define ASPEED_ADC_BATTERY_SENSING_DIV BIT(6) > +#define ASPEED_ADC_BATTERY_SENSING_DIV_2_3 FIELD_PREP(ASPEED_ADC_BATTERY_SENSING_DIV, 0) > +#define ASPEED_ADC_BATTERY_SENSING_DIV_1_3 FIELD_PREP(ASPEED_ADC_BATTERY_SENSING_DIV, 1) > #define ASPEED_ADC_CTRL_INIT_RDY BIT(8) > #define ASPEED_ADC_CH7_MODE BIT(12) > #define ASPEED_ADC_CH7_NORMAL FIELD_PREP(ASPEED_ADC_CH7_MODE, 0) > @@ -76,6 +79,11 @@ struct aspeed_adc_model_data { > unsigned int num_channels; > }; > > +struct adc_gain { > + u8 mult; > + u8 div; > +}; > + > struct aspeed_adc_data { > struct device *dev; > void __iomem *base; > @@ -87,6 +95,8 @@ struct aspeed_adc_data { > int vref; > u32 sample_period_ns; > int cv; > + bool battery_sensing; > + struct adc_gain battery_mode_gain; > }; > > #define ASPEED_CHAN(_idx, _data_reg_addr) { \ > @@ -185,14 +195,38 @@ static int aspeed_adc_read_raw(struct iio_dev *indio_dev, > int *val, int *val2, long mask) > { > struct aspeed_adc_data *data = iio_priv(indio_dev); > + u32 adc_engine_control_reg_val; > > switch (mask) { > case IIO_CHAN_INFO_RAW: > - *val = readw(data->base + chan->address) + data->cv; > - if (*val < 0) > - *val = 0; > - else if (*val >= ASPEED_ADC_MAX_RAW_DATA) > - *val = ASPEED_ADC_MAX_RAW_DATA; Not related to this patch, but can the device support a per channel reference selection? I.e. some channels use different internal references or external references from others? If so we should consider if it is necessary to expose that functionality in the dt-binding. > + if (data->battery_sensing && chan->channel == 7) { > + adc_engine_control_reg_val = > + readl(data->base + ASPEED_REG_ENGINE_CONTROL); > + writel(adc_engine_control_reg_val | > + ASPEED_ADC_CH7_BATTERY | > + ASPEED_ADC_BATTERY_SENSING_ENABLE, > + data->base + ASPEED_REG_ENGINE_CONTROL); > + /* > + * After enable battery sensing mode need to wait some time for adc stable > + * Experiment result is 1ms. > + */ > + mdelay(1); > + *val = readw(data->base + chan->address) + data->cv; > + if (*val < 0) > + *val = 0; > + else if (*val >= ASPEED_ADC_MAX_RAW_DATA) > + *val = ASPEED_ADC_MAX_RAW_DATA; > + *val = (*val * data->battery_mode_gain.mult) / > + data->battery_mode_gain.div; > + writel(adc_engine_control_reg_val, > + data->base + ASPEED_REG_ENGINE_CONTROL); > + } else { > + *val = readw(data->base + chan->address) + data->cv; > + if (*val < 0) > + *val = 0; > + else if (*val >= ASPEED_ADC_MAX_RAW_DATA) > + *val = ASPEED_ADC_MAX_RAW_DATA; > + } > return IIO_VAL_INT; > > case IIO_CHAN_INFO_SCALE: > @@ -392,6 +426,22 @@ static int aspeed_adc_probe(struct platform_device *pdev) > if (ret) > goto vref_config_error; > > + if (of_find_property(data->dev->of_node, "battery-sensing", NULL)) { > + if (model_data->version >= aspeed_adc_ast2600) { > + data->battery_sensing = 1; > + if (readl(data->base + ASPEED_REG_ENGINE_CONTROL) & > + ASPEED_ADC_BATTERY_SENSING_DIV_1_3) { > + data->battery_mode_gain.mult = 3; > + data->battery_mode_gain.div = 1; > + } else { > + data->battery_mode_gain.mult = 3; > + data->battery_mode_gain.div = 2; > + } > + } else > + dev_warn(&pdev->dev, > + "Failed to enable battey-sensing mode\n"); > + } > + > if (model_data->wait_init_sequence) { > adc_engine_control_reg_val = > readl(data->base + ASPEED_REG_ENGINE_CONTROL); _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel